Skip to content

Commit

Permalink
Skip the 4 failing tests in non-xattr because they rely on them
Browse files Browse the repository at this point in the history
  • Loading branch information
bbrks committed Jul 18, 2024
1 parent f5ab020 commit dc18ecd
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 0 deletions.
8 changes: 8 additions & 0 deletions db/import_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -894,6 +894,10 @@ func TestMetadataOnlyUpdate(t *testing.T) {
}

func TestImportResurrectionMou(t *testing.T) {
if !base.TestUseXattrs() {
t.Skip("This test requires xattrs because it relies on import")
}

base.SetUpTestLogging(t, base.LevelInfo, base.KeyMigrate, base.KeyImport, base.KeyCRUD)
db, ctx := setupTestDBWithOptionsAndImport(t, nil, DatabaseContextOptions{})
defer db.Close(ctx)
Expand Down Expand Up @@ -951,6 +955,10 @@ func TestImportResurrectionMou(t *testing.T) {
// update, even though it originated with an SDK delete, because the existing non-winning revision body will be
// promoted to winning.
func TestImportConflictWithTombstone(t *testing.T) {
if !base.TestUseXattrs() {
t.Skip("This test requires xattrs because it relies on import")
}

base.SetUpTestLogging(t, base.LevelInfo, base.KeyMigrate, base.KeyImport, base.KeyCRUD)
db, ctx := setupTestDBWithOptionsAndImport(t, nil, DatabaseContextOptions{})
defer db.Close(ctx)
Expand Down
7 changes: 7 additions & 0 deletions rest/changes_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -274,6 +274,9 @@ func TestWebhookWinningRevChangedEvent(t *testing.T) {
// - Update this doc again, triggering unused sequence range release
// - Write another doc and assert that the changes feed returns all expected docs
func TestJumpInSequencesAtAllocatorSkippedSequenceFill(t *testing.T) {
if !base.TestUseXattrs() {
t.Skip("This test requires xattrs because it writes directly to the xattr")
}

base.SetUpTestLogging(t, base.LevelDebug, base.KeyAll)

Expand Down Expand Up @@ -342,6 +345,10 @@ func TestJumpInSequencesAtAllocatorSkippedSequenceFill(t *testing.T) {
// - Update this doc again, triggering unused sequence range release
// - Write another doc and assert that the changes feed returns all expected docs
func TestJumpInSequencesAtAllocatorRangeInPending(t *testing.T) {
if !base.TestUseXattrs() {
t.Skip("This test requires xattrs because it writes directly to the xattr")
}

base.SetUpTestLogging(t, base.LevelDebug, base.KeyAll)

rt := NewRestTester(t, &RestTesterConfig{
Expand Down

0 comments on commit dc18ecd

Please sign in to comment.