-
Type: Bug
-
Resolution: Fixed
-
Priority: Minor - P4
-
Affects Version/s: 4.5.0
-
Component/s: None
The spec says that we should not send comments on server versions <4.4 for listIndexes, listDatabases, and listCollections operations. However, we have a bug in the `CommandOperation` class that will apply the comment to any command as long as it's of type "string", which overrides any operation-specific filtering of comments that is done prior to invoking executeCommand.
We missed this in testing because our custom format tests did not use the `comment:
{ $$exists: false }` assertion in the corresponding event checks.
AC
- Remove the override logic in command.ts
- Fix tests to correctly assert the nonexistence of the property