SERVER-66948 Catch NoSuchTransaction as an acceptable error in internal transactions coordinator cleanup test

(cherry picked from commit 10c8db0e325dd13c38b15ef7fef296355e1da3f9)

1 files changed, 9 insertions, 2 deletions

diff --git a/jstests/multiVersion/internal_sessions_setfcv_wait_for_transaction_coordinator_cleanup.js b/jstests/multiVersion/internal_sessions_setfcv_wait_for_transaction_coordinator_cleanup.js
index 4a9900dc445..e9b0aea9005 100644
--- a/jstests/multiVersion/internal_sessions_setfcv_wait_for_transaction_coordinator_cleanup.js
+++ b/jstests/multiVersion/internal_sessions_setfcv_wait_for_transaction_coordinator_cleanup.js
@@ -169,8 +169,15 @@ function runTestWithFailoverBeforeDocumentRemoval(lsid) {
             autocommit: false,
         }));
 
-        assert.commandWorked(conn.adminCommand(
-            {commitTransaction: 1, lsid: lsid, txnNumber: NumberLong(0), autocommit: false}));
+        // In the case where a commitTransaction retry attempt happens to execute after the
+        // transaction entry is removed from config.transactions and
+        // config.transaction_coordinators upon completion of the transaction and fcv downgrade, we
+        // have no way to recover the transaction decision, so the server will return
+        // NoSuchTransaction.
+        assert.commandWorkedOrFailedWithCode(
+            conn.adminCommand(
+                {commitTransaction: 1, lsid: lsid, txnNumber: NumberLong(0), autocommit: false}),
+            ErrorCodes.NoSuchTransaction);
     };
 
     const transactionCommitThread = new Thread(insertDocumentsInTransaction, st.s.host, txnParams);