mirror of
				https://github.com/catchorg/Catch2.git
				synced 2025-10-31 12:17:11 +01:00 
			
		
		
		
	Don’t include tests that rely on varying two-phase-lookup behaviour in approval tests
This commit is contained in:
		| @@ -830,6 +830,6 @@ with expansion: | ||||
|   "first" == "second" | ||||
|  | ||||
| =============================================================================== | ||||
| test cases: 159 | 115 passed | 42 failed |  2 failed as expected | ||||
| assertions: 915 | 819 passed | 78 failed | 18 failed as expected | ||||
| test cases: 157 | 113 passed | 42 failed |  2 failed as expected | ||||
| assertions: 913 | 817 passed | 78 failed | 18 failed as expected | ||||
|  | ||||
|   | ||||
| @@ -8757,32 +8757,6 @@ with expansion: | ||||
|   == | ||||
|   "{ StringMaker<has_maker> }" | ||||
|  | ||||
| ------------------------------------------------------------------------------- | ||||
| toString( vectors<has_maker_and_toString ) | ||||
| ------------------------------------------------------------------------------- | ||||
| ToStringWhich.cpp:<line number> | ||||
| ............................................................................... | ||||
|  | ||||
| ToStringWhich.cpp:<line number>: | ||||
| PASSED: | ||||
|   REQUIRE( Catch::toString( v ) == "{ StringMaker<has_maker_and_toString> }" ) | ||||
| with expansion: | ||||
|   "{ StringMaker<has_maker_and_toString> }" | ||||
|   == | ||||
|   "{ StringMaker<has_maker_and_toString> }" | ||||
|  | ||||
| ------------------------------------------------------------------------------- | ||||
| toString( vectors<has_toString ) | ||||
| ------------------------------------------------------------------------------- | ||||
| ToStringWhich.cpp:<line number> | ||||
| ............................................................................... | ||||
|  | ||||
| ToStringWhich.cpp:<line number>: | ||||
| PASSED: | ||||
|   REQUIRE( Catch::toString( v ) == "{ {?} }" ) | ||||
| with expansion: | ||||
|   "{ {?} }" == "{ {?} }" | ||||
|  | ||||
| ------------------------------------------------------------------------------- | ||||
| toString(enum w/operator<<) | ||||
| ------------------------------------------------------------------------------- | ||||
| @@ -9052,6 +9026,6 @@ MiscTests.cpp:<line number>: | ||||
| PASSED: | ||||
|  | ||||
| =============================================================================== | ||||
| test cases: 159 | 114 passed | 43 failed |  2 failed as expected | ||||
| assertions: 917 | 819 passed | 80 failed | 18 failed as expected | ||||
| test cases: 157 | 112 passed | 43 failed |  2 failed as expected | ||||
| assertions: 915 | 817 passed | 80 failed | 18 failed as expected | ||||
|  | ||||
|   | ||||
| @@ -1,6 +1,6 @@ | ||||
| <?xml version="1.1" encoding="UTF-8"?> | ||||
| <testsuites> | ||||
|   <testsuite name="<exe-name>" errors="13" failures="68" tests="918" hostname="tbd" time="{duration}" timestamp="{iso8601-timestamp}"> | ||||
|   <testsuite name="<exe-name>" errors="13" failures="68" tests="916" hostname="tbd" time="{duration}" timestamp="{iso8601-timestamp}"> | ||||
|     <testcase classname="global" name="# A test name that starts with a #" time="{duration}"/> | ||||
|     <testcase classname="global" name="'Not' checks that should fail" time="{duration}"> | ||||
|       <failure message="false != false" type="CHECK"> | ||||
| @@ -654,8 +654,6 @@ TrickyTests.cpp:<line number> | ||||
|     <testcase classname="global" name="toString( has_maker_and_toString )" time="{duration}"/> | ||||
|     <testcase classname="global" name="toString( has_toString )" time="{duration}"/> | ||||
|     <testcase classname="global" name="toString( vectors<has_maker )" time="{duration}"/> | ||||
|     <testcase classname="global" name="toString( vectors<has_maker_and_toString )" time="{duration}"/> | ||||
|     <testcase classname="global" name="toString( vectors<has_toString )" time="{duration}"/> | ||||
|     <testcase classname="global" name="toString(enum w/operator<<)" time="{duration}"/> | ||||
|     <testcase classname="global" name="toString(enum)" time="{duration}"/> | ||||
|     <testcase classname="global" name="vector<int> -> toString" time="{duration}"/> | ||||
|   | ||||
| @@ -9246,30 +9246,6 @@ there" | ||||
|       </Expression> | ||||
|       <OverallResult success="true"/> | ||||
|     </TestCase> | ||||
|     <TestCase name="toString( vectors<has_maker_and_toString )"> | ||||
|       <Expression success="true" type="REQUIRE" filename="projects/<exe-name>/ToStringWhich.cpp" > | ||||
|         <Original> | ||||
|           Catch::toString( v ) == "{ StringMaker<has_maker_and_toString> }" | ||||
|         </Original> | ||||
|         <Expanded> | ||||
|           "{ StringMaker<has_maker_and_toString> }" | ||||
| == | ||||
| "{ StringMaker<has_maker_and_toString> }" | ||||
|         </Expanded> | ||||
|       </Expression> | ||||
|       <OverallResult success="true"/> | ||||
|     </TestCase> | ||||
|     <TestCase name="toString( vectors<has_toString )"> | ||||
|       <Expression success="true" type="REQUIRE" filename="projects/<exe-name>/ToStringWhich.cpp" > | ||||
|         <Original> | ||||
|           Catch::toString( v ) == "{ {?} }" | ||||
|         </Original> | ||||
|         <Expanded> | ||||
|           "{ {?} }" == "{ {?} }" | ||||
|         </Expanded> | ||||
|       </Expression> | ||||
|       <OverallResult success="true"/> | ||||
|     </TestCase> | ||||
|     <TestCase name="toString(enum w/operator<<)"> | ||||
|       <Expression success="true" type="CHECK" filename="projects/<exe-name>/EnumToString.cpp" > | ||||
|         <Original> | ||||
| @@ -9527,7 +9503,7 @@ there" | ||||
|       </Section> | ||||
|       <OverallResult success="true"/> | ||||
|     </TestCase> | ||||
|     <OverallResults successes="819" failures="81" expectedFailures="18"/> | ||||
|     <OverallResults successes="817" failures="81" expectedFailures="18"/> | ||||
|   </Group> | ||||
|   <OverallResults successes="819" failures="80" expectedFailures="18"/> | ||||
|   <OverallResults successes="817" failures="80" expectedFailures="18"/> | ||||
| </Catch> | ||||
|   | ||||
| @@ -37,19 +37,21 @@ TEST_CASE( "toString( has_toString )", "[toString]" ) { | ||||
| } | ||||
|  | ||||
| // Call the overload | ||||
| TEST_CASE( "toString( has_maker )", "[toString]" ) { | ||||
| TEST_CASE( "toString( has_maker )", "toString]" ) { | ||||
|     has_maker item; | ||||
|     REQUIRE( Catch::toString( item ) == "StringMaker<has_maker>" ); | ||||
| } | ||||
|  | ||||
| // Call the overload | ||||
| TEST_CASE( "toString( has_maker_and_toString )", "[toString]" ) { | ||||
| TEST_CASE( "toString( has_maker_and_toString )", "[.][toString]" ) { | ||||
|     has_maker_and_toString item; | ||||
|     REQUIRE( Catch::toString( item ) == "toString( has_maker_and_toString )" ); | ||||
| } | ||||
|  | ||||
| // Vectors... | ||||
| TEST_CASE( "toString( vectors<has_toString )", "[toString]" ) { | ||||
|  | ||||
| // Don't run this in approval tests as it is sensitive to two phase lookup differences | ||||
| TEST_CASE( "toString( vectors<has_toString )", "[.][toString][!nonportable]" ) { | ||||
|     std::vector<has_toString> v(1); | ||||
|     // This invokes template<T> toString which actually gives us '{ ? }' | ||||
|     REQUIRE( Catch::toString( v ) == "{ {?} }" ); | ||||
| @@ -61,7 +63,8 @@ TEST_CASE( "toString( vectors<has_maker )", "[toString]" ) { | ||||
| } | ||||
|  | ||||
|  | ||||
| TEST_CASE( "toString( vectors<has_maker_and_toString )", "[toString]" ) { | ||||
| // Don't run this in approval tests as it is sensitive to two phase lookup differences | ||||
| TEST_CASE( "toString( vectors<has_maker_and_toString )", "[.][toString][!nonportable]" ) { | ||||
|     std::vector<has_maker_and_toString> v(1); | ||||
|     // Note: This invokes the template<T> toString -> StringMaker | ||||
|     REQUIRE( Catch::toString( v ) == "{ StringMaker<has_maker_and_toString> }" ); | ||||
|   | ||||
		Reference in New Issue
	
	Block a user
	 Phil Nash
					Phil Nash