Skip to content

Release/5.0.2 #79

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lennartvava merged 3 commits into from
Sep 18, 2026
Merged

Release/5.0.2 #79

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
73d66b3
Fix PHPUnit extension test discovery
Sep 18, 2026
cc58160
Merge pull request #78 from district09/feature/fix-phpunit-extension-…
lennartvava Sep 18, 2026
880bb0b
Merge branch 'main' into release/5.0.2
Sep 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,22 @@

All Notable changes to the **Quality Assurance - Drupal** package.

## [5.0.2]

### Fixed

- Limited extension PHPUnit test and source discovery to project paths. The
shared configuration now discovers tests from the `tests/src` root rather
than requiring every optional test-type directory. This prevents both
recursive Composer-dependency scans and errors for absent test directories.

## [5.0.1]

### Fixed

- Select the PHPUnit 11.5 extension and site configuration templates when the
installed test runner is PHPUnit 11.

## [5.0.0]

### Added
Expand Down Expand Up @@ -177,6 +193,8 @@ Older releases (2.x.x) see [CHANGELOG-2.x.md](CHANGELOG-1.x.md).

Older releases (1.x.x) see [CHANGELOG-1.x.md](CHANGELOG-1.x.md).

[5.0.2]: https://github.com/district09/php_package_qa-drupal/compare/5.0.1...5.0.2
[5.0.1]: https://github.com/district09/php_package_qa-drupal/compare/5.0.0...5.0.1
[5.0.0]: https://github.com/district09/php_package_qa-drupal/compare/4.1.4...5.0.0
[4.1.4]: https://github.com/district09/php_package_qa-drupal/compare/4.1.3...4.1.4
[4.1.3]: https://github.com/district09/php_package_qa-drupal/compare/4.1.2...4.1.3
Expand Down
23 changes: 3 additions & 20 deletions configs/phpunit-extension-11.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,25 +25,9 @@
</php>

<testsuites>
<testsuite name="unit">
<directory>**/tests/src/Unit</directory>
<exclude>vendor</exclude>
</testsuite>
<testsuite name="kernel">
<directory>**/tests/src/Kernel</directory>
<exclude>vendor</exclude>
</testsuite>
<testsuite name="functional">
<directory>**/tests/src/Functional</directory>
<exclude>vendor</exclude>
</testsuite>
<testsuite name="functional-javascript">
<directory>**/tests/src/FunctionalJavascript</directory>
<exclude>vendor</exclude>
</testsuite>
<testsuite name="build">
<directory>**/tests/src/Build</directory>
<exclude>vendor</exclude>
<testsuite name="tests">
<directory suffix="Test.php">tests/src</directory>
<directory suffix="Test.php">modules/**/tests/src</directory>
</testsuite>
</testsuites>

Expand All @@ -57,7 +41,6 @@
<source ignoreSuppressionOfDeprecations="true">
<include>
<directory suffix=".php">./src</directory>
<directory suffix=".php">./**/src</directory>
<directory suffix=".php">./modules/**/src</directory>
</include>
<exclude>
Expand Down
23 changes: 3 additions & 20 deletions configs/phpunit-extension.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,25 +25,9 @@
</php>

<testsuites>
<testsuite name="unit">
<directory>**/tests/src/Unit</directory>
<exclude>vendor</exclude>
</testsuite>
<testsuite name="kernel">
<directory>**/tests/src/Kernel</directory>
<exclude>vendor</exclude>
</testsuite>
<testsuite name="functional">
<directory>**/tests/src/Functional</directory>
<exclude>vendor</exclude>
</testsuite>
<testsuite name="functional-javascript">
<directory>**/tests/src/FunctionalJavascript</directory>
<exclude>vendor</exclude>
</testsuite>
<testsuite name="build">
<directory>**/tests/src/Build</directory>
<exclude>vendor</exclude>
<testsuite name="tests">
<directory suffix="Test.php">tests/src</directory>
<directory suffix="Test.php">modules/**/tests/src</directory>
</testsuite>
</testsuites>

Expand All @@ -57,7 +41,6 @@
<source ignoreSuppressionOfDeprecations="true">
<include>
<directory suffix=".php">./src</directory>
<directory suffix=".php">./**/src</directory>
<directory suffix=".php">./modules/**/src</directory>
</include>
<exclude>
Expand Down
15 changes: 15 additions & 0 deletions tests/src/Unit/GrumPHP/ConfigFileMergerTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,21 @@ public function testPhpunitConfigurationUsesCurrentRunnerSchema(): void {
);
}

/**
* Tests that extension test discovery is limited to project code.
*/
public function testExtensionPhpunitConfigurationExcludesVendorPaths(): void {
$task = (new \ReflectionClass(Phpunit::class))->newInstanceWithoutConstructor();
(new ConfigFileMerger())->mergeTaskConfig($task, TRUE);

$configuration = (string) file_get_contents('phpunit.qa-drupal.xml');
$document = new \DOMDocument();
self::assertTrue($document->loadXML($configuration));
self::assertStringContainsString('<directory suffix="Test.php">tests/src</directory>', $configuration);
self::assertStringNotContainsString('tests/src/Kernel', $configuration);
self::assertStringNotContainsString('<directory suffix=".php">./**/src', $configuration);
}

/**
* Runs the configuration merger with a PHPStan task instance.
*/
Expand Down
Loading