Skip to content

Commit def8714

Browse files
committed
🔥 Hot Fix: remove FileList tests
PR 1113 introduced a set of tests for the `FileList` class to accompany a bug fix. However, it now looks like these tests break the ability to run sniff tests for external standards using the PHPCS native test framework. This is problematic and even more so as this will silently fail in CI, as PHPUnit 9 and lower exits with a `0` exit code when no tests are found. For now, I'm removing the tests for the `FileList` class, while keeping the bugfix. I've confirmed this fixes the issue by running the tests of two external standards which both use the PHPCS native test framework. Both were not running their tests with PHPCS 3.13.3 and the tests run correctly again with the `FileList` class tests removed.
1 parent f1c9394 commit def8714

File tree

6 files changed

+0
-352
lines changed

6 files changed

+0
-352
lines changed

tests/Core/Files/FileList/AbstractFileListTestCase.php

Lines changed: 0 additions & 55 deletions
This file was deleted.

tests/Core/Files/FileList/AddFileTest.php

Lines changed: 0 additions & 138 deletions
This file was deleted.

tests/Core/Files/FileList/ConstructTest.php

Lines changed: 0 additions & 122 deletions
This file was deleted.

tests/Core/Files/FileList/FilterDouble.php

Lines changed: 0 additions & 31 deletions
This file was deleted.

tests/Core/Files/FileList/Fixtures/file1.php

Lines changed: 0 additions & 3 deletions
This file was deleted.

tests/Core/Files/FileList/Fixtures/file2.php

Lines changed: 0 additions & 3 deletions
This file was deleted.

0 commit comments

Comments
 (0)