Skip to content

Commit e11d8f2

Browse files
authored
Merge pull request #685 from PHPCSStandards/feature/ruleinclusionabsolutewindowstest-add-group
RuleInclusionAbsoluteWindowsTest: minor tweaks
2 parents f9c5f18 + 3d64cb2 commit e11d8f2

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

tests/Core/Ruleset/RuleInclusionAbsoluteWindowsTest.php

+4-8
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,9 @@
1616
/**
1717
* Tests for the \PHP_CodeSniffer\Ruleset class using a Windows-style absolute path to include a sniff.
1818
*
19-
* @covers \PHP_CodeSniffer\Ruleset
19+
* @covers \PHP_CodeSniffer\Ruleset
20+
* @requires OS ^WIN.*.
21+
* @group Windows
2022
*/
2123
final class RuleInclusionAbsoluteWindowsTest extends TestCase
2224
{
@@ -52,10 +54,6 @@ final class RuleInclusionAbsoluteWindowsTest extends TestCase
5254
*/
5355
public function initializeConfigAndRuleset()
5456
{
55-
if (DIRECTORY_SEPARATOR === '/') {
56-
$this->markTestSkipped('Windows specific test');
57-
}
58-
5957
$this->standard = __DIR__.'/'.basename(__FILE__, '.php').'.xml';
6058
$repoRootDir = dirname(dirname(dirname(__DIR__)));
6159

@@ -85,9 +83,7 @@ public function initializeConfigAndRuleset()
8583
*/
8684
public function resetRuleset()
8785
{
88-
if (DIRECTORY_SEPARATOR !== '/') {
89-
file_put_contents($this->standard, $this->contents);
90-
}
86+
file_put_contents($this->standard, $this->contents);
9187

9288
}//end resetRuleset()
9389

0 commit comments

Comments
 (0)