Merge pull request #27 from nextapps-be/php82-support

PHP 8.2 support

Author: gdebrauwer

.github/workflows/run-linting.yml

@@ -13,20 +13,20 @@ jobs:
               run: sudo apt-get update --fix-missing
 
             - name: Checkout code
-              uses: actions/checkout@v2
+              uses: actions/checkout@v3
 
             - name: Setup PHP
               uses: shivammathur/setup-php@v2
               with:
-                php-version: 8.1
+                php-version: 8.2
 
             - name: Get Composer cache cirectory
               id: composer-cache
               run: |
                 echo "::set-output name=dir::$(composer config cache-files-dir)"
 
             - name: Cache Composer packages
-              uses: actions/cache@v2
+              uses: actions/cache@v3
               with:
                 path: ${{ steps.composer-cache.outputs.dir }}
                 key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
@@ -39,5 +39,5 @@ jobs:
 
             - name: Execute linting
               run: |
-                vendor/bin/php-cs-fixer fix --dry-run
+                PHP_CS_FIXER_IGNORE_ENV=1 vendor/bin/php-cs-fixer fix --dry-run
                 vendor/bin/phpcs --colors --report-full

.github/workflows/run-tests.yml

@@ -10,10 +10,17 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                php: [7.3, 7.4, 8.0, 8.1]
+                php: [7.3, 7.4, 8.0, 8.1, 8.2]
                 laravel: [7.*, 8.*, 9.*, 10.*]
                 dependency-version: [prefer-lowest, prefer-stable]
                 exclude:
+                    -   php: 8.2
+                        laravel: 9.*
+                        dependency-version: prefer-lowest
+                    -   php: 8.2
+                        laravel: 8.*
+                    -   php: 8.2
+                        laravel: 7.*
                     -   php: 8.1
                         laravel: 8.*
                         dependency-version: prefer-lowest
@@ -43,7 +50,7 @@ jobs:
               run: sudo apt-get update --fix-missing
 
             - name: Checkout code
-              uses: actions/checkout@v2
+              uses: actions/checkout@v3
 
             - name: Setup PHP
               uses: shivammathur/setup-php@v2
@@ -57,7 +64,7 @@ jobs:
                 echo "::set-output name=dir::$(composer config cache-files-dir)"
 
             - name: Cache Composer packages
-              uses: actions/cache@v2
+              uses: actions/cache@v3
               with:
                 path: ${{ steps.composer-cache.outputs.dir }}
                 key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}

composer.json

@@ -28,7 +28,7 @@
         }
     ],
     "require": {
-        "php": "^7.3|^7.4|^8.0|^8.1",
+        "php": "^7.3|^7.4|^8.0|^8.1|^8.2",
         "illuminate/database": "^7.0|^8.0|^9.0|^10.0",
         "illuminate/notifications": "^7.0|^8.0|^9.0|^10.0",
         "illuminate/support": "^7.0|^8.0|^9.0|^10.0"

tests/Feature/SendVerificationCodeTest.php

@@ -51,14 +51,15 @@ function ($notification, $channels, $notifiable) {
     /** @test */
     public function it_sends_notification_using_provided_channel()
     {
-        $this->mock(NotificationFake::class, function ($mock) {
-            $mock->shouldReceive('route')
-                ->with('a-random-channel', 'taylor@laravel.com')
-                ->andReturnSelf()
-                ->shouldReceive('notify');
-        });
+        if (! method_exists(NotificationFake::class, 'assertSentOnDemand')) {
+            $this->markTestSkipped('assertSentOnDemand method not available on NotificationFake');
+        }
 
         VerificationCodeFacade::send('taylor@laravel.com', 'a-random-channel');
+
+        Notification::assertSentOnDemand(function (VerificationCodeCreated $notification, array $channels, object $notifiable) {
+            return $notifiable->routes === ['a-random-channel' => 'taylor@laravel.com'];
+        });
     }
 
     /** @test */