Merge pull request #56 from understand/feature/gh-actions-and-compatibility

Feature/gh actions and compatibility

Author: mauricius

.github/workflows/run-test.yml

@@ -0,0 +1,61 @@
+on: push
+name: run-tests
+jobs:
+  php-tests:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        php: [8.0, 7.4, 7.3, 7.2]
+        laravel: [8.*, 7.*, 6.*, 5.8.*, 5.7.*, 5.6.*, 5.5.*]
+
+        include:
+          - laravel: 8.*
+            testbench: 6.*
+          - laravel: 7.*
+            testbench: 5.*
+          - laravel: 6.*
+            testbench: 4.*
+          - laravel: 5.8.*
+            testbench: 3.8.*
+          - laravel: 5.7.*
+            testbench: 3.7.*
+          - laravel: 5.6.*
+            testbench: 3.6.*
+          - laravel: 5.5.*
+            testbench: 3.5.*
+
+        exclude:
+          - laravel: 8.*
+            php: 7.2
+          - laravel: 5.8.*
+            php: 8.0
+          - laravel: 5.7.*
+            php: 8.0
+          - laravel: 5.6.*
+            php: 8.0
+          - laravel: 5.5.*
+            php: 8.0
+          - laravel: 5.5.*
+            php: 7.4
+
+    name: P${{ matrix.php }} - L${{ matrix.laravel }}
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          extensions: curl
+          coverage: none
+
+      - name: Install dependencies
+        run: |
+            composer require "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" --no-interaction --no-update
+            composer update --prefer-dist --no-interaction
+
+      - name: Execute tests
+        run: vendor/bin/phpunit

.travis.yml

@@ -1,6 +1,6 @@
 {
     "name": "understand/understand-laravel",
-    "description": "Laravel 5, 6 and 7 service provider for Understand.io",
+    "description": "Laravel 5, 6, 7 and 8 service provider for Understand.io",
     "keywords": ["laravel", "understand.io", "understand", "logs", "laravel 5", "Laravel 6"],
     "license": "MIT",
     "authors": [
@@ -10,13 +10,13 @@
         }
     ],
     "require": {
-        "php": ">=5.5.0",
+        "php": "^5.5 || ^7.0 || ^8.0",
         "ext-curl": "*",
-        "illuminate/support": "~5.0|~6.0|~7.0|~8.0"
+        "illuminate/support": "^5.0 || ^6.0 || ^7.0 || ^8.0"
     },
     "require-dev": {
-        "phpunit/phpunit": "~6.0",
-        "orchestra/testbench": "^3.5",
+        "phpunit/phpunit": "^6.0 || ^7.0 || ^8.0 || ^9.0",
+        "orchestra/testbench": "3.* || 4.* || 5.* || 6.*",
         "mockery/mockery": "^1.2"
     },
     "autoload": {

composer.json

@@ -55,7 +55,7 @@ public function testStackTraceSerializationWithoutArgs()
 
         $this->assertEmpty($stackTraceArray[0]['args']);
     }
-    
+
     public function testUndefinedFunctionIndex()
     {
         $stackTrace = debug_backtrace();
@@ -66,31 +66,31 @@ public function testUndefinedFunctionIndex()
 
         $this->assertEmpty($stackTraceArray[0]['function']);
     }
-    
+
     public function testIncompleteClass()
     {
         $catched = false;
-        
-        try 
+
+        try
         {
             // trigger invalid argument exception
             // to test __PHP_Incomplete_Class argument serialisation in `stackTraceToArray`
             // `Object of class __PHP_Incomplete_Class could not be converted to string`
             $incompleteObject = unserialize('O:1:"a":1:{s:5:"value";s:3:"100";}');
 
             with(new Understand\UnderstandLaravel5\ExceptionEncoder())->exceptionToArray($incompleteObject);
-            
+
             $this->fail('Should never be reached');
-        } 
-        catch (\Exception $exception) 
+        }
+        catch (Exception $exception)
         {
             $this->assertIncompleteClassStackTrace($exception, 0);
             $catched = true;
         }
-        
+
         $this->assertTrue($catched);
     }
-    
+
     /**
      * @param object $exception
      * @param integer $index
@@ -101,7 +101,12 @@ protected function assertIncompleteClassStackTrace($exception, $index)
         $encoder = new Understand\UnderstandLaravel5\ExceptionEncoder();
         $stackTraceArray = $encoder->stackTraceToArray($exception->getTrace());
 
-        if (Str::startsWith(phpversion(), ['7.2', '7.3']))
+        if (Str::startsWith(phpversion(), ['7.4', '8.0'])) {
+            // As of PHP 7.4 Exception::getTrace()
+            // no longer contains "args" keys by default
+            $this->assertSame([], $stackTraceArray[$index]['args']);
+        }
+        else if (Str::startsWith(phpversion(), ['7.2', '7.3']))
         {
             $this->assertSame('__PHP_Incomplete_Class', $stackTraceArray[$index]['args'][0]);
         }