Merge pull request #645 from pierresh/return-type-in-test

test: add missing return types in test

Author: n0nag0n

tests/AutoloadTest.php

@@ -19,7 +19,7 @@ protected function setUp(): void
     }
 
     // Autoload a class
-    public function testAutoload()
+    public function testAutoload(): void
     {
         $this->app->register('user', User::class);
 
@@ -33,7 +33,7 @@ public function testAutoload()
     }
 
     // Check autoload failure
-    public function testMissingClass()
+    public function testMissingClass(): void
     {
         $test = null;
         $this->app->register('test', 'NonExistentClass');

tests/CollectionTest.php

@@ -17,39 +17,39 @@ protected function setUp(): void
     }
 
     // Get an item
-    public function testGet()
+    public function testGet(): void
     {
         $this->assertEquals(1, $this->collection->a);
     }
 
     // Set an item
-    public function testSet()
+    public function testSet(): void
     {
         $this->collection->c = 3;
         $this->assertEquals(3, $this->collection->c);
     }
 
     // Check if an item exists
-    public function testExists()
+    public function testExists(): void
     {
         $this->assertTrue(isset($this->collection->a));
     }
 
     // Unset an item
-    public function testUnset()
+    public function testUnset(): void
     {
         unset($this->collection->a);
         $this->assertFalse(isset($this->collection->a));
     }
 
     // Count items
-    public function testCount()
+    public function testCount(): void
     {
         $this->assertEquals(2, count($this->collection));
     }
 
     // Iterate through items
-    public function testIterate()
+    public function testIterate(): void
     {
         $items = [];
         foreach ($this->collection as $key => $value) {
@@ -59,39 +59,39 @@ public function testIterate()
         $this->assertEquals(['a' => 1, 'b' => 2], $items);
     }
 
-    public function testJsonSerialize()
+    public function testJsonSerialize(): void
     {
         $this->assertEquals(['a' => 1, 'b' => 2], $this->collection->jsonSerialize());
     }
 
-    public function testOffsetSetWithNullOffset()
+    public function testOffsetSetWithNullOffset(): void
     {
         $this->collection->offsetSet(null, 3);
         $this->assertEquals(3, $this->collection->offsetGet(0));
     }
 
-    public function testOffsetExists()
+    public function testOffsetExists(): void
     {
         $this->collection->a = 1;
         $this->assertTrue($this->collection->offsetExists('a'));
     }
 
-    public function testOffsetUnset()
+    public function testOffsetUnset(): void
     {
         $this->collection->a = 1;
         $this->assertTrue($this->collection->offsetExists('a'));
         $this->collection->offsetUnset('a');
         $this->assertFalse($this->collection->offsetExists('a'));
     }
 
-    public function testKeys()
+    public function testKeys(): void
     {
         $this->collection->a = 1;
         $this->collection->b = 2;
         $this->assertEquals(['a', 'b'], $this->collection->keys());
     }
 
-    public function testClear()
+    public function testClear(): void
     {
         $this->collection->a = 1;
         $this->collection->b = 2;

tests/DispatcherTest.php

@@ -331,7 +331,7 @@ public function testExecuteStringClassDefaultContainerButForgotInjectingEngine()
         $result = $this->dispatcher->execute([ContainerDefault::class, 'testTheContainer']);
     }
 
-    public function testContainerDicePdoWrapperTestBadParams()
+    public function testContainerDicePdoWrapperTestBadParams(): void
     {
         $dice = new \Dice\Dice();
         $this->dispatcher->setContainerHandler(function ($class, $params) use ($dice) {

tests/DocExamplesTest.php

@@ -27,7 +27,7 @@ protected function tearDown(): void
         Flight::clear();
     }
 
-    public function testMapNotFoundMethod()
+    public function testMapNotFoundMethod(): void
     {
         Flight::map('notFound', function () {
             Flight::json([], 404);
@@ -45,7 +45,7 @@ public function testMapNotFoundMethod()
         $this->assertEquals('[]', Flight::response()->getBody());
     }
 
-    public function testMapNotFoundMethodV2OutputBuffering()
+    public function testMapNotFoundMethodV2OutputBuffering(): void
     {
         Flight::map('notFound', function () {
             Flight::json([], 404);
@@ -64,7 +64,7 @@ public function testMapNotFoundMethodV2OutputBuffering()
         $this->assertEquals('[]', Flight::response()->getBody());
     }
 
-    public function testMapErrorMethod()
+    public function testMapErrorMethod(): void
     {
         Flight::map('error', function (Throwable $error) {
             // Handle error
@@ -75,7 +75,7 @@ public function testMapErrorMethod()
         $this->expectOutputString('Custom: Error');
     }
 
-    public function testGetRouterStatically()
+    public function testGetRouterStatically(): void
     {
         $router = Flight::router();
         Flight::request()->method = 'GET';