ci(Unit Tests): change workflow to be inline with phpcsExtra standard.

By coping PHPCSExtra standard's workflow, it might fix some issues and allow the CI to work.

Also removed phpunit 10 and 11 support, and changed the composer scripts.

Author: yCodeTech

.github/workflows/unit-tests.yml

@@ -86,7 +86,7 @@ jobs:
           - php: "8.2"
             phpcs_version: "4.x-dev"
 
-    name: "Test: PHP ${{ matrix.php }}"
+    name: "Test: PHP ${{ matrix.php }} - PHPCS ${{ matrix.phpcs_version }}"
 
     steps:
       - name: Checkout code
@@ -110,8 +110,8 @@ jobs:
           ini-values: ${{ steps.set_ini.outputs.PHP_INI }}
           coverage: none
 
-      - name: "Composer: set PHPCS version for tests (dev)"
-        if: ${{ contains( matrix.phpcs_version, 'dev') }}
+      - name: "Composer: set PHPCS version for tests (dev/specific version)"
+        if: ${{ matrix.phpcs_version != 'lowest' && matrix.phpcs_version != 'stable' }}
         run: composer require squizlabs/php_codesniffer:"${{ matrix.phpcs_version }}" --no-update --no-scripts --no-interaction
 
       - name: "Composer: use lock file when necessary"
@@ -133,26 +133,18 @@ jobs:
       - name: Composer info
         run: composer info
 
-      - name: Grab PHPUnit version
-        id: phpunit_version
-        run: echo "VERSION=$(vendor/bin/phpunit --version | grep --only-matching --max-count=1 --extended-regexp '\b[0-9]+\.[0-9]+')" >> "$GITHUB_OUTPUT"
+      - name: Grab PHPCS version
+        id: phpcs_version
+        # yamllint disable-line rule:line-length
+        run: echo "VERSION=$(vendor/bin/phpcs --version | grep --only-matching --max-count=1 --extended-regexp '\b[0-9]+\.[0-9]+')" >> "$GITHUB_OUTPUT"
 
-      - name: "DEBUG: Show grabbed PHPUnit version"
-        run: echo ${{ steps.phpunit_version.outputs.VERSION }}
+      - name: "DEBUG: Show grabbed PHPCS version"
+        run: echo ${{ steps.phpcs_version.outputs.VERSION }}
 
-      - name: Determine PHPUnit config file to use
-        id: phpunit_config
-        shell: bash
-        run: |
-          if [ "${{ startsWith( steps.phpunit_version.outputs.VERSION, '11.' ) }}" == "true" ]; then
-            echo 'FILE=phpunit.xml' >> "$GITHUB_OUTPUT"
-          elif [ "${{ startsWith( steps.phpunit_version.outputs.VERSION, '10.' ) }}" == "true" ]; then
-            echo 'FILE=phpunit.xml' >> "$GITHUB_OUTPUT"
-          else
-            echo 'FILE=phpunit-lte9.xml' >> "$GITHUB_OUTPUT"
-          fi
+      - name: Run the unit tests (PHPCS 3.x)
+        if: ${{ startsWith( steps.phpcs_version.outputs.VERSION, '3.' ) }}
+        run: composer test-phpcs3
 
-      - name: Run the unit tests
-        run: composer test -- -c ${{ steps.phpunit_config.outputs.FILE }}
-        env:
-          PHPCS_VERSION: ${{ matrix.phpcs_version }}
+      - name: Run the unit tests (PHPCS 4.x)
+        if: ${{ startsWith( steps.phpcs_version.outputs.VERSION, '4.' ) }}
+        run: composer test-phpcs4

composer.json

@@ -22,7 +22,7 @@
         "dealerdirect/phpcodesniffer-composer-installer": "^1.1"
     },
     "require-dev": {
-        "phpunit/phpunit": "^8.5||^9.6||^10.5||^11.5",
+        "phpunit/phpunit": "^8.5||^9.6",
         "phpcsstandards/phpcsdevtools": "^1.0",
         "php-parallel-lint/php-parallel-lint": "^1.4"
     },
@@ -40,10 +40,14 @@
         "lint": "@php ./vendor/php-parallel-lint/php-parallel-lint/parallel-lint . -e php --exclude vendor --exclude .git --exclude test_utils",
         "cs": "@php ./vendor/bin/phpcs --standard=yCodeTech",
         "cbf": "@php ./vendor/bin/phpcbf --standard=yCodeTech",
-        "test": [
+        "test-phpcs4": [
             "Composer\\Config::disableProcessTimeout",
             "@php ./vendor/bin/phpunit --no-coverage"
         ],
+        "test-phpcs3": [
+            "Composer\\Config::disableProcessTimeout",
+            "@php ./vendor/bin/phpunit --filter yCodeTech --no-coverage ./vendor/squizlabs/php_codesniffer/tests/AllTests.php"
+        ],
         "cs-test-file": "@cs ./test_utils/TestFile.php",
         "fix-test-file": "@cbf ./test_utils/TestFile.php",
         "restore-test-file": "bash ./test_utils/restore_errors.sh",