Drop support for PHP 8.0, Laravel 9

iv-craig · iv-craig · commit e6b578635215 · 2024-04-02T17:39:42.000-04:00
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -10,7 +10,7 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        php: [8.0, 8.1, 8.2, 8.3]
+        php: [8.1, 8.2, 8.3]
 
     name: Tests on PHP ${{ matrix.php }} - ${{ matrix.stability }}
 
diff --git a/composer.json b/composer.json
@@ -12,18 +12,18 @@
         }
     ],
     "require": {
-        "php": "^8.0",
-        "illuminate/notifications": "^9.0|^10.0|^11.0",
-        "illuminate/support": "^9.0|^10.0|^11.0",
+        "php": "^8.1",
+        "illuminate/notifications": "^10.0|^11.0",
+        "illuminate/support": "^10.0|^11.0",
         "interfax/interfax": "^2.0",
         "psr/log": "^2.0"
     },
     "require-dev": {
         "mockery/mockery": "^1.0",
         "mpdf/mpdf": "^8.0",
-        "orchestra/testbench": "^7.0|^8.0|^9.0",
+        "orchestra/testbench": "^8.0|^9.0",
         "phpstan/phpstan": "^1.9",
-        "phpunit/phpunit": "^9.5.10|^10.0|^11.0"
+        "phpunit/phpunit": "^10.0|^11.0"
     },
     "autoload": {
         "psr-4": {
diff --git a/tests/CouldNotSendNotificationExceptionTest.php b/tests/CouldNotSendNotificationExceptionTest.php
@@ -2,7 +2,6 @@
 
 namespace NotificationChannels\Interfax\Test;
 
-use PHPUnit\Framework\Attributes\Test;
 use NotificationChannels\Interfax\Exceptions\CouldNotSendNotification;
 use NotificationChannels\Interfax\InterfaxMessage;
 
@@ -20,8 +19,7 @@ public function setUp(): void
                             ->file('test-file.pdf');
     }
 
-    #[Test]
-    public function it_can_get_the_exception_user()
+    public function test_get_the_exception_user()
     {
         $exception = CouldNotSendNotification::serviceRespondedWithAnError($this->message, [
             'status' => 500,
@@ -31,8 +29,7 @@ public function it_can_get_the_exception_user()
         $this->assertInstanceOf(TestNotifiable::class, $exception->getUser());
     }
 
-    #[Test]
-    public function it_can_get_the_exception_metadata()
+    public function test_get_the_exception_metadata()
     {
         $exception = CouldNotSendNotification::serviceRespondedWithAnError($this->message, [
             'status' => 500,
@@ -42,8 +39,7 @@ public function it_can_get_the_exception_metadata()
         $this->assertSame('Some sample metadata.', $exception->getMetadata()['key']);
     }
 
-    #[Test]
-    public function it_can_get_the_default_exception_message()
+    public function test_get_the_default_exception_message()
     {
         $exception = CouldNotSendNotification::serviceRespondedWithAnError($this->message, [
             'status' => 500,
@@ -53,8 +49,7 @@ public function it_can_get_the_default_exception_message()
         $this->assertSame('The fax failed to send via InterFAX.', $exception->getMessage());
     }
 
-    #[Test]
-    public function it_can_get_a_custom_exception_message()
+    public function test_get_a_custom_exception_message()
     {
         $exceptionMessage = 'This is a test.';
 
@@ -66,8 +61,7 @@ public function it_can_get_a_custom_exception_message()
         $this->assertSame($exceptionMessage, $exception->getMessage());
     }
 
-    #[Test]
-    public function it_can_get_the_exception_attributes()
+    public function test_get_the_exception_attributes()
     {
         $exception = CouldNotSendNotification::serviceRespondedWithAnError($this->message, [
             'status' => 500,
diff --git a/tests/InterfaxChannelTest.php b/tests/InterfaxChannelTest.php
@@ -3,7 +3,6 @@
 namespace NotificationChannels\Interfax\Test;
 
 use Illuminate\Notifications\Notification;
-use PHPUnit\Framework\Attributes\Test;
 use Interfax\Client as InterfaxClient;
 use Interfax\Resource as InterfaxResource;
 use Mockery;
@@ -36,8 +35,7 @@ public function setUp(): void
         $this->channel = new InterfaxChannel($this->interfax);
     }
 
-    #[Test]
-    public function it_can_send_notification_with_a_single_file()
+    public function test_send_notification_with_a_single_file()
     {
         $this->interfax->expects('deliver')
             ->once()
@@ -56,8 +54,7 @@ public function it_can_send_notification_with_a_single_file()
         $this->channel->send(new TestNotifiable, new TestNotificationWithSingleFile);
     }
 
-    #[Test]
-    public function it_can_send_notification_with_files()
+    public function test_send_notification_with_files()
     {
         $this->interfax->expects('deliver')
             ->once()
@@ -82,8 +79,7 @@ public function it_can_send_notification_with_files()
         $this->channel->send(new TestNotifiable, new TestNotificationWithFiles);
     }
 
-    #[Test]
-    public function it_can_send_notification_pdf_as_stream()
+    public function test_send_notification_pdf_as_stream()
     {
         $this->interfax->expects('deliver')
             ->with(Mockery::on(function ($output) {
@@ -111,8 +107,7 @@ public function it_can_send_notification_pdf_as_stream()
         $this->channel->send(new TestNotifiable, new TestNotificationAsStreamPdf);
     }
 
-    #[Test]
-    public function it_can_send_notification_html_as_stream()
+    public function test_send_notification_html_as_stream()
     {
         $filename = 'test-file.html';
         $this->addFile($filename);
@@ -145,14 +140,12 @@ public function it_can_send_notification_html_as_stream()
         $this->channel->send(new TestNotifiable, new TestNotificationAsStreamHtml);
     }
 
-    #[Test]
-    public function it_can_return_early_when_no_fax_number_provided()
+    public function test_return_early_when_no_fax_number_provided()
     {
         $this->assertNull($this->channel->send(new TestNotifiableNotSendable, new TestNotificationWithFiles));
     }
 
-    #[Test]
-    public function it_can_refresh_the_file_response()
+    public function test_refresh_the_file_response()
     {
         $this->resource
              ->expects('refresh')
@@ -170,8 +163,7 @@ public function it_can_refresh_the_file_response()
         $this->channel->send(new TestNotifiable, new TestNotificationWithRefresh);
     }
 
-    #[Test]
-    public function it_can_throw_the_exception()
+    public function test_throw_the_exception()
     {
         $this->expectException(CouldNotSendNotification::class);
 
diff --git a/tests/InterfaxMessageTest.php b/tests/InterfaxMessageTest.php
@@ -2,13 +2,11 @@
 
 namespace NotificationChannels\Interfax\Test;
 
-use PHPUnit\Framework\Attributes\Test;
 use NotificationChannels\Interfax\InterfaxMessage;
 
 class InterfaxMessageTest extends TestCase
 {
-    #[Test]
-    public function it_should_check_the_status_via_refresh()
+    public function test_check_the_status_via_refresh()
     {
         $message = (new InterfaxMessage)
                         ->checkStatus()
@@ -18,8 +16,7 @@ public function it_should_check_the_status_via_refresh()
         $this->assertTrue($message->shouldCheckStatus());
     }
 
-    #[Test]
-    public function it_should_not_check_the_status_via_refresh_manual()
+    public function test_not_check_the_status_via_refresh_manual()
     {
         $message = (new InterfaxMessage)
                         ->checkStatus(false)
@@ -29,8 +26,7 @@ public function it_should_not_check_the_status_via_refresh_manual()
         $this->assertFalse($message->shouldCheckStatus());
     }
 
-    #[Test]
-    public function it_should_not_check_the_status_via_refresh_default()
+    public function test_not_check_the_status_via_refresh_default()
     {
         $message = (new InterfaxMessage)
                         ->user(new TestNotifiable)
@@ -39,8 +35,7 @@ public function it_should_not_check_the_status_via_refresh_default()
         $this->assertFalse($message->shouldCheckStatus());
     }
 
-    #[Test]
-    public function it_should_set_the_file_chunk_size_filename()
+    public function test_set_the_file_chunk_size_filename()
     {
         $this->increaseChunkSize();
 
@@ -54,8 +49,7 @@ public function it_should_set_the_file_chunk_size_filename()
         $this->assertSame($this->chunkSize, $this->getChunkSize($delivery));
     }
 
-    #[Test]
-    public function it_should_set_the_file_chunk_size_file_array()
+    public function test_set_the_file_chunk_size_file_array()
     {
         $this->increaseChunkSize();
 
@@ -69,8 +63,7 @@ public function it_should_set_the_file_chunk_size_file_array()
         $this->assertSame($this->chunkSize, $this->getChunkSize($delivery));
     }
 
-    #[Test]
-    public function it_should_set_the_file_chunk_size_file_object()
+    public function test_set_the_file_chunk_size_file_object()
     {
         $this->increaseChunkSize();
         $client = new \Interfax\Client;

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,6 @@`
`2`	`2`
`3`	`3`	`namespace NotificationChannels\Interfax\Test;`
`4`	`4`
`5`		`-use PHPUnit\Framework\Attributes\Test;`
`6`	`5`	`use NotificationChannels\Interfax\Exceptions\CouldNotSendNotification;`
`7`	`6`	`use NotificationChannels\Interfax\InterfaxMessage;`
`8`	`7`
`@@ -20,8 +19,7 @@ public function setUp(): void`
`20`	`19`	`->file('test-file.pdf');`
`21`	`20`	`}`
`22`	`21`
`23`		`- #[Test]`
`24`		`- public function it_can_get_the_exception_user()`
	`22`	`+ public function test_get_the_exception_user()`
`25`	`23`	`{`
`26`	`24`	`$exception = CouldNotSendNotification::serviceRespondedWithAnError($this->message, [`
`27`	`25`	`'status' => 500,`
`@@ -31,8 +29,7 @@ public function it_can_get_the_exception_user()`
`31`	`29`	`$this->assertInstanceOf(TestNotifiable::class, $exception->getUser());`
`32`	`30`	`}`
`33`	`31`
`34`		`- #[Test]`
`35`		`- public function it_can_get_the_exception_metadata()`
	`32`	`+ public function test_get_the_exception_metadata()`
`36`	`33`	`{`
`37`	`34`	`$exception = CouldNotSendNotification::serviceRespondedWithAnError($this->message, [`
`38`	`35`	`'status' => 500,`
`@@ -42,8 +39,7 @@ public function it_can_get_the_exception_metadata()`
`42`	`39`	`$this->assertSame('Some sample metadata.', $exception->getMetadata()['key']);`
`43`	`40`	`}`
`44`	`41`
`45`		`- #[Test]`
`46`		`- public function it_can_get_the_default_exception_message()`
	`42`	`+ public function test_get_the_default_exception_message()`
`47`	`43`	`{`
`48`	`44`	`$exception = CouldNotSendNotification::serviceRespondedWithAnError($this->message, [`
`49`	`45`	`'status' => 500,`
`@@ -53,8 +49,7 @@ public function it_can_get_the_default_exception_message()`
`53`	`49`	`$this->assertSame('The fax failed to send via InterFAX.', $exception->getMessage());`
`54`	`50`	`}`
`55`	`51`
`56`		`- #[Test]`
`57`		`- public function it_can_get_a_custom_exception_message()`
	`52`	`+ public function test_get_a_custom_exception_message()`
`58`	`53`	`{`
`59`	`54`	`$exceptionMessage = 'This is a test.';`
`60`	`55`
`@@ -66,8 +61,7 @@ public function it_can_get_a_custom_exception_message()`
`66`	`61`	`$this->assertSame($exceptionMessage, $exception->getMessage());`
`67`	`62`	`}`
`68`	`63`
`69`		`- #[Test]`
`70`		`- public function it_can_get_the_exception_attributes()`
	`64`	`+ public function test_get_the_exception_attributes()`
`71`	`65`	`{`
`72`	`66`	`$exception = CouldNotSendNotification::serviceRespondedWithAnError($this->message, [`
`73`	`67`	`'status' => 500,`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,6 @@`
`3`	`3`	`namespace NotificationChannels\Interfax\Test;`
`4`	`4`
`5`	`5`	`use Illuminate\Notifications\Notification;`
`6`		`-use PHPUnit\Framework\Attributes\Test;`
`7`	`6`	`use Interfax\Client as InterfaxClient;`
`8`	`7`	`use Interfax\Resource as InterfaxResource;`
`9`	`8`	`use Mockery;`
`@@ -36,8 +35,7 @@ public function setUp(): void`
`36`	`35`	`$this->channel = new InterfaxChannel($this->interfax);`
`37`	`36`	`}`
`38`	`37`
`39`		`- #[Test]`
`40`		`- public function it_can_send_notification_with_a_single_file()`
	`38`	`+ public function test_send_notification_with_a_single_file()`
`41`	`39`	`{`
`42`	`40`	`$this->interfax->expects('deliver')`
`43`	`41`	`->once()`
`@@ -56,8 +54,7 @@ public function it_can_send_notification_with_a_single_file()`
`56`	`54`	`$this->channel->send(new TestNotifiable, new TestNotificationWithSingleFile);`
`57`	`55`	`}`
`58`	`56`
`59`		`- #[Test]`
`60`		`- public function it_can_send_notification_with_files()`
	`57`	`+ public function test_send_notification_with_files()`
`61`	`58`	`{`
`62`	`59`	`$this->interfax->expects('deliver')`
`63`	`60`	`->once()`
`@@ -82,8 +79,7 @@ public function it_can_send_notification_with_files()`
`82`	`79`	`$this->channel->send(new TestNotifiable, new TestNotificationWithFiles);`
`83`	`80`	`}`
`84`	`81`
`85`		`- #[Test]`
`86`		`- public function it_can_send_notification_pdf_as_stream()`
	`82`	`+ public function test_send_notification_pdf_as_stream()`
`87`	`83`	`{`
`88`	`84`	`$this->interfax->expects('deliver')`
`89`	`85`	`->with(Mockery::on(function ($output) {`
`@@ -111,8 +107,7 @@ public function it_can_send_notification_pdf_as_stream()`
`111`	`107`	`$this->channel->send(new TestNotifiable, new TestNotificationAsStreamPdf);`
`112`	`108`	`}`
`113`	`109`
`114`		`- #[Test]`
`115`		`- public function it_can_send_notification_html_as_stream()`
	`110`	`+ public function test_send_notification_html_as_stream()`
`116`	`111`	`{`
`117`	`112`	`$filename = 'test-file.html';`
`118`	`113`	`$this->addFile($filename);`
`@@ -145,14 +140,12 @@ public function it_can_send_notification_html_as_stream()`
`145`	`140`	`$this->channel->send(new TestNotifiable, new TestNotificationAsStreamHtml);`
`146`	`141`	`}`
`147`	`142`
`148`		`- #[Test]`
`149`		`- public function it_can_return_early_when_no_fax_number_provided()`
	`143`	`+ public function test_return_early_when_no_fax_number_provided()`
`150`	`144`	`{`
`151`	`145`	`$this->assertNull($this->channel->send(new TestNotifiableNotSendable, new TestNotificationWithFiles));`
`152`	`146`	`}`
`153`	`147`
`154`		`- #[Test]`
`155`		`- public function it_can_refresh_the_file_response()`
	`148`	`+ public function test_refresh_the_file_response()`
`156`	`149`	`{`
`157`	`150`	`$this->resource`
`158`	`151`	`->expects('refresh')`
`@@ -170,8 +163,7 @@ public function it_can_refresh_the_file_response()`
`170`	`163`	`$this->channel->send(new TestNotifiable, new TestNotificationWithRefresh);`
`171`	`164`	`}`
`172`	`165`
`173`		`- #[Test]`
`174`		`- public function it_can_throw_the_exception()`
	`166`	`+ public function test_throw_the_exception()`
`175`	`167`	`{`
`176`	`168`	`$this->expectException(CouldNotSendNotification::class);`
`177`	`169`
Original file line number	Diff line number	Diff line change
`@@ -2,13 +2,11 @@`
`2`	`2`
`3`	`3`	`namespace NotificationChannels\Interfax\Test;`
`4`	`4`
`5`		`-use PHPUnit\Framework\Attributes\Test;`
`6`	`5`	`use NotificationChannels\Interfax\InterfaxMessage;`
`7`	`6`
`8`	`7`	`class InterfaxMessageTest extends TestCase`
`9`	`8`	`{`
`10`		`- #[Test]`
`11`		`- public function it_should_check_the_status_via_refresh()`
	`9`	`+ public function test_check_the_status_via_refresh()`
`12`	`10`	`{`
`13`	`11`	`$message = (new InterfaxMessage)`
`14`	`12`	`->checkStatus()`
`@@ -18,8 +16,7 @@ public function it_should_check_the_status_via_refresh()`
`18`	`16`	`$this->assertTrue($message->shouldCheckStatus());`
`19`	`17`	`}`
`20`	`18`
`21`		`- #[Test]`
`22`		`- public function it_should_not_check_the_status_via_refresh_manual()`
	`19`	`+ public function test_not_check_the_status_via_refresh_manual()`
`23`	`20`	`{`
`24`	`21`	`$message = (new InterfaxMessage)`
`25`	`22`	`->checkStatus(false)`
`@@ -29,8 +26,7 @@ public function it_should_not_check_the_status_via_refresh_manual()`
`29`	`26`	`$this->assertFalse($message->shouldCheckStatus());`
`30`	`27`	`}`
`31`	`28`
`32`		`- #[Test]`
`33`		`- public function it_should_not_check_the_status_via_refresh_default()`
	`29`	`+ public function test_not_check_the_status_via_refresh_default()`
`34`	`30`	`{`
`35`	`31`	`$message = (new InterfaxMessage)`
`36`	`32`	`->user(new TestNotifiable)`
`@@ -39,8 +35,7 @@ public function it_should_not_check_the_status_via_refresh_default()`
`39`	`35`	`$this->assertFalse($message->shouldCheckStatus());`
`40`	`36`	`}`
`41`	`37`
`42`		`- #[Test]`
`43`		`- public function it_should_set_the_file_chunk_size_filename()`
	`38`	`+ public function test_set_the_file_chunk_size_filename()`
`44`	`39`	`{`
`45`	`40`	`$this->increaseChunkSize();`
`46`	`41`
`@@ -54,8 +49,7 @@ public function it_should_set_the_file_chunk_size_filename()`
`54`	`49`	`$this->assertSame($this->chunkSize, $this->getChunkSize($delivery));`
`55`	`50`	`}`
`56`	`51`
`57`		`- #[Test]`
`58`		`- public function it_should_set_the_file_chunk_size_file_array()`
	`52`	`+ public function test_set_the_file_chunk_size_file_array()`
`59`	`53`	`{`
`60`	`54`	`$this->increaseChunkSize();`
`61`	`55`
`@@ -69,8 +63,7 @@ public function it_should_set_the_file_chunk_size_file_array()`
`69`	`63`	`$this->assertSame($this->chunkSize, $this->getChunkSize($delivery));`
`70`	`64`	`}`
`71`	`65`
`72`		`- #[Test]`
`73`		`- public function it_should_set_the_file_chunk_size_file_object()`
	`66`	`+ public function test_set_the_file_chunk_size_file_object()`
`74`	`67`	`{`
`75`	`68`	`$this->increaseChunkSize();`
`76`	`69`	`$client = new \Interfax\Client;`