Add support for arguments to the test runner

Author: shivammathur

.github/workflows/pecl.yml

@@ -40,6 +40,9 @@ on:
         description: 'Test runner to use'
         required: false
         default: 'run-tests.php'
+      test-runner-args:
+        description: 'Arguments for the test runner'
+        required: false
 jobs:
   get-extension-matrix:
     runs-on: ubuntu-latest
@@ -79,6 +82,7 @@ jobs:
           libs: ${{ inputs.libs }}
           run-tests: ${{ inputs.run-tests }}
           test-runner: ${{ inputs.test-runner }}
+          test-runner-args: ${{ inputs.test-runner-args }}
           build-directory: C:\build
         env:
           artifact-naming-scheme: pecl

README.md

@@ -78,6 +78,7 @@ Build a specific version of a PHP extension.
 - `build-directory` (optional) - The directory to build the extension in, defaults to the user's temporary directory.
 - `run-tests` (optional) - Run the extension tests. Defaults to `true`.
 - `test-runner` (optional) - The test runner to use. Defaults to `run-tests.php`.
+- `test-runner-args` (optional) - Arguments to pass to the test runner.
 - `test-opcache-mode` (optional) - Run tests with opcache `on`, `off` or `both`. Defaults to `off`.
 - `test-workers` (optional) - The number of workers to use when running tests. Defaults to `8`.
 - `auth-token` (optional) - Authentication token to use in case the extension is hosted on a private repository.

extension/BuildPhpExtension/private/Invoke-Tests.ps1

@@ -45,19 +45,24 @@ Function Invoke-Tests {
             } elseif(-not(Test-Path $env:TEST_RUNNER)) {
                 throw "Test runner $env:TEST_RUNNER does not exist."
             }
-            $test_runner_args = @(
-                '-q',
-                '--offline',
-                '--show-diff',
-                '--show-slow 1000',
-                '--set-timeout 120',
-                '-g FAIL,XFAIL,BORK,WARN,LEAK,SKIP'
-            )
+
+            if($null -ne $env:TEST_RUNNER_ARGS -and $env:TEST_RUNNER_ARGS -ne '') {
+                $test_runner_args = $env:TEST_RUNNER_ARGS -split '\s+'
+            } else {
+                $test_runner_args = @(
+                    '-q',
+                    '--offline',
+                    '--show-diff',
+                    '--show-slow 1000',
+                    '--set-timeout 120',
+                    '-g FAIL,XFAIL,BORK,WARN,LEAK,SKIP'
+                )
+            }
             $test_workers = 8
             if($null -ne $env:TEST_WORKERS -and $env:TEST_WORKERS -ne '') {
                 $test_workers = $env:TEST_WORKERS
             }
-            if($Config.php_version -ge '7.4') {
+            if($Config.php_version -ge '7.4' -and $test_runner_args -notcontains '-j') {
                 $test_runner_args += ('-j' + $test_workers)
             }
 

extension/action.yml

@@ -39,6 +39,9 @@ inputs:
     description: Test runner to use
     required: false
     default: 'run-tests.php'
+  test-runner-args:
+    description: Test runner to use
+    required: false
   test-opcache-mode:
     description: Test opcache mode
     required: false
@@ -67,6 +70,7 @@ runs:
         ARTIFACT_NAMING_SCHEME: ${{env.artifact-naming-scheme}}
         RUN_TESTS: ${{inputs.run-tests}}
         TEST_RUNNER: ${{inputs.test-runner}}
+        TEST_RUNNER_ARGS: ${{inputs.test-runner-args}}
         TEST_OPCACHE_MODE: ${{inputs.test-opcache-mode}}
         TEST_WORKERS: ${{inputs.test-workers}}
         BUILD_DIRECTORY: ${{inputs.build-directory}}