Copybara Firestore

PiperOrigin-RevId: 333174406

Author: Googler

firestore/src/tests/validation_test.cc

@@ -571,35 +571,35 @@ TEST_F(ValidationTest, QueriesWithNullOrNaNFiltersOtherThanEqualityFail) {
     FAIL() << "should throw exception";
   } catch (const FirestoreException& exception) {
     EXPECT_STREQ(
-        "Invalid Query. Null supports only equality comparisons (via "
-        "whereEqualTo()).",
+        "Invalid Query. Null only supports comparisons via whereEqualTo() and "
+        "whereNotEqualTo().",
         exception.what());
   }
   try {
     collection.WhereArrayContains("a", FieldValue::Null());
     FAIL() << "should throw exception";
   } catch (const FirestoreException& exception) {
     EXPECT_STREQ(
-        "Invalid Query. Null supports only equality comparisons (via "
-        "whereEqualTo()).",
+        "Invalid Query. Null only supports comparisons via whereEqualTo() and "
+        "whereNotEqualTo().",
         exception.what());
   }
   try {
     collection.WhereGreaterThan("a", FieldValue::Double(NAN));
     FAIL() << "should throw exception";
   } catch (const FirestoreException& exception) {
     EXPECT_STREQ(
-        "Invalid Query. NaN supports only equality comparisons (via "
-        "whereEqualTo()).",
+        "Invalid Query. NaN only supports comparisons via whereEqualTo() and "
+        "whereNotEqualTo().",
         exception.what());
   }
   try {
     collection.WhereArrayContains("a", FieldValue::Double(NAN));
     FAIL() << "should throw exception";
   } catch (const FirestoreException& exception) {
     EXPECT_STREQ(
-        "Invalid Query. NaN supports only equality comparisons (via "
-        "whereEqualTo()).",
+        "Invalid Query. NaN only supports comparisons via whereEqualTo() and "
+        "whereNotEqualTo().",
         exception.what());
   }
 }
@@ -735,8 +735,9 @@ TEST_F(ValidationTest, QueriesWithDifferentInequalityFieldsFail) {
     FAIL() << "should throw exception";
   } catch (const FirestoreException& exception) {
     EXPECT_STREQ(
-        "All where filters other than whereEqualTo() must be on the same "
-        "field. But you have filters on 'x' and 'y'",
+        "All where filters with an inequality (notEqualTo, notIn, lessThan, "
+        "lessThanOrEqualTo, greaterThan, or greaterThanOrEqualTo) must be on "
+        "the same field. But you have filters on 'x' and 'y'",
         exception.what());
   }
 }