fix(NODE-5745): Ignore Read/Write Concern in Atlas Search Index Helpers

src/collection.ts

@@ -1052,7 +1052,7 @@ export class Collection<TSchema extends Document = Document> {
   /**
    * Returns all search indexes for the current collection.
    *
-   * @param options - The options for the list indexes operation.
+   * @param options - The options for the list indexes operation. This method will ignore 'readConcern' and 'writeConcern' keys if they are present in 'options' (since there is no Atlas support).
    *
    * @remarks Only available when used against a 7.0+ Atlas cluster.
    */
@@ -1061,7 +1061,7 @@ export class Collection<TSchema extends Document = Document> {
    * Returns all search indexes for the current collection.
    *
    * @param name - The name of the index to search for.  Only indexes with matching index names will be returned.
-   * @param options - The options for the list indexes operation.
+   * @param options - The options for the list indexes operation. This method will ignore 'readConcern' and 'writeConcern' keys if they are present in 'options' (since there is no Atlas support).
    *
    * @remarks Only available when used against a 7.0+ Atlas cluster.
    */
@@ -1072,14 +1072,18 @@ export class Collection<TSchema extends Document = Document> {
   ): ListSearchIndexesCursor {
     options =
       typeof indexNameOrOptions === 'object' ? indexNameOrOptions : options == null ? {} : options;
+
+    /* eslint-disable-next-line @typescript-eslint/no-unused-vars */
+    const cleanedOptions = (({ readConcern, writeConcern, ...rest }) => rest)(options);
+
     const indexName =
       indexNameOrOptions == null
         ? null
         : typeof indexNameOrOptions === 'object'
         ? null
         : indexNameOrOptions;
 
-    return new ListSearchIndexesCursor(this as TODO_NODE_3286, indexName, options);
+    return new ListSearchIndexesCursor(this as TODO_NODE_3286, indexName, cleanedOptions);
   }
 
   /**

test/integration/index-management/node-specific/searchIndexIgnoresReadWriteConcern-with-transaction.json

@@ -0,0 +1,263 @@
+{
+    "description": "when in a transaction, search index operations ignore read and write concern",
+    "schemaVersion": "1.4",
+    "createEntities": [
+      {
+        "client": {
+          "id": "client0",
+          "useMultipleMongoses": false,
+          "uriOptions": {
+            "readConcernLevel": "local",
+            "w": 1
+          },
+          "observeEvents": [
+            "commandStartedEvent"
+          ]
+        }
+      },
+      {
+        "session": {
+          "id": "session0",
+          "client": "client0"
+        }
+      },
+      {
+        "database": {
+          "id": "database0",
+          "client": "client0",
+          "databaseName": "database0"
+        }
+      },
+      {
+        "collection": {
+          "id": "collection0",
+          "database": "database0",
+          "collectionName": "collection0"
+        }
+      }
+    ],
+    "runOnRequirements": [
+      {
+        "minServerVersion": "7.0.0",
+        "topologies": [
+          "replicaset",
+          "load-balanced",
+          "sharded"
+        ],
+        "serverless": "forbid"
+      }
+    ],
+    "tests": [
+      {
+        "description": "when in a transaction, createSearchIndex ignores read and write concern",
+        "operations": [
+          {
+            "name": "createSearchIndex",
+            "object": "collection0",
+            "arguments": {
+              "session": "session0",
+              "model": {
+                "definition": {
+                  "mappings": {
+                    "dynamic": true
+                  }
+                }
+              }
+            },
+            "expectError": {
+              "isError": true,
+              "errorContains": "Atlas"
+            }
+          }
+        ],
+        "expectEvents": [
+          {
+            "client": "client0",
+            "events": [
+              {
+                "commandStartedEvent": {
+                  "command": {
+                    "createSearchIndexes": "collection0",
+                    "indexes": [
+                      {
+                        "definition": {
+                          "mappings": {
+                            "dynamic": true
+                          }
+                        }
+                      }
+                    ],
+                    "$db": "database0",
+                    "writeConcern": {
+                      "$$exists": false
+                    },
+                    "readConcern": {
+                      "$$exists": false
+                    }
+                  }
+                }
+              }
+            ]
+          }
+        ]
+      },
+      {
+        "description": "when in a transaction, createSearchIndexes ignores read and write concern",
+        "operations": [
+          {
+            "name": "createSearchIndexes",
+            "object": "collection0",
+            "arguments": {
+              "session": "session0",
+              "models": []
+            },
+            "expectError": {
+              "isError": true,
+              "errorContains": "Atlas"
+            }
+          }
+        ],
+        "expectEvents": [
+          {
+            "client": "client0",
+            "events": [
+              {
+                "commandStartedEvent": {
+                  "command": {
+                    "createSearchIndexes": "collection0",
+                    "indexes": [],
+                    "$db": "database0",
+                    "writeConcern": {
+                      "$$exists": false
+                    },
+                    "readConcern": {
+                      "$$exists": false
+                    }
+                  }
+                }
+              }
+            ]
+          }
+        ]
+      },
+      {
+        "description": "when in a transaction, dropSearchIndex ignores read and write concern",
+        "operations": [
+          {
+            "name": "dropSearchIndex",
+            "object": "collection0",
+            "arguments": {
+              "session": "session0",
+              "name": "test index"
+            },
+            "expectError": {
+              "isError": true,
+              "errorContains": "Atlas"
+            }
+          }
+        ],
+        "expectEvents": [
+          {
+            "client": "client0",
+            "events": [
+              {
+                "commandStartedEvent": {
+                  "command": {
+                    "dropSearchIndex": "collection0",
+                    "name": "test index",
+                    "$db": "database0",
+                    "writeConcern": {
+                      "$$exists": false
+                    },
+                    "readConcern": {
+                      "$$exists": false
+                    }
+                  }
+                }
+              }
+            ]
+          }
+        ]
+      },
+      {
+        "description": "when in a transaction, listSearchIndexes ignores read and write concern",
+        "operations": [
+          {
+            "name": "listSearchIndexes",
+            "object": "collection0",
+            "expectError": {
+              "isError": true,
+              "errorContains": "Atlas"
+            }
+          }
+        ],
+        "expectEvents": [
+          {
+            "client": "client0",
+            "events": [
+              {
+                "commandStartedEvent": {
+                  "command": {
+                    "aggregate": "collection0",
+                    "pipeline": [
+                      {
+                        "$listSearchIndexes": {}
+                      }
+                    ],
+                    "writeConcern": {
+                      "$$exists": false
+                    },
+                    "readConcern": {
+                      "$$exists": false
+                    }
+                  }
+                }
+              }
+            ]
+          }
+        ]
+      },
+      {
+        "description": "when in a transaction, updateSearchIndex ignores the read and write concern",
+        "operations": [
+          {
+            "name": "updateSearchIndex",
+            "object": "collection0",
+            "arguments": {
+              "session": "session0",
+              "name": "test index",
+              "definition": {}
+            },
+            "expectError": {
+              "isError": true,
+              "errorContains": "Atlas"
+            }
+          }
+        ],
+        "expectEvents": [
+          {
+            "client": "client0",
+            "events": [
+              {
+                "commandStartedEvent": {
+                  "command": {
+                    "updateSearchIndex": "collection0",
+                    "name": "test index",
+                    "definition": {},
+                    "$db": "database0",
+                    "writeConcern": {
+                      "$$exists": false
+                    },
+                    "readConcern": {
+                      "$$exists": false
+                    }
+                  }
+                }
+              }
+            ]
+          }
+        ]
+      }
+    ]
+  }
+

test/integration/index-management/search-index-management.test.ts

@@ -0,0 +1,8 @@
+import { join } from 'path';
+
+import { loadSpecTests } from '../../spec';
+import { runUnifiedSuite } from '../../tools/unified-spec-runner/runner';
+
+describe('Search Index Management Tests (Node Specific)', function () {
+  runUnifiedSuite(loadSpecTests('../integration/index-management/node-specific'));
+});