Merge pull request #185 from rollbar/fixed/phpunit-tests-failing

Fixed failing PHPUnit tests

Author: danielmorell

.gitignore

@@ -3,4 +3,4 @@ composer.lock
 .DS_Store
 .idea
 .vscode
-.phpunit.result.cache
+.phpunit.cache

phpunit.xml

@@ -1,16 +1,24 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
          bootstrap="tests/bootstrap.php"
-         colors="true"
-         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.0/phpunit.xsd">
-    <coverage>
+         cacheDirectory=".phpunit.cache"
+         executionOrder="depends,defects"
+         requireCoverageMetadata="true"
+         beStrictAboutCoverageMetadata="true"
+         beStrictAboutOutputDuringTests="true"
+         displayDetailsOnPhpunitDeprecations="true"
+         failOnPhpunitDeprecation="true"
+         failOnRisky="true"
+         failOnWarning="true">
+    <source restrictNotices="true" restrictWarnings="true">
         <include>
             <directory suffix=".php">./src</directory>
         </include>
-    </coverage>
+    </source>
     <testsuites>
         <testsuite name="Test Suite">
-            <directory suffix=".php">./tests/</directory>
+            <directory suffix="Test.php">./tests/</directory>
         </testsuite>
     </testsuites>
 </phpunit>

tests/TestCase.php tests/AbstractTestCase.phptests/TestCase.php renamed to tests/AbstractTestCase.php

@@ -6,7 +6,7 @@
 use Mockery;
 use Rollbar\Laravel\RollbarServiceProvider;
 
-abstract class TestCase extends \Orchestra\Testbench\TestCase
+abstract class AbstractTestCase extends \Orchestra\Testbench\TestCase
 {
     protected string $access_token = 'B42nHP04s06ov18Dv8X7VI4nVUs6w04X';
 

tests/RollbarTest.php

@@ -2,11 +2,19 @@
 
 namespace Rollbar\Laravel\Tests;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\UsesClass;
 use Rollbar\Laravel\MonologHandler;
 use Rollbar\Laravel\AgentHandler;
+use Rollbar\Laravel\RollbarServiceProvider;
+use Rollbar\Laravel\TelemetryListener;
 use Rollbar\RollbarLogger;
 
-class RollbarTest extends TestCase
+#[UsesClass(AgentHandler::class)]
+#[UsesClass(MonologHandler::class)]
+#[UsesClass(TelemetryListener::class)]
+#[CoversClass(RollbarServiceProvider::class)]
+class RollbarTest extends AbstractTestCase
 {
     public function testBinding()
     {

tests/TelemetryListenerTest.php

@@ -8,10 +8,16 @@
 use Illuminate\Log\Events\MessageLogged;
 use Illuminate\Routing\Events\RouteMatched;
 use Illuminate\Routing\Route;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\UsesClass;
+use Rollbar\Laravel\RollbarServiceProvider;
+use Rollbar\Laravel\TelemetryListener;
 use Rollbar\Rollbar;
 use Rollbar\Telemetry\EventLevel;
 
-class TelemetryListenerTest extends TestCase
+#[UsesClass(RollbarServiceProvider::class)]
+#[CoversClass(TelemetryListener::class)]
+class TelemetryListenerTest extends AbstractTestCase
 {
     protected function setUp(): void
     {