SERVER-22109 validate index spec namespace in applyOperation_inlock

(cherry picked from commit 51f0cfa865fe090a5208d5bff47faa5fbe6a92c1)

1 files changed, 41 insertions, 0 deletions

diff --git a/jstests/core/apply_ops1.js b/jstests/core/apply_ops1.js
index cd2b57216e8..430af6b62a3 100644
--- a/jstests/core/apply_ops1.js
+++ b/jstests/core/apply_ops1.js
@@ -45,6 +45,47 @@
       'applyOps should fail on non-"n" operation type with empty "ns" field value'
     );
 
+    // Missing 'o' field value in an operation of type 'i' on 'system.indexes' collection.
+    assert.commandFailedWithCode(
+        db.adminCommand({applyOps: [{op: 'i', ns: db.getName() + '.system.indexes'}]}),
+        4, // ErrorCodes.NoSuchKey
+        'applyOps should fail on system.indexes insert operation without "o" field');
+
+    // Non-object 'o' field value in an operation of type 'i' on 'system.indexes' collection.
+    assert.commandFailedWithCode(
+        db.adminCommand({applyOps: [{op: 'i', ns: db.getName() + '.system.indexes', o: 'bar'}]}),
+        14, // ErrorCodes.TypeMismatch
+        'applyOps should fail on system.indexes insert operation with non-object "o" field');
+
+    // Missing 'ns' field in index spec.
+    assert.commandFailedWithCode(
+        db.adminCommand({applyOps: [{op: 'i', ns: db.getName() + '.system.indexes', o: {
+            key: {a: 1},
+            name: 'a_1',
+        }}]}),
+        4, // ErrorCodes.NoSuchKey
+        'applyOps should fail on system.indexes insert operation with missing index namespace');
+
+    // Non-string 'ns' field in index spec.
+    assert.commandFailedWithCode(
+        db.adminCommand({applyOps: [{op: 'i', ns: db.getName() + '.system.indexes', o: {
+            ns: 12345,
+            key: {a: 1},
+            name: 'a_1',
+        }}]}),
+        14, // ErrorCodes.TypeMismatch
+        'applyOps should fail on system.indexes insert operation with non-string index namespace');
+
+    // Invalid 'ns' field in index spec.
+    assert.commandFailedWithCode(
+        db.adminCommand({applyOps: [{op: 'i', ns: db.getName() + '.system.indexes', o: {
+            ns: 'invalid_namespace',
+            key: {a: 1},
+            name: 'a_1',
+        }}]}),
+        73, // ErrorCodes.InvalidNamespace
+        'applyOps should fail on system.indexes insert operation with invalid index namespace');
+
     // Valid 'ns' field value in unknown operation type 'x'.
     assert.commandFailed(
       db.adminCommand({applyOps: [{op: 'x', ns: t.getFullName()}]}),