From e367ac29b2fdba15914968f55303a4fc9be46d61 Mon Sep 17 00:00:00 2001
From: Edward Hennis <ed@ripple.com>
Date: Tue, 14 Dec 2021 16:27:14 -0500
Subject: [PATCH] Add a test case that is true regardless of transaction
 ordering

---
 src/test/app/TxQ_test.cpp | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/src/test/app/TxQ_test.cpp b/src/test/app/TxQ_test.cpp
index 2ade9e8e307..6a53663baba 100644
--- a/src/test/app/TxQ_test.cpp
+++ b/src/test/app/TxQ_test.cpp
@@ -1372,6 +1372,14 @@ class TxQ1_test : public beast::unit_test::suite
         env.close();
         checkMetrics(env, 3, 10, 6, 5, 256);
 
+        // This next test should remain unchanged regardless of
+        // transaction ordering
+        BEAST_EXPECT(
+            aliceSeq + bobSeq + charlieSeq + dariaSeq + elmoSeq + fredSeq +
+                gwenSeq + hankSeq + 6 ==
+            env.seq(alice) + env.seq(bob) + env.seq(charlie) + env.seq(daria) +
+                env.seq(elmo) + env.seq(fred) + env.seq(gwen) + env.seq(hank));
+        // These tests may change if TxQ ordering is changed
         BEAST_EXPECT(aliceSeq + 1 == env.seq(alice));
         BEAST_EXPECT(bobSeq + 1 == env.seq(bob));
         BEAST_EXPECT(charlieSeq == env.seq(charlie));
@@ -1408,6 +1416,14 @@ class TxQ1_test : public beast::unit_test::suite
         env.close();
         checkMetrics(env, 4, 12, 7, 6, 256);
 
+        // This next test should remain unchanged regardless of
+        // transaction ordering
+        BEAST_EXPECT(
+            aliceSeq + bobSeq + charlieSeq + dariaSeq + elmoSeq + fredSeq +
+                gwenSeq + hankSeq + 7 ==
+            env.seq(alice) + env.seq(bob) + env.seq(charlie) + env.seq(daria) +
+                env.seq(elmo) + env.seq(fred) + env.seq(gwen) + env.seq(hank));
+        // These tests may change if TxQ ordering is changed
         BEAST_EXPECT(fredSeq + 4 == env.seq(fred));
         BEAST_EXPECT(gwenSeq + 1 == env.seq(gwen));
         BEAST_EXPECT(hankSeq + 1 == env.seq(hank));