Skip to content

Commit 4bd593a

Browse files
authored
CBG-4538 tests: do not expect revSeqNo as a particular value (#7675)
1 parent 60d9909 commit 4bd593a

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

rest/importtest/import_test.go

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -70,8 +70,9 @@ func TestImportFeed(t *testing.T) {
7070

7171
xattrs, _, err := dataStore.GetXattrs(rt.Context(), mobileKey, []string{base.MouXattrName, base.VirtualXattrRevSeqNo})
7272
require.NoError(t, err)
73-
require.Equal(t, uint64(2), db.RetrieveDocRevSeqNo(t, xattrs[base.VirtualXattrRevSeqNo]))
74-
require.Equal(t, uint64(1), getMou(t, xattrs[base.MouXattrName]).PreviousRevSeqNo)
73+
// can not assert on a specific revseqno due to revseqno starting at the max_cas of any previously deleted document
74+
require.Greater(t, db.RetrieveDocRevSeqNo(t, xattrs[base.VirtualXattrRevSeqNo]), uint64(0))
75+
require.Greater(t, getMou(t, xattrs[base.MouXattrName]).PreviousRevSeqNo, uint64(0))
7576
// Attempt to get the document via Sync Gateway.
7677
response := rt.SendAdminRequest("GET", "/{{.keyspace}}/"+mobileKey, "")
7778
assert.Equal(t, 200, response.Code)
@@ -2606,9 +2607,9 @@ func TestPrevRevNoPopulationImportFeed(t *testing.T) {
26062607
require.NoError(t, err)
26072608

26082609
revNo := db.RetrieveDocRevSeqNo(t, xattrs[base.VirtualXattrRevSeqNo])
2610+
// can not assert on a specific revseqno due to revseqno starting at the max_cas of any previously deleted document
26092611
mou := getMou(t, xattrs[base.MouXattrName])
2610-
// curr rev no should be 2, so prev rev is 1
2611-
assert.Equal(t, revNo-1, mou.PreviousRevSeqNo)
2612+
require.Greater(t, revNo, mou.PreviousRevSeqNo)
26122613

26132614
err = dataStore.Set(mobileKey, 0, nil, []byte(`{"test":"update"}`))
26142615
require.NoError(t, err)
@@ -2620,10 +2621,10 @@ func TestPrevRevNoPopulationImportFeed(t *testing.T) {
26202621
xattrs, _, err = dataStore.GetXattrs(ctx, mobileKey, []string{base.MouXattrName, base.VirtualXattrRevSeqNo})
26212622
require.NoError(t, err)
26222623

2624+
// can not assert on a specific revseqno due to revseqno starting at the max_cas of any previously deleted document
26232625
revNo = db.RetrieveDocRevSeqNo(t, xattrs[base.VirtualXattrRevSeqNo])
26242626
mou = getMou(t, xattrs[base.MouXattrName])
2625-
// curr rev no should be 4, so prev rev is 3
2626-
assert.Equal(t, revNo-1, mou.PreviousRevSeqNo)
2627+
require.Greater(t, revNo, mou.PreviousRevSeqNo)
26272628

26282629
}
26292630

0 commit comments

Comments
 (0)