Skip to content

Commit 91e5466

Browse files
torcolvingregns1
andauthored
Update test comment description (#7826)
Co-authored-by: Gregory Newman-Smith <gregory.newmansmith@couchbase.com>
1 parent 858e8cd commit 91e5466

File tree

1 file changed

+21
-18
lines changed

1 file changed

+21
-18
lines changed

rest/replicatortest/replicator_test_legacy_rev_test.go

Lines changed: 21 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -826,28 +826,31 @@ func TestActiveReplicatorConflictPreUpgradedVersionOneSide(t *testing.T) {
826826
name string
827827
activePeerHasPostUpgradeVersion bool
828828
}{
829-
// +-----------------+-------------------+----------------------------------------+-------------------+----------------------------------------+--+--+--+--+--+
830-
// | | SGW1 | | SGW2 | | | | | | |
831-
// +-----------------+-------------------+----------------------------------------+-------------------+----------------------------------------+--+--+--+--+--+
832-
// | | Rev Tree | HLV | Rev Tree | HLV | | | | | |
833-
// +-----------------+-------------------+----------------------------------------+-------------------+----------------------------------------+--+--+--+--+--+
834-
// | Initial State | 2-def,1-abc | 100@SGW1 | 2-abc,1-abc | none | | | | | |
835-
// +-----------------+-------------------+----------------------------------------+-------------------+----------------------------------------+--+--+--+--+--+
836-
// | Expected Result | 3-abc,2-abc,1-abc | encodedFromRev3@Revision+Tree+Encoding | 3-abc,2-abc,1-abc | encodedFromRev3@Revision+Tree+Encoding | | | | | |
837-
// +-----------------+-------------------+----------------------------------------+-------------------+----------------------------------------+--+--+--+--+--+
829+
// +-----------------+-------------------+--------------------------------------+-------------------+
830+
// | | SGW1 | | SGW2 | |
831+
// +-----------------+-------------------+------------------+-------------------+-------------------+
832+
// | | Rev Tree | HLV | Rev Tree | HLV |
833+
// +-----------------+-------------------+------------------+-------------------+-------------------+
834+
// | Initial State | 2-def,1-abc | 100@SGW1 | 2-abc,1-abc | none |
835+
// +-----------------+-------------------+------------------+-------------------+-------------------+
836+
// | Expected Result | 3-abc,2-abc,1-abc | 3abc@RTE;100@SGW | 3-abc,2-abc,1-abc | 3abc@RTE;100@SGW1 |
837+
// +-----------------+-------------------+--------------------------------------+-------------------+
838838
{
839839
name: "active peer has post upgrade version that wins",
840840
activePeerHasPostUpgradeVersion: true,
841841
},
842-
// +-----------------+-------------+----------+-------------+----------+--+--+--+--+--+
843-
// | | SGW1 | | SGW2 | | | | | | |
844-
// +-----------------+-------------+----------+-------------+----------+--+--+--+--+--+
845-
// | | Rev Tree | HLV | Rev Tree | HLV | | | | | |
846-
// +-----------------+-------------+----------+-------------+----------+--+--+--+--+--+
847-
// | Initial State | 2-abc,1-abc | none | 2-def,1-abc | 100@SGW2 | | | | | |
848-
// +-----------------+-------------+----------+-------------+----------+--+--+--+--+--+
849-
// | Expected Result | 2-def,1-abc | 100@SGW2 | 2-def,1-abc | 100@SGW2 | | | | | |
850-
// +-----------------+-------------+----------+-------------+----------+--+--+--+--+--+
842+
// The below test cases updates the document again on active peer to ensure we can push back to passive with
843+
// no conflict after initial conflict resolution. Hence the expected result HLV contains active peer SGW1 as
844+
// current version.
845+
// +-----------------+-------------+-------------------------------+-------------+--------------------------------+
846+
// | | SGW1 | | SGW2 | |
847+
// +-----------------+-------------+-------------------------------+-------------+--------------------------------+
848+
// | | Rev Tree | HLV | Rev Tree | HLV |
849+
// +-----------------+-------------+-------------------------------+-------------+--------------------------------+
850+
// | Initial State | 2-abc,1-abc | none | 2-def,1-abc | 100@SGW2 |
851+
// +-----------------+-------------+-------------------------------+-------------+--------------------------------+
852+
// | Expected Result | 2-def,1-abc | 1100@SGW1;2def@RTE,oldcas@SGW2| 2-def,1-abc | 1100@SGW1;2def@RTE,oldcas@SGW2 |
853+
// +-----------------+-------------+-------------------------------+-------------+--------------------------------+
851854
{
852855
name: "passive peer has post upgrade version that wins",
853856
activePeerHasPostUpgradeVersion: false,

0 commit comments

Comments
 (0)