SERVER-24508 BSONObj::ComparatorInterface

BSONObj instances should now be compared via the comparator
interface's evaluate() method. This preferred over using
BSONObj::woCompare() directly. If the comparison doesn't
require any database semantics (e.g. there is no collation),
there is a global instance of the SimpleBSONObjComparator
which should be used for BSONObj comparisons. If the
comparison requires special semantics, then callers must
instantiate their own comparator object.

1 files changed, 10 insertions, 10 deletions

diff --git a/src/mongo/s/request_types/remove_shard_from_zone_request_test.cpp b/src/mongo/s/request_types/remove_shard_from_zone_request_test.cpp
index 0115affed0c..f687d98f46b 100644
--- a/src/mongo/s/request_types/remove_shard_from_zone_request_test.cpp
+++ b/src/mongo/s/request_types/remove_shard_from_zone_request_test.cpp
@@ -64,11 +64,11 @@ TEST(RemoveShardFromZoneRequest, CommandBuilderShouldAlwaysCreateConfigCommand)
     request.appendAsConfigCommand(&builder);
     auto cmdObj = builder.obj();
 
-    ASSERT_EQ(BSON("_configsvrRemoveShardFromZone"
-                   << "a"
-                   << "zone"
-                   << "z"),
-              cmdObj);
+    ASSERT_BSONOBJ_EQ(BSON("_configsvrRemoveShardFromZone"
+                           << "a"
+                           << "zone"
+                           << "z"),
+                      cmdObj);
 }
 
 TEST(RemoveShardFromZoneRequest, MissingZoneErrors) {
@@ -123,11 +123,11 @@ TEST(CfgRemoveShardFromZoneRequest, BasicValidConfigCommand) {
     request.appendAsConfigCommand(&builder);
     auto cmdObj = builder.obj();
 
-    ASSERT_EQ(BSON("_configsvrRemoveShardFromZone"
-                   << "a"
-                   << "zone"
-                   << "z"),
-              cmdObj);
+    ASSERT_BSONOBJ_EQ(BSON("_configsvrRemoveShardFromZone"
+                           << "a"
+                           << "zone"
+                           << "z"),
+                      cmdObj);
 }
 
 TEST(CfgRemoveShardFromZoneRequest, MissingZoneErrors) {