Make global_object_test slightly stricter.

PiperOrigin-RevId: 528527391
google · May 1, 2023 · 4fb5fb1 · 4fb5fb1
1 parent 2a7f261
commit 4fb5fb1
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/implementation/global_object_test.cc b/implementation/global_object_test.cc
@@ -49,7 +49,7 @@ TEST_F(JniTest, GlobalObject_CallsNewAndDeleteOnNewObject) {
 
   GlobalObject<kClass> global_object{};
 
-  EXPECT_NE(jobject{global_object}, nullptr);
+  EXPECT_EQ(jobject{global_object}, AsGlobal(Fake<jobject>()));
 }
 
 TEST_F(JniTest, GlobalObject_ConstructsFromNonStandardConstructor) {
@@ -116,9 +116,11 @@ TEST_F(JniTest, GlobalObject_CallsDeleteOnceAfterAMoveConstruction) {
   static constexpr Class kClass{
       "com/google/CallsDeleteOnceAfterAMoveConstruction"};
   GlobalObject<kClass> global_object_1{AdoptGlobal{}, Fake<jobject>()};
-  EXPECT_NE(jobject{global_object_1}, nullptr);
+  EXPECT_EQ(jobject{global_object_1}, Fake<jobject>());
   GlobalObject<kClass> global_object_2{std::move(global_object_1)};
-  EXPECT_NE(jobject{global_object_2}, nullptr);
+
+  EXPECT_EQ(jobject{global_object_1}, nullptr);  // NOLINT
+  EXPECT_EQ(jobject{global_object_2}, Fake<jobject>());
 }
 
 TEST_F(JniTest, GlobalObject_FunctionsProperlyInSTLContainer) {