-
Type: Spec Change
-
Resolution: Unresolved
-
Priority: Minor - P4
-
None
-
Component/s: Command Logging and Monitoring
-
None
-
Needed
Now that find "modifiers" are deprecated, we should replace use of them in spec tests with their normal property replacements.
The one example i found is "A successful find event with options" in the command monitoring tests:
"modifiers": { "$comment": "test", "$hint": { "_id": 1 }, "$max": { "_id": 6 }, "$maxTimeMS": 6000, "$min": { "_id": 0 }, "$returnKey": false, "$showDiskLoc": false, }
could be replaced with
"comment" : "test", "hint": { "_id": 1 }, "max": { "_id": 6 }, "maxTimeMS": 6000, "min": { "_id": 0 }, "returnKey": true, "showRecordId": true,
This will make it possible for drivers that have removed modifiers support to implement the test.
It's possible that we still want to test modifiers translation, but if so, let's figure out a way to signify in the test that drivers that don't support modifiers should skip the test.
- is duplicated by
-
DRIVERS-956 Command monitoring find tests are outdated
- Closed