SERVER-72438 Validate no writeErrors due to shard key targeting

1 files changed, 12 insertions, 3 deletions

diff --git a/jstests/sharding/transactions_targeting_errors.js b/jstests/sharding/transactions_targeting_errors.js
index 90ddfb52a95..d8488a6197a 100644
--- a/jstests/sharding/transactions_targeting_errors.js
+++ b/jstests/sharding/transactions_targeting_errors.js
@@ -1,11 +1,10 @@
-// Verifies targeting errors encountered in a transaction lead to write errors.
+// Verifies targeting errors encountered in a transaction lead to write errors when write without
+// shard key feature is not enabled.
 //
 // @tags: [uses_transactions]
 (function() {
 "use strict";
 
-// TODO: SERVER-72438 Change transaction_targeting_errors.js to validate writeErrors that aren't due
-// to shard key targeting.
 load("jstests/sharding/updateOne_without_shard_key/libs/write_without_shard_key_test_util.js");
 
 const dbName = "test";
@@ -20,6 +19,16 @@ const session = st.s.startSession();
 const sessionDB = session.getDatabase("test");
 
 if (WriteWithoutShardKeyTestUtil.isWriteWithoutShardKeyFeatureEnabled(sessionDB)) {
+    session.startTransaction();
+    assert.commandWorked(sessionDB.runCommand(
+        {update: collName, updates: [{q: {skey: {$lte: 5}}, u: {$set: {x: 1}}, multi: false}]}));
+    assert.commandWorked(session.abortTransaction_forTesting());
+
+    session.startTransaction();
+    assert.commandWorked(
+        sessionDB.runCommand({delete: collName, deletes: [{q: {skey: {$lte: 5}}, limit: 1}]}));
+    assert.commandWorked(session.abortTransaction_forTesting());
+
     st.stop();
     return;
 }