Skip to content

Commit af41534

Browse files
jrfnljrfnl
committed
Tests: split off two tests for the Runner class
... to their own test classes using the new `AbstractRunnerTestCase` base class.
1 parent 90e4083 commit af41534

File tree

4 files changed

+145
-103
lines changed

4 files changed

+145
-103
lines changed

‎tests/Core/Generators/GeneratorTest.php‎

Lines changed: 0 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
namespace PHP_CodeSniffer\Tests\Core\Generators;
1010

1111
use PHP_CodeSniffer\Ruleset;
12-
use PHP_CodeSniffer\Runner;
1312
use PHP_CodeSniffer\Tests\ConfigDouble;
1413
use PHP_CodeSniffer\Tests\Core\Generators\Fixtures\MockGenerator;
1514
use PHPUnit\Framework\TestCase;
@@ -225,54 +224,4 @@ public function testGetTitleFallbackToFilename()
225224
}//end testGetTitleFallbackToFilename()
226225

227226

228-
/**
229-
* Test that the documentation for each standard passed on the command-line is shown separately.
230-
*
231-
* @covers \PHP_CodeSniffer\Runner::runPHPCS
232-
*
233-
* @return void
234-
*/
235-
public function testGeneratorWillShowEachStandardSeparately()
236-
{
237-
if (PHP_CODESNIFFER_CBF === true) {
238-
$this->markTestSkipped('This test needs CS mode to run');
239-
}
240-
241-
$standard = __DIR__.'/OneDocTest.xml';
242-
$_SERVER['argv'] = [
243-
'phpcs',
244-
'--generator=Text',
245-
"--standard=$standard,PSR1",
246-
'--report-width=80',
247-
];
248-
249-
$regex = '`^
250-
\R* # Optional blank line at the start.
251-
(?:
252-
(?P<delimiter>-++\R) # Line with dashes.
253-
\|[ ]GENERATORTEST[ ]CODING[ ]STANDARD:[ ][^\|]+\|\R # Doc title line with prefix expected for first standard.
254-
(?P>delimiter) # Line with dashes.
255-
\R(?:[^\r\n]+\R)+\R{2} # Standard description.
256-
) # Only expect this group once.
257-
(?:
258-
(?P>delimiter) # Line with dashes.
259-
\|[ ]PSR1[ ]CODING[ ]STANDARD:[ ][^\|]+\|\R # Doc title line with prefix expected for second standard.
260-
(?P>delimiter) # Line with dashes.
261-
\R(?:[^\r\n]+\R)+\R # Standard description.
262-
(?:
263-
-+[ ]CODE[ ]COMPARISON[ ]-+\R # Code Comparison starter line with dashes.
264-
(?:(?:[^\r\n]+\R)+(?P>delimiter)){2} # Arbitrary text followed by a delimiter line.
265-
)* # Code comparison is optional and can exist multiple times.
266-
\R+
267-
){3,} # This complete group should occur at least three times.
268-
`x';
269-
270-
$this->expectOutputRegex($regex);
271-
272-
$runner = new Runner();
273-
$runner->runPHPCS();
274-
275-
}//end testGeneratorWillShowEachStandardSeparately()
276-
277-
278227
}//end class

‎tests/Core/Ruleset/ExplainTest.php‎

Lines changed: 0 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
namespace PHP_CodeSniffer\Tests\Core\Ruleset;
1111

1212
use PHP_CodeSniffer\Ruleset;
13-
use PHP_CodeSniffer\Runner;
1413
use PHP_CodeSniffer\Tests\ConfigDouble;
1514
use PHPUnit\Framework\TestCase;
1615

@@ -210,55 +209,4 @@ public function testExplainWithDeprecatedSniffs()
210209
}//end testExplainWithDeprecatedSniffs()
211210

212211

213-
/**
214-
* Test that each standard passed on the command-line is explained separately.
215-
*
216-
* @covers \PHP_CodeSniffer\Runner::runPHPCS
217-
*
218-
* @return void
219-
*/
220-
public function testExplainWillExplainEachStandardSeparately()
221-
{
222-
if (PHP_CODESNIFFER_CBF === true) {
223-
$this->markTestSkipped('This test needs CS mode to run');
224-
}
225-
226-
$standard = __DIR__.'/ExplainSingleSniffTest.xml';
227-
$_SERVER['argv'] = [
228-
'phpcs',
229-
'-e',
230-
"--standard=PSR1,$standard",
231-
'--report-width=80',
232-
];
233-
234-
$expected = PHP_EOL;
235-
$expected .= 'The PSR1 standard contains 8 sniffs'.PHP_EOL.PHP_EOL;
236-
$expected .= 'Generic (4 sniffs)'.PHP_EOL;
237-
$expected .= '------------------'.PHP_EOL;
238-
$expected .= ' Generic.Files.ByteOrderMark'.PHP_EOL;
239-
$expected .= ' Generic.NamingConventions.UpperCaseConstantName'.PHP_EOL;
240-
$expected .= ' Generic.PHP.DisallowAlternativePHPTags'.PHP_EOL;
241-
$expected .= ' Generic.PHP.DisallowShortOpenTag'.PHP_EOL.PHP_EOL;
242-
$expected .= 'PSR1 (3 sniffs)'.PHP_EOL;
243-
$expected .= '---------------'.PHP_EOL;
244-
$expected .= ' PSR1.Classes.ClassDeclaration'.PHP_EOL;
245-
$expected .= ' PSR1.Files.SideEffects'.PHP_EOL;
246-
$expected .= ' PSR1.Methods.CamelCapsMethodName'.PHP_EOL.PHP_EOL;
247-
$expected .= 'Squiz (1 sniff)'.PHP_EOL;
248-
$expected .= '---------------'.PHP_EOL;
249-
$expected .= ' Squiz.Classes.ValidClassName'.PHP_EOL.PHP_EOL;
250-
251-
$expected .= 'The ExplainSingleSniffTest standard contains 1 sniff'.PHP_EOL.PHP_EOL;
252-
$expected .= 'Squiz (1 sniff)'.PHP_EOL;
253-
$expected .= '---------------'.PHP_EOL;
254-
$expected .= ' Squiz.Scope.MethodScope'.PHP_EOL;
255-
256-
$this->expectOutputString($expected);
257-
258-
$runner = new Runner();
259-
$runner->runPHPCS();
260-
261-
}//end testExplainWillExplainEachStandardSeparately()
262-
263-
264212
}//end class

‎tests/Core/Runner/RunPHPCSExplainTest.php‎

Lines changed: 73 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,73 @@
1+
<?php
2+
/**
3+
* Tests the wiring in of the explain functionality in the Runner class.
4+
*
5+
* @author Juliette Reinders Folmer <phpcs_nospam@adviesenzo.nl>
6+
* @copyright 2023 Juliette Reinders Folmer. All rights reserved.
7+
* @license https://github.com/PHPCSStandards/PHP_CodeSniffer/blob/master/licence.txt BSD Licence
8+
*/
9+
10+
namespace PHP_CodeSniffer\Tests\Core\Runner;
11+
12+
use PHP_CodeSniffer\Runner;
13+
use PHP_CodeSniffer\Tests\Core\Runner\AbstractRunnerTestCase;
14+
15+
/**
16+
* Tests the wiring in of the explain functionality in the Runner class.
17+
*
18+
* @covers \PHP_CodeSniffer\Runner::runPHPCS
19+
*/
20+
final class RunPHPCSExplainTest extends AbstractRunnerTestCase
21+
{
22+
23+
24+
/**
25+
* Test that each standard passed on the command-line is explained separately.
26+
*
27+
* @return void
28+
*/
29+
public function testExplainWillExplainEachStandardSeparately()
30+
{
31+
if (PHP_CODESNIFFER_CBF === true) {
32+
$this->markTestSkipped('This test needs CS mode to run');
33+
}
34+
35+
$standard = dirname(__DIR__).'/Ruleset/ExplainSingleSniffTest.xml';
36+
$_SERVER['argv'] = [
37+
'phpcs',
38+
'-e',
39+
"--standard=PSR1,$standard",
40+
'--report-width=80',
41+
];
42+
43+
$expected = PHP_EOL;
44+
$expected .= 'The PSR1 standard contains 8 sniffs'.PHP_EOL.PHP_EOL;
45+
$expected .= 'Generic (4 sniffs)'.PHP_EOL;
46+
$expected .= '------------------'.PHP_EOL;
47+
$expected .= ' Generic.Files.ByteOrderMark'.PHP_EOL;
48+
$expected .= ' Generic.NamingConventions.UpperCaseConstantName'.PHP_EOL;
49+
$expected .= ' Generic.PHP.DisallowAlternativePHPTags'.PHP_EOL;
50+
$expected .= ' Generic.PHP.DisallowShortOpenTag'.PHP_EOL.PHP_EOL;
51+
$expected .= 'PSR1 (3 sniffs)'.PHP_EOL;
52+
$expected .= '---------------'.PHP_EOL;
53+
$expected .= ' PSR1.Classes.ClassDeclaration'.PHP_EOL;
54+
$expected .= ' PSR1.Files.SideEffects'.PHP_EOL;
55+
$expected .= ' PSR1.Methods.CamelCapsMethodName'.PHP_EOL.PHP_EOL;
56+
$expected .= 'Squiz (1 sniff)'.PHP_EOL;
57+
$expected .= '---------------'.PHP_EOL;
58+
$expected .= ' Squiz.Classes.ValidClassName'.PHP_EOL.PHP_EOL;
59+
60+
$expected .= 'The ExplainSingleSniffTest standard contains 1 sniff'.PHP_EOL.PHP_EOL;
61+
$expected .= 'Squiz (1 sniff)'.PHP_EOL;
62+
$expected .= '---------------'.PHP_EOL;
63+
$expected .= ' Squiz.Scope.MethodScope'.PHP_EOL;
64+
65+
$this->expectOutputString($expected);
66+
67+
$runner = new Runner();
68+
$runner->runPHPCS();
69+
70+
}//end testExplainWillExplainEachStandardSeparately()
71+
72+
73+
}//end class

‎tests/Core/Runner/RunPHPCSGeneratorTest.php‎

Lines changed: 72 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
1+
<?php
2+
/**
3+
* Tests the wiring in of the Generator functionality in the Runner class.
4+
*
5+
* @copyright 2024 PHPCSStandards and contributors
6+
* @license https://github.com/PHPCSStandards/PHP_CodeSniffer/blob/master/licence.txt BSD Licence
7+
*/
8+
9+
namespace PHP_CodeSniffer\Tests\Core\Runner;
10+
11+
use PHP_CodeSniffer\Runner;
12+
use PHP_CodeSniffer\Tests\Core\Runner\AbstractRunnerTestCase;
13+
14+
/**
15+
* Tests the wiring in of the Generator functionality in the Runner class.
16+
*
17+
* @covers \PHP_CodeSniffer\Runner::runPHPCS
18+
* @group Windows
19+
*/
20+
final class RunPHPCSGeneratorTest extends AbstractRunnerTestCase
21+
{
22+
23+
24+
/**
25+
* Test that the documentation for each standard passed on the command-line is shown separately.
26+
*
27+
* @return void
28+
*/
29+
public function testGeneratorWillShowEachStandardSeparately()
30+
{
31+
if (PHP_CODESNIFFER_CBF === true) {
32+
$this->markTestSkipped('This test needs CS mode to run');
33+
}
34+
35+
$standard = dirname(__DIR__).'/Generators/OneDocTest.xml';
36+
$_SERVER['argv'] = [
37+
'phpcs',
38+
'--generator=Text',
39+
"--standard=$standard,PSR1",
40+
'--report-width=80',
41+
];
42+
43+
$regex = '`^
44+
\R* # Optional blank line at the start.
45+
(?:
46+
(?P<delimiter>-++\R) # Line with dashes.
47+
\|[ ]GENERATORTEST[ ]CODING[ ]STANDARD:[ ][^\|]+\|\R # Doc title line with prefix expected for first standard.
48+
(?P>delimiter) # Line with dashes.
49+
\R(?:[^\r\n]+\R)+\R{2} # Standard description.
50+
) # Only expect this group once.
51+
(?:
52+
(?P>delimiter) # Line with dashes.
53+
\|[ ]PSR1[ ]CODING[ ]STANDARD:[ ][^\|]+\|\R # Doc title line with prefix expected for second standard.
54+
(?P>delimiter) # Line with dashes.
55+
\R(?:[^\r\n]+\R)+\R # Standard description.
56+
(?:
57+
-+[ ]CODE[ ]COMPARISON[ ]-+\R # Code Comparison starter line with dashes.
58+
(?:(?:[^\r\n]+\R)+(?P>delimiter)){2} # Arbitrary text followed by a delimiter line.
59+
)* # Code comparison is optional and can exist multiple times.
60+
\R+
61+
){3,} # This complete group should occur at least three times.
62+
`x';
63+
64+
$this->expectOutputRegex($regex);
65+
66+
$runner = new Runner();
67+
$runner->runPHPCS();
68+
69+
}//end testGeneratorWillShowEachStandardSeparately()
70+
71+
72+
}//end class

0 commit comments

Comments
 (0)