SERVER-35864 Ensure the active transaction on the session has not already been commit or aborted when receiving a prepareTransaction command

1 files changed, 54 insertions, 0 deletions

diff --git a/jstests/core/txns/prepare_committed_transaction.js b/jstests/core/txns/prepare_committed_transaction.js
new file mode 100644
index 00000000000..6e9eb35ba5c
--- /dev/null
+++ b/jstests/core/txns/prepare_committed_transaction.js
@@ -0,0 +1,54 @@
+/**
+ * Test error cases when calling prepare on a committed transaction.
+ *
+ * @tags: [uses_transactions]
+ */
+(function() {
+    "use strict";
+
+    const dbName = "test";
+    const collName = "prepare_committed_transaction";
+    const testDB = db.getSiblingDB(dbName);
+    const testColl = testDB.getCollection(collName);
+
+    testColl.drop({writeConcern: {w: "majority"}});
+    assert.commandWorked(testDB.runCommand({create: collName, writeConcern: {w: "majority"}}));
+
+    const session = db.getMongo().startSession({causalConsistency: false});
+    const sessionDB = session.getDatabase(dbName);
+    const sessionColl = sessionDB.getCollection(collName);
+
+    const doc = {x: 1};
+
+    jsTestLog("Test that calling prepare on a committed transaction fails.");
+    session.startTransaction();
+    assert.commandWorked(sessionColl.insert(doc));
+    session.commitTransaction();
+    assert.commandFailedWithCode(
+        sessionDB.adminCommand(
+            {prepareTransaction: 1, txnNumber: NumberLong(0), autocommit: false}),
+        ErrorCodes.TransactionCommitted);
+
+    jsTestLog("Test the error precedence when calling prepare on a committed transaction but not " +
+              "providing txnNumber to prepareTransaction.");
+    assert.commandFailedWithCode(sessionDB.adminCommand({prepareTransaction: 1, autocommit: false}),
+                                 ErrorCodes.InvalidOptions);
+
+    jsTestLog("Test the error precedence when calling prepare on a committed transaction but not " +
+              "providing autocommit to prepareTransaction.");
+    assert.commandFailedWithCode(
+        sessionDB.adminCommand({prepareTransaction: 1, txnNumber: NumberLong(0)}),
+        ErrorCodes.InvalidOptions);
+
+    jsTestLog("Test the error precedence when calling prepare on a committed transaction and " +
+              "providing startTransaction to prepareTransaction.");
+    assert.commandFailedWithCode(sessionDB.adminCommand({
+        prepareTransaction: 1,
+        txnNumber: NumberLong(0),
+        autocommit: false,
+        startTransaction: true
+    }),
+                                 ErrorCodes.ConflictingOperationInProgress);
+
+    session.endSession();
+}());