Convert test suite from PHPUnit to Pest (#2912)

* Convert test suite from PHPUnit to Pest

- Replace phpunit/phpunit with pestphp/pest and pestphp/pest-plugin-laravel
- Convert all 26 test files from class-based PHPUnit to Pest functional syntax
- Use Pest expectations (expect()) instead of PHPUnit assertions
- Create TestCase subclasses for files needing pre-setUp properties
- Add tests/Pest.php and tests/ContentPolicy.php
- Update GitHub Actions workflows to use vendor/bin/pest
- 599 tests, 1463 assertions all passing

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Fix CI: add pest-plugin to allow-plugins, drop PHP 8.0/8.1 and Laravel 8/9 from matrix

Pest v3 requires PHP ^8.2 and pest-plugin-laravel requires Laravel ^10+,
so these older combinations cannot be tested with Pest.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Reorganize test suite into subdirectories and simplify TestCase by replacing subclasses with setup methods

* Fix styling

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: freekmurze <483853+freekmurze@users.noreply.github.com>

Author: freekmurze

.github/workflows/run-tests.yml

@@ -9,8 +9,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [8.5, 8.4, 8.3, 8.2, 8.1, 8.0]
-        laravel: ["^12.0", "^11.0", "^10.0", "^9.0", "^8.12"]
+        php: [8.5, 8.4, 8.3, 8.2]
+        laravel: ["^12.0", "^11.0", "^10.0"]
         dependency-version: [prefer-lowest, prefer-stable]
         include:
           - laravel: "^12.0"
@@ -19,33 +19,11 @@ jobs:
             testbench: 9.*
           - laravel: "^10.0"
             testbench: 8.*
-          - laravel: "^9.0"
-            testbench: 7.*
-          - laravel: "^8.12"
-            testbench: "^6.23"
         exclude:
-            - laravel: "^12.0"
-              php: 8.1
-            - laravel: "^12.0"
-              php: 8.0
             - laravel: "^11.0"
               dependency-version: prefer-lowest
-            - laravel: "^11.0"
-              php: 8.1
-            - laravel: "^11.0"
-              php: 8.0
-            - laravel: "^10.0"
-              php: 8.0
             - laravel: "^10.0"
               php: 8.5
-            - laravel: "^9.0"
-              php: 8.5
-            - laravel: "^8.12"
-              php: 8.3
-            - laravel: "^8.12"
-              php: 8.4
-            - laravel: "^8.12"
-              php: 8.5
 
     name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }}
 
@@ -60,14 +38,10 @@ jobs:
           extensions: curl, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, iconv
           coverage: none
 
-      - name: Install dependencies (remove passport)
-        run: composer remove --dev laravel/passport --no-interaction --no-update
-        if: matrix.laravel == '^8.12'
-
       - name: Install dependencies
         run: |
           composer require "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" "symfony/console:>=4.3.4" "mockery/mockery:^1.3.2" "nesbot/carbon:>=2.72.6" --no-interaction --no-update
           composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction
 
       - name: Execute tests
-        run: vendor/bin/phpunit
+        run: vendor/bin/pest

.github/workflows/test-cache-drivers.yml

@@ -37,31 +37,31 @@ jobs:
 
       - name: Execute tests - memcached cache driver
         run: |
-          vendor/bin/phpunit
+          vendor/bin/pest
         env:
           CACHE_DRIVER: memcached
 
       - name: Execute tests - redis cache driver
         run: |
-          vendor/bin/phpunit
+          vendor/bin/pest
         env:
           CACHE_DRIVER: redis
           REDIS_PORT: ${{ job.services.redis.ports['6379'] }}
 
       - name: Execute tests - database cache driver
         run: |
-          vendor/bin/phpunit
+          vendor/bin/pest
         env:
           CACHE_DRIVER: database
 
       - name: Execute tests - file cache driver
         run: |
-          vendor/bin/phpunit
+          vendor/bin/pest
         env:
           CACHE_DRIVER: file
 
       - name: Execute tests - array cache driver
         run: |
-          vendor/bin/phpunit
+          vendor/bin/pest
         env:
           CACHE_DRIVER: array

composer.json

@@ -32,7 +32,8 @@
         "laravel/passport": "^11.0|^12.0",
         "laravel/pint": "^1.0",
         "orchestra/testbench": "^6.23|^7.0|^8.0|^9.0|^10.0",
-        "phpunit/phpunit": "^9.4|^10.1|^11.5"
+        "pestphp/pest": "^2.0|^3.0",
+        "pestphp/pest-plugin-laravel": "^2.0|^3.0"
     },
     "minimum-stability": "dev",
     "prefer-stable": true,
@@ -50,7 +51,10 @@
         }
     },
     "config": {
-        "sort-packages": true
+        "sort-packages": true,
+        "allow-plugins": {
+            "pestphp/pest-plugin": true
+        }
     },
     "extra": {
         "branch-alias": {
@@ -64,7 +68,7 @@
         }
     },
     "scripts": {
-        "test": "phpunit",
+        "test": "pest",
         "format": "pint",
         "analyse": "echo 'Checking dependencies...' && composer require --dev larastan/larastan && phpstan analyse"
     }

src/helpers.php

@@ -1,9 +1,12 @@
 <?php
 
+use Spatie\Permission\Guard;
+use Spatie\Permission\PermissionRegistrar;
+
 if (! function_exists('getModelForGuard')) {
     function getModelForGuard(string $guard): ?string
     {
-        return Spatie\Permission\Guard::getModelForGuard($guard);
+        return Guard::getModelForGuard($guard);
     }
 
 }
@@ -14,7 +17,7 @@ function getModelForGuard(string $guard): ?string
      */
     function setPermissionsTeamId($id)
     {
-        app(\Spatie\Permission\PermissionRegistrar::class)->setPermissionsTeamId($id);
+        app(PermissionRegistrar::class)->setPermissionsTeamId($id);
     }
 }
 
@@ -24,6 +27,6 @@ function setPermissionsTeamId($id)
      */
     function getPermissionsTeamId()
     {
-        return app(\Spatie\Permission\PermissionRegistrar::class)->getPermissionsTeamId();
+        return app(PermissionRegistrar::class)->getPermissionsTeamId();
     }
 }