Drop Laravel 9 support (#51)

* Drop Laravel 9 support

* fix phpstan

* update test

Author: norbybaru

.github/workflows/main.yml

@@ -16,7 +16,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.0'
+          php-version: '8.3'
           coverage: none
 
       - name: Install composer dependencies

.github/workflows/phpstan.yml

@@ -1,43 +1,44 @@
 name: Run Unit Tests
 
-on: pull_request
+on:
+  pull_request:
+  push:
+    branches:
+      - main
 
 jobs:
   build:
     runs-on: ubuntu-latest
+    # These permissions are needed to interact with GitHub's OIDC Token endpoint.
+    permissions:
+      id-token: write
+      contents: read
 
     strategy:
       fail-fast: false
       matrix:
-        php: [ 8.0, 8.1, 8.2, 8.3, 8.4 ]
-        laravel: [ 9.*, 10.*, 11.*, 12.* ]
-        dependency-version: [ prefer-lowest, prefer-stable ]
+        php: [8.2, 8.3, 8.4]
+        laravel: [10.*, 11.*, 12.*]
+        dependency-version: [prefer-lowest, prefer-stable]
         exclude:
-          - laravel: 9.*
-            php: 8.3
-          - laravel: 9.*
-            php: 8.4
-          - laravel: 10.*
-            php: 8.0
           - laravel: 10.*
             php: 8.4
-          - laravel: 11.*
-            php: 8.0
-          - laravel: 11.*
-            php: 8.1
           - laravel: 11.*
             php: 8.4
-          - laravel: 12.*
-            php: 8.0
-          - laravel: 12.*
-            php: 8.1
 
     name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }}
 
     steps:
       - name: Checkout
         uses: actions/checkout@v4
 
+      - name: Configure AWS credentials
+        if:  github.ref == 'refs/heads/main' && github.event_name != 'pull_request'
+        uses: aws-actions/configure-aws-credentials@v4
+        with:
+          role-to-assume: ${{ vars.AWS_ASSUME_ROLE }}
+          aws-region: ${{ secrets.AWS_REGION }}
+
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
@@ -60,6 +61,12 @@ jobs:
         run: |
           composer require "laravel/framework:${{ matrix.laravel }}" --no-interaction --no-update
           composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction
-
+      
       - name: Run PHPUnit tests
-        run: vendor/bin/phpunit --testsuite Unit
+        if: github.event_name == 'pull_request'
+        run: composer test:unit
+  
+      - name: Run PHPUnit tests
+        if:  github.ref == 'refs/heads/main' && github.event_name != 'pull_request'
+        run: composer test
+

.github/workflows/run-tests.yml

@@ -7,6 +7,8 @@ on:
 jobs:
   update:
     runs-on: ubuntu-latest
+    permissions:
+      contents: write
 
     steps:
       - name: Checkout code

.github/workflows/update-changelog.yml

@@ -30,25 +30,26 @@
         }
     ],
     "require": {
-        "php": "^8.0",
+        "php": "^8.1",
         "aws/aws-sdk-php": "^3.209",
-        "illuminate/support": "^9.52|^10.0|^11.0|^12.0",
+        "illuminate/support": "^10.0|^11.0|^12.0",
         "spatie/enum": "^3.13"
     },
     "minimum-stability": "dev",
     "prefer-stable": true,
     "scripts": {
         "analyse": "vendor/bin/phpstan analyse",
-        "test": "phpunit"
+        "test": "phpunit",
+        "test:unit": "phpunit --testsuite Unit"
     },
     "config": {
         "sort-packages": true
     },
     "require-dev": {
         "guzzlehttp/guzzle": "^7.4",
-        "nunomaduro/larastan": "^1.0|^2.0|^3.0",
-        "orchestra/testbench": "^7.0|^8.0|^9.0|^10.0",
-        "phpunit/phpunit": "^9.5|^10.0|^11.0|^12.0",
+        "larastan/larastan": "^2.0|^3.0",
+        "orchestra/testbench": "^8.0|^9.0|^10.0",
+        "phpunit/phpunit": "^10.0|^11.0|^12.0",
         "vlucas/phpdotenv": "^5.4"
     }
 }

composer.json

@@ -1,15 +1,12 @@
 includes:
-    - ./vendor/nunomaduro/larastan/extension.neon
+    - ./vendor/larastan/larastan/extension.neon
 
 parameters:
-
     paths:
         - src
-        - config
 
     # The level 9 is the highest level
     level: 5
 
     tmpDir: build/phpstan
     checkModelProperties: true
-    checkMissingIterableValueType: true

phpstan.neon.dist

@@ -55,7 +55,7 @@ public function setMultiMeasuresValues(string $name, mixed $value, ?ValueTypeEnu
         $this->measureValues[] = [
             'Name' => $name,
             'Value' => $value,
-            'Type' => $type?->value ?? ValueTypeEnum::VARCHAR()->value,
+            'Type' => $type->value ?? ValueTypeEnum::VARCHAR()->value,
         ];
 
         return $this;

src/Builder/TimestreamPayloadBuilder.php

@@ -40,6 +40,7 @@ public function it_should_ingest_multi_measure_records()
                 ...$payload->toRecords(),
             ];
 
+            // Write in batches of 100
             if (count($records) === 100) {
                 $timestreamWriter = TimestreamWriterDto::make($records)->forTable('test');
 

tests/Feature/PayloadWriterFeatureTest.php

@@ -29,7 +29,7 @@ public function test_it_should_return_results()
 
     public function test_it_should_return_paginated_results()
     {
-        $max = 5000;
+        $max = 300;
         $queryBuilder = TimestreamBuilder::query()
             ->select('*')
             ->from(config('timestream.database'), 'test')

tests/Feature/ReaderFeatureTest.php

@@ -60,7 +60,7 @@ protected function readCSV(string $filePath): \Generator
         $handle = fopen($filePath, "r");
 
         while (!feof($handle)) {
-            yield fgetcsv($handle, 5000);
+            yield fgetcsv($handle, 300);
         }
 
         fclose($handle);