Skip data provider build when requirements are not satisfied

Author: MauricioFauth

src/Framework/TestBuilder.php

@@ -12,6 +12,7 @@
 use function assert;
 use PHPUnit\Metadata\Api\DataProvider;
 use PHPUnit\Metadata\Api\Groups;
+use PHPUnit\Metadata\Api\Requirements;
 use PHPUnit\Metadata\BackupGlobals;
 use PHPUnit\Metadata\BackupStaticProperties;
 use PHPUnit\Metadata\ExcludeGlobalVariableFromBackup;
@@ -37,10 +38,11 @@ public function build(ReflectionClass $theClass, string $methodName): Test
     {
         $className = $theClass->getName();
 
-        $data = (new DataProvider)->providedData(
-            $className,
-            $methodName,
-        );
+        $data = null;
+
+        if ((new Requirements)->requirementsNotSatisfiedFor($className, $methodName) === []) {
+            $data = (new DataProvider)->providedData($className, $methodName);
+        }
 
         if ($data !== null) {
             return $this->buildDataProviderTestSuite(

tests/end-to-end/data-provider/requires-phpunit.phpt

@@ -14,34 +14,21 @@ PHPUnit %s by Sebastian Bergmann and contributors.
 
 Runtime: %s
 
-SSD.                                                                4 / 4 (100%)
+S..SS                                                               5 / 5 (100%)
 
 Time: %s, Memory: %s
 
-There were 2 PHPUnit errors:
+There were 3 skipped tests:
 
-1) PHPUnit\TestFixture\DataProviderRequiresPhpUnitTest::testWithDataProviderThatThrows
-The data provider specified for PHPUnit\TestFixture\DataProviderRequiresPhpUnitTest::testWithDataProviderThatThrows is invalid
-Should have been skipped.
-
-%s:%d
-
-2) PHPUnit\TestFixture\DataProviderRequiresPhpUnitTest::testWithDataProviderExternalThatThrows
-The data provider specified for PHPUnit\TestFixture\DataProviderRequiresPhpUnitTest::testWithDataProviderExternalThatThrows is invalid
-Should have been skipped.
-
-%s:%d
-
---
-
-There were 2 skipped tests:
+1) PHPUnit\TestFixture\DataProviderRequiresPhpUnitTest::testWithInvalidDataProvider
+PHPUnit < 10 is required.
 
-1) PHPUnit\TestFixture\DataProviderRequiresPhpUnitTest::testWithInvalidDataProvider with data set #0 (true)
+2) PHPUnit\TestFixture\DataProviderRequiresPhpUnitTest::testWithDataProviderThatThrows
 PHPUnit < 10 is required.
 
-2) PHPUnit\TestFixture\DataProviderRequiresPhpUnitTest::testWithInvalidDataProvider with data set #1 (true)
+3) PHPUnit\TestFixture\DataProviderRequiresPhpUnitTest::testWithDataProviderExternalThatThrows
 PHPUnit < 10 is required.
 
-ERRORS!
-Tests: 4, Assertions: 2, Errors: 2, PHPUnit Deprecations: 1, Skipped: 2.
+OK, but some tests were skipped!
+Tests: 5, Assertions: 2, Skipped: 3.
 

tests/end-to-end/generic/invalid-requirements.phpt

@@ -20,9 +20,9 @@ Time: %s, Memory: %s
 
 There were 2 PHPUnit test runner warnings:
 
-1) Method PHPUnit\TestFixture\InvalidRequirementsTest::testInvalidVersionConstraint is annotated using an invalid version requirement: Version constraint ~~9.0 is not supported.
+1) Class PHPUnit\TestFixture\InvalidRequirementsTest is annotated using an invalid version requirement: Version constraint ~~9.0 is not supported.
 
-2) Class PHPUnit\TestFixture\InvalidRequirementsTest is annotated using an invalid version requirement: Version constraint ~~9.0 is not supported.
+2) Method PHPUnit\TestFixture\InvalidRequirementsTest::testInvalidVersionConstraint is annotated using an invalid version requirement: Version constraint ~~9.0 is not supported.
 
 WARNINGS!
 Tests: 1, Assertions: 1, Warnings: 2.