[ci] Run required PR tests without path filters (#230)

Author: coisa

.github/workflows/tests.yml

@@ -48,13 +48,9 @@ jobs:
             php-version: ${{ steps.resolve.outputs.php-version }}
             php-version-source: ${{ steps.resolve.outputs.php-version-source }}
             test-matrix: ${{ steps.resolve.outputs.test-matrix }}
-            run-tests: ${{ steps.scope.outputs.run-tests }}
-            test-scope-reason: ${{ steps.scope.outputs.reason }}
 
         steps:
             - uses: actions/checkout@v6
-              with:
-                fetch-depth: 0
             - name: Checkout dev-tools workflow action source
               uses: actions/checkout@v6
               with:
@@ -68,47 +64,6 @@ jobs:
               id: resolve
               uses: ./.dev-tools-actions/.github/actions/php/resolve-version
 
-            - name: Resolve test workflow scope
-              id: scope
-              shell: bash
-              env:
-                BASE_SHA: ${{ github.event.pull_request.base.sha || '' }}
-              run: |
-                set -euo pipefail
-
-                if [ "${GITHUB_EVENT_NAME}" != "pull_request" ]; then
-                  {
-                    echo "run-tests=true"
-                    echo "reason=non-pull-request-event"
-                  } >> "$GITHUB_OUTPUT"
-
-                  exit 0
-                fi
-
-                git fetch --no-tags --depth=1 origin "${BASE_SHA}"
-                git diff --name-only "${BASE_SHA}...HEAD" > changed-files.txt
-
-                run_tests=false
-                while IFS= read -r changed_file; do
-                  case "${changed_file}" in
-                    src/*|tests/*|composer.json|composer.lock|.github/actions/*|.github/workflows/tests.yml|resources/github-actions/tests.yml)
-                      run_tests=true
-                      break
-                      ;;
-                  esac
-                done < changed-files.txt
-
-                if [ "${run_tests}" = "true" ]; then
-                  reason="test-sensitive-pr-diff"
-                else
-                  reason="no-test-sensitive-pr-diff"
-                fi
-
-                {
-                  echo "run-tests=${run_tests}"
-                  echo "reason=${reason}"
-                } >> "$GITHUB_OUTPUT"
-
     tests:
         needs: resolve_php
         name: Run Tests
@@ -119,9 +74,7 @@ jobs:
             TESTS_ROOT_VERSION: ${{ github.event_name == 'pull_request' && format('dev-{0}', github.event.pull_request.head.ref) || 'dev-main' }}
         steps:
             - uses: actions/checkout@v6
-              if: needs.resolve_php.outputs.run-tests == 'true'
             - name: Checkout dev-tools workflow action source
-              if: needs.resolve_php.outputs.run-tests == 'true'
               uses: actions/checkout@v6
               with:
                 repository: php-fast-forward/dev-tools
@@ -131,7 +84,6 @@ jobs:
                   .github/actions
 
             - name: Setup PHP and install dependencies
-              if: needs.resolve_php.outputs.run-tests == 'true'
               uses: ./.dev-tools-actions/.github/actions/php/setup-composer
               with:
                 php-version: ${{ matrix.php-version }}
@@ -141,29 +93,21 @@ jobs:
                 install-options: --prefer-dist --no-progress --no-interaction --no-plugins --no-scripts
 
             - name: Composer Audit
-              if: needs.resolve_php.outputs.run-tests == 'true'
               env:
                 COMPOSER_ROOT_VERSION: ${{ env.TESTS_ROOT_VERSION }}
               run: composer audit
 
             - name: Resolve minimum coverage
-              if: needs.resolve_php.outputs.run-tests == 'true'
               id: minimum-coverage
               run: echo "value=${INPUT_MIN_COVERAGE:-80}" >> "$GITHUB_OUTPUT"
               env:
                   INPUT_MIN_COVERAGE: ${{ inputs.min-coverage }}
 
             - name: Run PHPUnit tests
-              if: needs.resolve_php.outputs.run-tests == 'true'
               env:
                 COMPOSER_ROOT_VERSION: ${{ env.TESTS_ROOT_VERSION }}
               run: composer dev-tools tests -- --coverage=.dev-tools/coverage --min-coverage=${{ steps.minimum-coverage.outputs.value }}
 
-            - name: Mark tests intentionally skipped
-              if: needs.resolve_php.outputs.run-tests != 'true'
-              run: |
-                echo "No test-sensitive PR diff detected; required test context completed without running PHPUnit."
-
     dependency-health:
         needs: resolve_php
         name: Dependency Health
@@ -172,9 +116,7 @@ jobs:
             TESTS_ROOT_VERSION: ${{ github.event_name == 'pull_request' && format('dev-{0}', github.event.pull_request.head.ref) || 'dev-main' }}
         steps:
             - uses: actions/checkout@v6
-              if: needs.resolve_php.outputs.run-tests == 'true'
             - name: Checkout dev-tools workflow action source
-              if: needs.resolve_php.outputs.run-tests == 'true'
               uses: actions/checkout@v6
               with:
                 repository: php-fast-forward/dev-tools
@@ -184,24 +126,17 @@ jobs:
                   .github/actions
 
             - name: Setup PHP and install dependencies
-              if: needs.resolve_php.outputs.run-tests == 'true'
               uses: ./.dev-tools-actions/.github/actions/php/setup-composer
               with:
                 php-version: ${{ needs.resolve_php.outputs.php-version }}
                 root-version: ${{ env.TESTS_ROOT_VERSION }}
                 install-options: --prefer-dist --no-progress --no-interaction --no-plugins --no-scripts
 
             - name: Run dependency health check
-              if: needs.resolve_php.outputs.run-tests == 'true'
               env:
                 COMPOSER_ROOT_VERSION: ${{ env.TESTS_ROOT_VERSION }}
               run: composer dev-tools dependencies -- --max-outdated=${{ inputs.max-outdated || -1 }}
 
-            - name: Mark dependency health intentionally skipped
-              if: needs.resolve_php.outputs.run-tests != 'true'
-              run: |
-                echo "No test-sensitive PR diff detected; dependency health completed without running dependency analysis."
-
     summarize:
         if: ${{ always() }}
         name: Summarize Test Workflow
@@ -229,8 +164,6 @@ jobs:
                   - Workflow PHP version: `${{ needs.resolve_php.outputs.php-version }}`
                   - PHP version source: `${{ needs.resolve_php.outputs.php-version-source }}`
                   - Test matrix: `${{ needs.resolve_php.outputs.test-matrix }}`
-                  - Test-sensitive diff: `${{ needs.resolve_php.outputs.run-tests }}`
-                  - Test scope reason: `${{ needs.resolve_php.outputs.test-scope-reason }}`
                   - Minimum coverage threshold: `${{ inputs.min-coverage || 80 }}`
                   - Dependency health `max-outdated`: `${{ inputs.max-outdated || -1 }}`
                   - Tests job result: `${{ needs.tests.result }}`

CHANGELOG.md

@@ -9,7 +9,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
-- Keep required PHPUnit matrix checks reporting after workflow-managed `.github/wiki` pointer commits by moving PR test-skip decisions inside the tests workflow and aligning the packaged consumer test wrapper (#230)
+- Keep required PHPUnit matrix checks reporting after workflow-managed `.github/wiki` pointer commits by running the pull-request test workflow without top-level path filters and aligning the packaged consumer test wrapper (#230)
 
 ## [1.21.0] - 2026-04-24
 

docs/advanced/branch-protection-and-bot-commits.rst

@@ -97,20 +97,12 @@ to refresh generated pointers manually. The preferred path is to allow bot
 updates on PR branches while keeping ``main`` protected.
 
 Required test checks must still report for workflow-managed pointer commits.
-The tests workflow therefore triggers on every pull request update and resolves
-whether the effective pull request diff contains test-sensitive files inside the
-workflow itself. If the latest commit only refreshes ``.github/wiki`` and the
-pull request has no source, test, Composer, test-workflow, packaged test-wrapper,
-or local-action changes, the required ``Run Tests`` matrix jobs complete with an
-intentional skip message instead of leaving branch protection waiting for
-missing checks. If the pull request does include test-sensitive changes, the
-matrix runs even when the newest commit is only the generated wiki pointer.
-
-Test workflow concurrency cancels older in-progress runs for the same pull
-request. Because the skip decision uses the effective pull request diff instead
-of only the latest commit, a generated wiki-pointer commit still runs the matrix
-when the pull request contains source, test, Composer, test-workflow, packaged
-test-wrapper, or local-action changes.
+The tests workflow therefore triggers on every pull request update without
+top-level path filters. This ensures GitHub always creates the required
+``Run Tests`` matrix checks for the latest pull request head, including bot
+commits that only refresh ``.github/wiki``. Test workflow concurrency cancels
+older in-progress runs for the same pull request so the newest commit owns the
+required check contexts.
 
 At a high level, the workflows need permission to read repository contents,
 write generated preview commits, update pull request comments, and publish Pages