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, 20 insertions, 16 deletions

diff --git a/src/mongo/db/fts/fts_query_impl_test.cpp b/src/mongo/db/fts/fts_query_impl_test.cpp
index 43585e8a982..6cb8ca8f9de 100644
--- a/src/mongo/db/fts/fts_query_impl_test.cpp
+++ b/src/mongo/db/fts/fts_query_impl_test.cpp
@@ -158,9 +158,10 @@ TEST(FTSQueryImpl, Phrase1) {
     q.setDiacriticSensitive(false);
     ASSERT(q.parse(TEXT_INDEX_VERSION_3).isOK());
 
-    ASSERT_EQUALS(q.toBSON(),
-                  fromjson("{terms: ['fun', 'phrase', 'test'], negatedTerms: [], phrases: ['phrase "
-                           "test'], negatedPhrases: []}"));
+    ASSERT_BSONOBJ_EQ(
+        q.toBSON(),
+        fromjson("{terms: ['fun', 'phrase', 'test'], negatedTerms: [], phrases: ['phrase "
+                 "test'], negatedPhrases: []}"));
     ASSERT_TRUE(q.getTermsForBounds() == q.getPositiveTerms());
 }
 
@@ -182,7 +183,7 @@ TEST(FTSQueryImpl, HyphenDirectlyBeforePhraseShouldNegateEntirePhrase) {
     q.setCaseSensitive(false);
     q.setDiacriticSensitive(false);
     ASSERT(q.parse(TEXT_INDEX_VERSION_3).isOK());
-    ASSERT_EQUALS(
+    ASSERT_BSONOBJ_EQ(
         q.toBSON(),
         fromjson(
             "{terms: ['fun'], negatedTerms: [], phrases: [], negatedPhrases: ['phrase test']}"));
@@ -195,9 +196,10 @@ TEST(FTSQueryImpl, HyphenSurroundedByWhitespaceBeforePhraseShouldNotNegateEntire
     q.setCaseSensitive(false);
     q.setDiacriticSensitive(false);
     ASSERT(q.parse(TEXT_INDEX_VERSION_3).isOK());
-    ASSERT_EQUALS(q.toBSON(),
-                  fromjson("{terms: ['fun', 'phrase', 'test'], negatedTerms: [], phrases: ['phrase "
-                           "test'], negatedPhrases: []}"));
+    ASSERT_BSONOBJ_EQ(
+        q.toBSON(),
+        fromjson("{terms: ['fun', 'phrase', 'test'], negatedTerms: [], phrases: ['phrase "
+                 "test'], negatedPhrases: []}"));
 }
 
 TEST(FTSQueryImpl, HyphenBetweenTermAndPhraseShouldBeTreatedAsDelimiter) {
@@ -207,9 +209,10 @@ TEST(FTSQueryImpl, HyphenBetweenTermAndPhraseShouldBeTreatedAsDelimiter) {
     q.setCaseSensitive(false);
     q.setDiacriticSensitive(false);
     ASSERT(q.parse(TEXT_INDEX_VERSION_3).isOK());
-    ASSERT_EQUALS(q.toBSON(),
-                  fromjson("{terms: ['fun', 'phrase', 'test'], negatedTerms: [], phrases: ['phrase "
-                           "test'], negatedPhrases: []}"));
+    ASSERT_BSONOBJ_EQ(
+        q.toBSON(),
+        fromjson("{terms: ['fun', 'phrase', 'test'], negatedTerms: [], phrases: ['phrase "
+                 "test'], negatedPhrases: []}"));
 }
 
 TEST(FTSQueryImpl, HyphenShouldNegateAllSucceedingPhrasesSeparatedByHyphens) {
@@ -219,9 +222,9 @@ TEST(FTSQueryImpl, HyphenShouldNegateAllSucceedingPhrasesSeparatedByHyphens) {
     q.setCaseSensitive(false);
     q.setDiacriticSensitive(false);
     ASSERT(q.parse(TEXT_INDEX_VERSION_3).isOK());
-    ASSERT_EQUALS(q.toBSON(),
-                  fromjson("{terms: ['anoth', 'phrase'], negatedTerms: [], phrases: ['another "
-                           "phrase'], negatedPhrases: ['really fun', 'stuff here']}"));
+    ASSERT_BSONOBJ_EQ(q.toBSON(),
+                      fromjson("{terms: ['anoth', 'phrase'], negatedTerms: [], phrases: ['another "
+                               "phrase'], negatedPhrases: ['really fun', 'stuff here']}"));
 }
 
 TEST(FTSQueryImpl, CaseSensitiveOption) {
@@ -304,9 +307,10 @@ TEST(FTSQueryImpl, Mix1) {
     q.setCaseSensitive(false);
     q.setDiacriticSensitive(false);
     ASSERT(q.parse(TEXT_INDEX_VERSION_3).isOK());
-    ASSERT_EQUALS(q.toBSON(),
-                  fromjson("{terms: ['industri'], negatedTerms: ['melbourn', 'physic'], phrases: "
-                           "['industry'], negatedPhrases: []}"));
+    ASSERT_BSONOBJ_EQ(
+        q.toBSON(),
+        fromjson("{terms: ['industri'], negatedTerms: ['melbourn', 'physic'], phrases: "
+                 "['industry'], negatedPhrases: []}"));
 }
 
 TEST(FTSQueryImpl, NegPhrase2) {