diff --git a/googlemock/test/gmock-matchers_test.cc b/googlemock/test/gmock-matchers_test.cc index 186d8aae..1a87d230 100644 --- a/googlemock/test/gmock-matchers_test.cc +++ b/googlemock/test/gmock-matchers_test.cc @@ -3020,13 +3020,13 @@ TEST(MatcherAssertionTest, WorksWhenMatcherIsNotSatisfied) { EXPECT_FATAL_FAILURE(ASSERT_THAT(n, Gt(10)), "Value of: n\n" "Expected: is > 10\n" - " Actual: 5" + OfType("unsigned short")); + " Actual: 5" + OfType(internal::GetTypeName())); n = 0; EXPECT_NONFATAL_FAILURE( EXPECT_THAT(n, AllOf(Le(7), Ge(5))), "Value of: n\n" "Expected: (is <= 7) and (is >= 5)\n" - " Actual: 0" + OfType("unsigned short")); + " Actual: 0" + OfType(internal::GetTypeName())); } // Tests that ASSERT_THAT() and EXPECT_THAT() work when the argument @@ -3042,7 +3042,7 @@ TEST(MatcherAssertionTest, WorksForByRefArguments) { "Expected: does not reference the variable @"); // Tests the "Actual" part. EXPECT_FATAL_FAILURE(ASSERT_THAT(n, Not(Ref(n))), - "Actual: 0" + OfType("int") + ", which is located @"); + "Actual: 0" + OfType(internal::GetTypeName()) + ", which is located @"); } // Tests that ASSERT_THAT() and EXPECT_THAT() work when the matcher is @@ -3062,7 +3062,7 @@ TEST(MatcherAssertionTest, WorksForMonomorphicMatcher) { EXPECT_NONFATAL_FAILURE(EXPECT_THAT(5, is_greater_than_5), "Value of: 5\n" "Expected: is > 5\n" - " Actual: 5" + OfType("int")); + " Actual: 5" + OfType(internal::GetTypeName())); } // Tests floating-point matchers. @@ -3705,14 +3705,14 @@ TEST(PointeeTest, CanExplainMatchResult) { const Matcher m2 = Pointee(GreaterThan(1)); // NOLINT long n = 3; // NOLINT - EXPECT_EQ("which points to 3" + OfType("long") + ", which is 2 more than 1", + EXPECT_EQ("which points to 3" + OfType(internal::GetTypeName()) + ", which is 2 more than 1", Explain(m2, &n)); } TEST(PointeeTest, AlwaysExplainsPointee) { const Matcher m = Pointee(0); int n = 42; - EXPECT_EQ("which points to 42" + OfType("int"), Explain(m, &n)); + EXPECT_EQ("which points to 42" + OfType(internal::GetTypeName()), Explain(m, &n)); } // An uncopyable class. @@ -3871,11 +3871,11 @@ TEST(FieldTest, CanExplainMatchResult) { AStruct a; a.x = 1; - EXPECT_EQ("whose given field is 1" + OfType("int"), Explain(m, a)); + EXPECT_EQ("whose given field is 1" + OfType(internal::GetTypeName()), Explain(m, a)); m = Field(&AStruct::x, GreaterThan(0)); EXPECT_EQ( - "whose given field is 1" + OfType("int") + ", which is 1 more than 0", + "whose given field is 1" + OfType(internal::GetTypeName()) + ", which is 1 more than 0", Explain(m, a)); } @@ -3884,10 +3884,10 @@ TEST(FieldTest, CanExplainMatchResultWithFieldName) { AStruct a; a.x = 1; - EXPECT_EQ("whose field `field_name` is 1" + OfType("int"), Explain(m, a)); + EXPECT_EQ("whose field `field_name` is 1" + OfType(internal::GetTypeName()), Explain(m, a)); m = Field("field_name", &AStruct::x, GreaterThan(0)); - EXPECT_EQ("whose field `field_name` is 1" + OfType("int") + + EXPECT_EQ("whose field `field_name` is 1" + OfType(internal::GetTypeName()) + ", which is 1 more than 0", Explain(m, a)); } @@ -3964,11 +3964,11 @@ TEST(FieldForPointerTest, CanExplainMatchResult) { AStruct a; a.x = 1; EXPECT_EQ("", Explain(m, static_cast(nullptr))); - EXPECT_EQ("which points to an object whose given field is 1" + OfType("int"), + EXPECT_EQ("which points to an object whose given field is 1" + OfType(internal::GetTypeName()), Explain(m, &a)); m = Field(&AStruct::x, GreaterThan(0)); - EXPECT_EQ("which points to an object whose given field is 1" + OfType("int") + + EXPECT_EQ("which points to an object whose given field is 1" + OfType(internal::GetTypeName()) + ", which is 1 more than 0", Explain(m, &a)); } @@ -3979,12 +3979,12 @@ TEST(FieldForPointerTest, CanExplainMatchResultWithFieldName) { a.x = 1; EXPECT_EQ("", Explain(m, static_cast(nullptr))); EXPECT_EQ( - "which points to an object whose field `field_name` is 1" + OfType("int"), + "which points to an object whose field `field_name` is 1" + OfType(internal::GetTypeName()), Explain(m, &a)); m = Field("field_name", &AStruct::x, GreaterThan(0)); EXPECT_EQ("which points to an object whose field `field_name` is 1" + - OfType("int") + ", which is 1 more than 0", + OfType(internal::GetTypeName()) + ", which is 1 more than 0", Explain(m, &a)); } @@ -4157,11 +4157,11 @@ TEST(PropertyTest, CanExplainMatchResult) { AClass a; a.set_n(1); - EXPECT_EQ("whose given property is 1" + OfType("int"), Explain(m, a)); + EXPECT_EQ("whose given property is 1" + OfType(internal::GetTypeName()), Explain(m, a)); m = Property(&AClass::n, GreaterThan(0)); EXPECT_EQ( - "whose given property is 1" + OfType("int") + ", which is 1 more than 0", + "whose given property is 1" + OfType(internal::GetTypeName()) + ", which is 1 more than 0", Explain(m, a)); } @@ -4170,10 +4170,10 @@ TEST(PropertyTest, CanExplainMatchResultWithPropertyName) { AClass a; a.set_n(1); - EXPECT_EQ("whose property `fancy_name` is 1" + OfType("int"), Explain(m, a)); + EXPECT_EQ("whose property `fancy_name` is 1" + OfType(internal::GetTypeName()), Explain(m, a)); m = Property("fancy_name", &AClass::n, GreaterThan(0)); - EXPECT_EQ("whose property `fancy_name` is 1" + OfType("int") + + EXPECT_EQ("whose property `fancy_name` is 1" + OfType(internal::GetTypeName()) + ", which is 1 more than 0", Explain(m, a)); } @@ -4261,12 +4261,12 @@ TEST(PropertyForPointerTest, CanExplainMatchResult) { a.set_n(1); EXPECT_EQ("", Explain(m, static_cast(nullptr))); EXPECT_EQ( - "which points to an object whose given property is 1" + OfType("int"), + "which points to an object whose given property is 1" + OfType(internal::GetTypeName()), Explain(m, &a)); m = Property(&AClass::n, GreaterThan(0)); EXPECT_EQ("which points to an object whose given property is 1" + - OfType("int") + ", which is 1 more than 0", + OfType(internal::GetTypeName()) + ", which is 1 more than 0", Explain(m, &a)); } @@ -4277,12 +4277,12 @@ TEST(PropertyForPointerTest, CanExplainMatchResultWithPropertyName) { a.set_n(1); EXPECT_EQ("", Explain(m, static_cast(nullptr))); EXPECT_EQ("which points to an object whose property `fancy_name` is 1" + - OfType("int"), + OfType(internal::GetTypeName()), Explain(m, &a)); m = Property("fancy_name", &AClass::n, GreaterThan(0)); EXPECT_EQ("which points to an object whose property `fancy_name` is 1" + - OfType("int") + ", which is 1 more than 0", + OfType(internal::GetTypeName()) + ", which is 1 more than 0", Explain(m, &a)); } @@ -4316,11 +4316,11 @@ int IntFunction(int input) { return input == 42 ? 80 : 90; } TEST(ResultOfTest, CanExplainMatchResult) { Matcher matcher = ResultOf(&IntFunction, Ge(85)); - EXPECT_EQ("which is mapped by the given callable to 90" + OfType("int"), + EXPECT_EQ("which is mapped by the given callable to 90" + OfType(internal::GetTypeName()), Explain(matcher, 36)); matcher = ResultOf(&IntFunction, GreaterThan(85)); - EXPECT_EQ("which is mapped by the given callable to 90" + OfType("int") + + EXPECT_EQ("which is mapped by the given callable to 90" + OfType(internal::GetTypeName()) + ", which is 5 more than 85", Explain(matcher, 36)); }