upgrade PHP requirement: 8.0 → 8.1, code adjustments

Author: enmaboya

.github/workflows/ci.yml

@@ -10,7 +10,6 @@ jobs:
       max-parallel: 3
       matrix:
         php:
-          - '8.0'
           - '8.1'
           - '8.2'
           - '8.3'
@@ -20,33 +19,17 @@ jobs:
         normalize: [ false ]
         validate: [ false ]
         laravel:
-          - '8.22'
-          - '9.0'
-          - '10.0'
-          - '11.0.3'
-          - '12.0'
+          - '10.48.29'
+          - '11.45.1'
+          - '12.19.3'
         exclude:
-          - php: '8.0'
-            laravel: '9.0'
-          - php: '8.0'
-            laravel: '10.0'
           - php: '8.1'
-            laravel: '8.22'
+            laravel: '11.45.1'
           - php: '8.1'
-            laravel: '9.0'
-          - php: '8.2'
-            laravel: '8.22'
-          - php: '8.2'
-            laravel: '9.0'
-          - php: '8.3'
-            laravel: '8.22'
-          - php: '8.3'
-            laravel: '9.0'
-          - php: '8.4'
-            laravel: '12.0'
+            laravel: '12.19.3'
         include:
-          - php: '8.0'
-            laravel: '9.39.0'
+          - php: '8.1'
+            laravel: '10.48.29'
             analysis: true
             coverage: 'xdebug'
             normalize: true
@@ -69,9 +52,6 @@ jobs:
         if: matrix.validate == true
         run: composer validate --strict
 
-      - name: Install Laravel legacy factories support
-        if: matrix.laravel != '7.0'
-        run: composer require "laravel/legacy-factories:^1.1" --dev --no-interaction --no-update
 
       #- name: Install Laravel and Orchestra Testbench
       # run: composer require "illuminate/contracts:${{ matrix.laravel }}" --no-interaction --no-update
@@ -94,11 +74,11 @@ jobs:
         run: composer install --prefer-dist --no-progress
 
       - name: Normalize composer file
-        if: matrix.normalize == true && matrix.php != '8.0'
+        if: matrix.normalize == true
         run: composer normalize --dry-run
 
       - name: Run test suite
-        run: vendor/bin/phpunit -v
+        run: vendor/bin/phpunit
 
       - name: Run static code analysis
         if: matrix.analysis == true

composer.json

@@ -24,20 +24,19 @@
         }
     ],
     "require": {
-        "php": "^8.0 || ^8.1 || ^8.2 || ^8.3 || ^8.4",
+        "php": "^8.1 || ^8.2 || ^8.3 || ^8.4",
         "ext-json": "*",
         "funeralzone/valueobjects": "^0.5",
         "jenssegers/agent": "^2.6",
-        "laravel/framework": "^8.0 || ^9.0 || ^10.0 || ^11.0 || ^12.0",
-        "gnikyt/basic-shopify-api": "^9.0 || ^10.0 || ^11.0"
+        "laravel/framework": "^10.0 || ^11.0 || ^12.0",
+        "gnikyt/basic-shopify-api": "^11.0"
     },
     "require-dev": {
         "laravel/legacy-factories": "^v1.3.0",
         "ergebnis/composer-normalize": "^2.8",
         "friendsofphp/php-cs-fixer": "^3.0",
         "mockery/mockery": "^1.0",
-        "orchestra/database": "~3.8 || ~4.0 || ~5.0 || ~6.0 || ~7.0 || ~8.0 || ~9.0",
-        "orchestra/testbench": "~3.8 || ~4.0 || ~5.0 || ~6.0 || ~7.0 || ~8.0 || ~9.0",
+        "orchestra/testbench": "^8.0 || ^9.0",
         "phpstan/phpstan": "^0.12",
         "phpunit/phpunit": "~8.0 || ^9.0 || ^10.0 || ^11.0"
     },

phpunit.xml.dist

