Drop 7.2 support. Bump Laravel 7 to 26. Cover early framework bug.

Author: aarondfrancis

.github/workflows/tests.yml

@@ -9,7 +9,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ 7.2, 7.3, 7.4, 8.0 ]
+        php: [ 7.3, 7.4, 8.0 ]
         laravel: [ 7.*, 8.* ]
         dependency-version: [ prefer-lowest, prefer-stable ]
         include:
@@ -23,11 +23,6 @@ jobs:
           - laravel: 8.*
             testbench: 6.*
 
-        exclude:
-          # Laravel 8 requires PHP 7.3.
-          - laravel: 8.*
-            php: 7.2
-
     name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }}
 
     steps:

composer.json

@@ -10,10 +10,10 @@
         }
     ],
     "require": {
-        "php": "^7.2|^8.0",
-        "illuminate/support": "^7.7.0|^8.0",
-        "illuminate/console": "^7.7.0|^8.0",
-        "illuminate/http": "^7.7.0|^8.0",
+        "php": "^7.3|^8.0",
+        "illuminate/support": "^7.26.0|^8.0",
+        "illuminate/console": "^7.26.0|^8.0",
+        "illuminate/http": "^7.26.0|^8.0",
         "guzzlehttp/guzzle": "^7.2"
     },
     "require-dev": {

tests/MiddlewareAndComponentTest.php

@@ -16,7 +16,6 @@ public function getEnvironmentSetUp($app)
     {
         config()->set('torchlight.blade_components', true);
         config()->set('torchlight.token', 'token');
-
     }
 
     protected function getView($view)
@@ -30,13 +29,15 @@ protected function getView($view)
 
     protected function nullApiResponse()
     {
+        $this->withoutExceptionHandling();
         Http::fake([
             'api.torchlight.dev/*' => Http::response(null, 200),
         ]);
     }
 
     protected function legitApiResponse()
     {
+        $this->withoutExceptionHandling();
         $response = [
             "blocks" => [[
                 "id" => "real_response_id",
@@ -84,6 +85,7 @@ public function it_sends_a_simple_request_with_highlighted_response()
 
         $this->assertEquals(
             '<code class="torchlight" style="background-color: #292D3E;">this is the highlighted response from the server</code>',
+            // See https://github.com/laravel/framework/pull/35874/files for the rtrim reasoning.
             rtrim($response->content())
         );
     }