Skip to content

Commit 584e004

Browse files
committed
Tests: add @group Windows to more tests
As part of PR 663, test runs on the Windows OS where introduced in CI/GH Actions. While, generally speaking, all tests should run and pass on Windows too (and they do), for the purposes of CI, a specific test group named `Windows` was introduced to allow for selectively only running the tests which _may_ have a different outcome on Windows. This group is used in the code coverage job to prevent the Windows builds taking forever. While most PHPCS code is OS agnostic, there are a number of places in the code base where OS-based conditions are used. This commit adds the `@group Windows` annotation to few more tests which hits these OS-based conditions to make sure those conditions are also hit in the code coverage builds.
1 parent 76b2aa7 commit 584e004

File tree

2 files changed

+3
-0
lines changed

2 files changed

+3
-0
lines changed

src/Standards/Generic/Tests/Debug/CSSLintUnitTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@
1616
* Unit test class for the CSSLint sniff.
1717
*
1818
* @covers \PHP_CodeSniffer\Standards\Generic\Sniffs\Debug\CSSLintSniff
19+
* @covers \PHP_CodeSniffer\Config::getExecutablePath
20+
* @group Windows
1921
*/
2022
final class CSSLintUnitTest extends AbstractSniffUnitTest
2123
{

tests/Core/Fixer/GenerateDiffTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
* test running the fixer itself, nor generating a diff based on a fixer run.
2222
*
2323
* @covers PHP_CodeSniffer\Fixer::generateDiff
24+
* @group Windows
2425
*/
2526
final class GenerateDiffTest extends TestCase
2627
{

0 commit comments

Comments
 (0)