@@ -1,20 +1,21 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/8.0/phpunit.xsd"
-         backupGlobals="false"
-         backupStaticAttributes="false"
-         colors="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
-         processIsolation="false"
-         stopOnError="true"
-         stopOnFailure="true"
-         stopOnIncomplete="false"
-         stopOnSkipped="false"
-         verbose="false"
->
-    <coverage>
+    xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd" backupGlobals="false"
+    colors="true" processIsolation="false" stopOnError="true" stopOnFailure="true"
+    stopOnIncomplete="false" stopOnSkipped="false" cacheDirectory=".phpunit.cache"
+    backupStaticProperties="false">
+    <testsuites>
+        <testsuite name="Laravel Shopify Test Suite">
+            <directory>tests</directory>
+        </testsuite>
+    </testsuites>
+    <logging />
+    <php>
+        <env name="APP_KEY" value="AckfSECXIvnK5r28GVIWUAxmbBSjTsmF" />
+        <env name="SHOPIFY_API_KEY" value="00000000000000000000000000000000" />
+        <env name="SHOPIFY_API_SECRET" value="00000000000000000000000000000000" />
+    </php>
+    <source>
         <include>
             <directory suffix=".php">src/</directory>
         </include>
@@ -26,20 +27,5 @@
             <directory>src/Messaging/Events/</directory>
             <file>src/ShopifyAppProvider.php</file>
         </exclude>
-        <report>
-            <clover outputFile="build/logs/clover.xml"/>
-            <text outputFile="php://stdout"/>
-        </report>
-    </coverage>
-    <testsuites>
-        <testsuite name="Laravel Shopify Test Suite">
-            <directory>tests</directory>
-        </testsuite>
-    </testsuites>
-    <logging/>
-    <php>
-        <env name="APP_KEY" value="AckfSECXIvnK5r28GVIWUAxmbBSjTsmF"/>
-        <env name="SHOPIFY_API_KEY" value="00000000000000000000000000000000"/>
-        <env name="SHOPIFY_API_SECRET" value="00000000000000000000000000000000"/>
-    </php>
+    </source>
 </phpunit>

src/resources/database/factories/ChargeFactory.php

@@ -12,7 +12,7 @@
 $factory->define($chargeModel, function (Faker $faker) {
     return [
         'charge_id' => $faker->randomNumber(8),
-        'name' => $faker->word,
+        'name' => $faker->word(),
         'price' => $faker->randomFloat(),
         'status' => ChargeStatus::ACCEPTED()->toNative(),
     ];

src/resources/database/factories/PlanFactory.php

@@ -10,15 +10,15 @@
 
 $factory->define($planModel, function (Faker $faker) {
     return [
-        'name' => $faker->word,
+        'name' => $faker->word(),
         'price' => $faker->randomFloat(),
     ];
 });
 
 $factory->state($planModel, 'usage', function ($faker) {
     return [
         'capped_amount' => $faker->randomFloat(),
-        'terms' => $faker->sentence,
+        'terms' => $faker->sentence(),
     ];
 });
 

src/resources/database/factories/ShopFactory.php

@@ -7,9 +7,9 @@
 
 $factory->define($model, function (Faker $faker) {
     return [
-        'name' => "{$faker->domainWord}.myshopify.com",
-        'password' => str_replace('-', '', $faker->uuid),
-        'email' => $faker->email,
+        'name' => "{$faker->domainWord()}.myshopify.com",
+        'password' => str_replace('-', '', $faker->uuid()),
+        'email' => $faker->email(),
     ];
 });
 

tests/TestCase.php

@@ -10,7 +10,7 @@
 use Illuminate\Support\Carbon;
 use Illuminate\Support\Facades\App;
 use Illuminate\Support\Facades\Request as FacadesRequest;
-use Orchestra\Database\ConsoleServiceProvider;
+// use Orchestra\Database\ConsoleServiceProvider;
 use Orchestra\Testbench\TestCase as OrchestraTestCase;
 use Osiset\ShopifyApp\Contracts\ShopModel;
 use Osiset\ShopifyApp\Objects\Values\Hmac;
@@ -81,7 +81,7 @@ protected function getPackageProviders($app): array
         // ConsoleServiceProvider required to make migrations work
         return [
             ShopifyAppProvider::class,
-            ConsoleServiceProvider::class,
+            // ConsoleServiceProvider::class,
         ];
     }