Skip to content

Commit e7fa943

Browse files
committed
Generator tests: minor updates after review
1 parent 3dcbb7f commit e7fa943

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

tests/Core/Generators/GeneratorTest.php

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -147,8 +147,6 @@ public function testGeneratingDocs($standard, $expected)
147147
$config = new ConfigDouble(["--standard=$standard"]);
148148
$ruleset = new Ruleset($config);
149149

150-
// Make the test OS independent.
151-
$expected = str_replace("\n", PHP_EOL, $expected);
152150
$this->expectOutputString($expected);
153151

154152
$generator = new MockGenerator($ruleset);
@@ -164,14 +162,16 @@ public function testGeneratingDocs($standard, $expected)
164162
*/
165163
public static function dataGeneratingDocs()
166164
{
167-
$multidocExpected = "No Content\n";
168-
$multidocExpected .= "Code Comparison Only, Missing Standard Block\n";
169-
$multidocExpected .= "One Standard Block, Code Comparison\n";
170-
$multidocExpected .= "One Standard Block, No Code\n";
171-
$multidocExpected .= "One Standard Block, Two Code Comparisons\n";
172-
$multidocExpected .= "Two Standard Blocks, No Code\n";
173-
$multidocExpected .= "Two Standard Blocks, One Code Comparison\n";
174-
$multidocExpected .= "Two Standard Blocks, Three Code Comparisons\n";
165+
$multidocExpected = [];
166+
$multidocExpected[] = 'No Content';
167+
$multidocExpected[] = 'Code Comparison Only, Missing Standard Block';
168+
$multidocExpected[] = 'One Standard Block, Code Comparison';
169+
$multidocExpected[] = 'One Standard Block, No Code';
170+
$multidocExpected[] = 'One Standard Block, Two Code Comparisons';
171+
$multidocExpected[] = 'Two Standard Blocks, No Code';
172+
$multidocExpected[] = 'Two Standard Blocks, One Code Comparison';
173+
$multidocExpected[] = 'Two Standard Blocks, Three Code Comparisons';
174+
$multidocExpected = implode(PHP_EOL, $multidocExpected).PHP_EOL;
175175

176176
return [
177177
'Standard without docs' => [
@@ -180,7 +180,7 @@ public static function dataGeneratingDocs()
180180
],
181181
'Standard with one doc file' => [
182182
'standard' => __DIR__.'/OneDocTest.xml',
183-
'expected' => "One Standard Block, No Code\n",
183+
'expected' => 'One Standard Block, No Code'.PHP_EOL,
184184
],
185185
'Standard with multiple doc files' => [
186186
'standard' => __DIR__.'/StructureDocsTest.xml',
@@ -226,7 +226,7 @@ public function testGeneratorWillShowEachStandardSeparately()
226226
(?:.+?(?P>delimiter)\R){2} # Arbitrary text followed by a delimiter line.
227227
)* # Code comparison is optional and can exist multiple times.
228228
\R+
229-
){3,} # This complete group should occur at least six times.
229+
){3,} # This complete group should occur at least three times.
230230
`sx';
231231

232232
$this->expectOutputRegex($regex);

0 commit comments

Comments
 (0)