Skip to content

Commit 302c5db

Browse files
committed
Refactor SpeedTrapListener not to use BaseTestListener that doesn't exist anymore. Also, implement newer listener interface
1 parent ee2c276 commit 302c5db

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

src/SpeedTrapListener.php

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,18 @@
33

44
namespace JohnKary\PHPUnit\Listener;
55

6-
use PHPUnit\Framework\{AssertionFailedError, TestSuite, Test, TestCase, BaseTestListener, Warning};
6+
use PHPUnit\Framework\{
7+
TestListener, TestListenerDefaultImplementation, TestSuite, Test, TestCase
8+
};
79

810
/**
911
* A PHPUnit TestListener that exposes your slowest running tests by outputting
1012
* results directly to the console.
1113
*/
12-
class SpeedTrapListener extends BaseTestListener
14+
class SpeedTrapListener implements TestListener
1315
{
16+
use TestListenerDefaultImplementation;
17+
1418
/**
1519
* Internal tracking for test suites.
1620
*
@@ -52,7 +56,7 @@ public function __construct(array $options = [])
5256
* @param Test $test
5357
* @param float $time
5458
*/
55-
public function endTest(Test $test, $time)
59+
public function endTest(Test $test, float $time): void
5660
{
5761
if (!$test instanceof TestCase) return;
5862

@@ -69,7 +73,7 @@ public function endTest(Test $test, $time)
6973
*
7074
* @param TestSuite $suite
7175
*/
72-
public function startTestSuite(TestSuite $suite)
76+
public function startTestSuite(TestSuite $suite): void
7377
{
7478
$this->suites++;
7579
}
@@ -79,7 +83,7 @@ public function startTestSuite(TestSuite $suite)
7983
*
8084
* @param TestSuite $suite
8185
*/
82-
public function endTestSuite(TestSuite $suite)
86+
public function endTestSuite(TestSuite $suite): void
8387
{
8488
$this->suites--;
8589

0 commit comments

Comments
 (0)