Fix TransactionQueue test that checks wrong semantics

stellar · Mar 30, 2020 · f6fc4a3 · f6fc4a3
1 parent 198d647
commit f6fc4a3
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/herder/test/TransactionQueueTests.cpp b/src/herder/test/TransactionQueueTests.cpp
@@ -609,10 +609,12 @@ TEST_CASE("TransactionQueue", "[herder][TransactionQueue]")
         test.check({{{account1, 1, {txSeqA1T1, txSeqA1T2}},
                      {account2, 1, {txSeqA2T1, txSeqA2T2}}}});
         test.removeApplied({txSeqA1T1, txSeqA2T2});
-        test.check({{{account1, 0, {txSeqA1T2}}, {account2, 0, {txSeqA2T1}}}});
+        test.check({{{account1, 0, {txSeqA1T2}}, {account2}}});
         test.removeApplied({txSeqA1T2});
-        test.check({{{account1}, {account2, 0, {txSeqA2T1}}}});
+        test.check({{{account1}, {account2}}});
         test.add(txSeqA1T1, TransactionQueue::AddResult::ADD_STATUS_PENDING);
+        test.check({{{account1, 0, {txSeqA1T1}}, {account2}}});
+        test.add(txSeqA2T1, TransactionQueue::AddResult::ADD_STATUS_PENDING);
         test.check({{{account1, 0, {txSeqA1T1}}, {account2, 0, {txSeqA2T1}}}});
         test.add(txSeqA2T2, TransactionQueue::AddResult::ADD_STATUS_PENDING);
         test.check({{{account1, 0, {txSeqA1T1}},