Changeset 54394

Timestamp:

10/05/2022 02:04:05 PM (2 years ago)

Author:

SergeyBiryukov

Message:

Tests: Ignore EOL differences in Style Engine API tests.

Unix vs. Windows EOL style mismatches can cause misleading failures in tests using the heredoc syntax (<<<) or multiline strings as the expected result.

This resolves two failures when running the test suite on Windows along the lines of:

1) Tests_Style_Engine_wpStyleEngineCSSRule::test_should_prettify_css_rule_output
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
 #Warning: Strings contain different line endings!
-'.baptiste {
-       margin-left: 0;
-       font-family: Detective Sans;
+'.baptiste {
+       margin-left: 0;
+       font-family: Detective Sans;
}'

/var/www/tests/phpunit/tests/style-engine/wpStyleEngineCssRule.php:159

Follow-up to [46612], [48443], [48466], [49691], [51135], [53282], [53319], [54156].

See #56467, #55652.

Location:

trunk/tests/phpunit/tests/style-engine

Files:

• wpStyleEngineCssRule.php (modified) (1 diff)
• wpStyleEngineProcessor.php (modified) (1 diff)

trunk/tests/phpunit/tests/style-engine/wpStyleEngineCssRule.php

@@ -157 +157 @@
 }';
 
-        $this->assertSame( $expected, $css_rule->get_css( true ) );
+        $this->assertSameIgnoreEOL( $expected, $css_rule->get_css( true ) );
     }
 }

trunk/tests/phpunit/tests/style-engine/wpStyleEngineProcessor.php

@@ -93 +93 @@
 }
 ';
-        $this->assertSame(
+        $this->assertSameIgnoreEOL(
             $expected,
             $a_wonderful_processor->get_css( array( 'prettify' => true ) )