Check for requirements before creating a separate process

Signed-off-by: Maurício Meneghini Fauth <[email protected]>
sebastianbergmann · Sep 26, 2024 · 0858a4b · 0858a4b
1 parent 5f53b1f
commit 0858a4b
Show file tree

Hide file tree

Showing 7 changed files with 25 additions and 37 deletions.
diff --git a/src/Framework/TestCase.php b/src/Framework/TestCase.php
@@ -513,7 +513,7 @@ final public function run(): void
  return;
  }
 
- if (!$this->shouldRunInSeparateProcess()) {
+ if (!$this->shouldRunInSeparateProcess() || (new Requirements)->requirementsNotSatisfiedFor(static::class, $this->name) !== []) {
  (new TestRunner)->run($this);
  } else {
  (new TestRunner)->runInSeparateProcess(

diff --git a/tests/end-to-end/regression/4391-separate-class-requires-in-class.phpt b/tests/end-to-end/regression/4391-separate-class-requires-in-class.phpt
@@ -23,11 +23,9 @@ PHPUnit %s by Sebastian Bergmann and contributors.
 
 Runtime: %s
 
+SS 2 / 2 (100%)
 
+Time: %s, Memory: %s
 
-An error occurred inside PHPUnit.
-
-Message: Call to undefined function proc_open()
-Location: %s:%d
-
-%a
+OK, but some tests were skipped!
+Tests: 2, Assertions: 0, Skipped: 2.
diff --git a/tests/end-to-end/regression/4391-separate-class-requires-in-method.phpt b/tests/end-to-end/regression/4391-separate-class-requires-in-method.phpt
@@ -23,11 +23,9 @@ PHPUnit %s by Sebastian Bergmann and contributors.
 
 Runtime: %s
 
+SS 2 / 2 (100%)
 
+Time: %s, Memory: %s
 
-An error occurred inside PHPUnit.
-
-Message: Call to undefined function proc_open()
-Location: %s:%d
-
-%a
+OK, but some tests were skipped!
+Tests: 2, Assertions: 0, Skipped: 2.
diff --git a/tests/end-to-end/regression/4391-separate-requires-in-class.phpt b/tests/end-to-end/regression/4391-separate-requires-in-class.phpt
@@ -23,11 +23,9 @@ PHPUnit %s by Sebastian Bergmann and contributors.
 
 Runtime: %s
 
+S 1 / 1 (100%)
 
+Time: %s, Memory: %s
 
-An error occurred inside PHPUnit.
-
-Message: Call to undefined function proc_open()
-Location: %s:%d
-
-%a
+OK, but some tests were skipped!
+Tests: 1, Assertions: 0, Skipped: 1.
diff --git a/tests/end-to-end/regression/4391-separate-requires-in-method.phpt b/tests/end-to-end/regression/4391-separate-requires-in-method.phpt
@@ -23,11 +23,9 @@ PHPUnit %s by Sebastian Bergmann and contributors.
 
 Runtime: %s
 
+S 1 / 1 (100%)
 
+Time: %s, Memory: %s
 
-An error occurred inside PHPUnit.
-
-Message: Call to undefined function proc_open()
-Location: %s:%d
-
-%a
+OK, but some tests were skipped!
+Tests: 1, Assertions: 0, Skipped: 1.
diff --git a/tests/end-to-end/regression/4391-separate-tests-requires-in-class.phpt b/tests/end-to-end/regression/4391-separate-tests-requires-in-class.phpt
@@ -23,11 +23,9 @@ PHPUnit %s by Sebastian Bergmann and contributors.
 
 Runtime: %s
 
+SS 2 / 2 (100%)
 
+Time: %s, Memory: %s
 
-An error occurred inside PHPUnit.
-
-Message: Call to undefined function proc_open()
-Location: %s:%d
-
-%a
+OK, but some tests were skipped!
+Tests: 2, Assertions: 0, Skipped: 2.
diff --git a/tests/end-to-end/regression/4391-separate-tests-requires-in-method.phpt b/tests/end-to-end/regression/4391-separate-tests-requires-in-method.phpt
@@ -23,11 +23,9 @@ PHPUnit %s by Sebastian Bergmann and contributors.
 
 Runtime: %s
 
+SS 2 / 2 (100%)
 
+Time: %s, Memory: %s
 
-An error occurred inside PHPUnit.
-
-Message: Call to undefined function proc_open()
-Location: %s:%d
-
-%a
+OK, but some tests were skipped!
+Tests: 2, Assertions: 0, Skipped: 2.