Add void return to tests

tests/_support/Commands/AppInfo.php

@@ -23,12 +23,12 @@ class AppInfo extends BaseCommand
     protected $arguments   = ['draft' => 'unused'];
     protected $description = 'Displays basic application information.';
 
-    public function run(array $params)
+    public function run(array $params): void
     {
         CLI::write('CI Version: ' . CLI::color(CodeIgniter::CI_VERSION, 'red'));
     }
 
-    public function bomb()
+    public function bomb(): void
     {
         try {
             CLI::color('test', 'white', 'Background');
@@ -37,7 +37,7 @@ public function bomb()
         }
     }
 
-    public function helpme()
+    public function helpme(): void
     {
         $this->call('help');
     }

tests/_support/Commands/InvalidCommand.php

@@ -27,7 +27,7 @@ public function __construct()
         throw new ReflectionException();
     }
 
-    public function run(array $params)
+    public function run(array $params): void
     {
         CLI::write('CI Version: ' . CLI::color(CodeIgniter::CI_VERSION, 'red'));
     }

tests/_support/Commands/LanguageCommand.php

@@ -27,7 +27,7 @@ class LanguageCommand extends BaseCommand
         '--sort' => 'Turn on/off the sortImports flag.',
     ];
 
-    public function run(array $params)
+    public function run(array $params): void
     {
         $this->setHasClassName(false);
         $params[0] = 'Foobar';

tests/_support/Commands/ParamsReveal.php

@@ -21,7 +21,7 @@ class ParamsReveal extends BaseCommand
     protected $description = 'Reveal params';
     public static $args;
 
-    public function run(array $params)
+    public function run(array $params): void
     {
         static::$args = $params;
     }

tests/_support/Commands/Unsuffixable.php

@@ -65,7 +65,7 @@ class Unsuffixable extends BaseCommand
     /**
      * Actually execute a command.
      */
-    public function run(array $params)
+    public function run(array $params): void
     {
         $this->component = 'Command';
         $this->directory = 'Commands';

tests/_support/Controllers/Popcorn.php

@@ -33,7 +33,7 @@ public function pop()
         return $this->respond('Oops', 567, 'Surprise');
     }
 
-    public function popper()
+    public function popper(): void
     {
         throw new RuntimeException('Surprise', 500);
     }
@@ -61,12 +61,12 @@ public function index3()
         return $this->response->setJSON(['lang' => $this->request->getLocale()]);
     }
 
-    public function canyon()
+    public function canyon(): void
     {
         echo 'Hello-o-o ' . $this->request->getGet('foo');
     }
 
-    public function cat()
+    public function cat(): void
     {
     }
 

tests/_support/Database/Migrations/20160428212500_Create_test_tables.php

@@ -15,7 +15,7 @@
 
 class Migration_Create_test_tables extends Migration
 {
-    public function up()
+    public function up(): void
     {
         // User Table
         $this->forge->addField([
@@ -160,7 +160,7 @@ public function up()
         }
     }
 
-    public function down()
+    public function down(): void
     {
         $this->forge->dropTable('user', true);
         $this->forge->dropTable('job', true);

tests/_support/Database/Seeds/AnotherSeeder.php

@@ -15,7 +15,7 @@
 
 class AnotherSeeder extends Seeder
 {
-    public function run()
+    public function run(): void
     {
         $row = [
             'name'    => 'Jerome Lohan',

tests/_support/Database/Seeds/CITestSeeder.php

@@ -15,7 +15,7 @@
 
 class CITestSeeder extends Seeder
 {
-    public function run()
+    public function run(): void
     {
         // Job Data
         $data = [

tests/_support/Filters/Customfilter.php

@@ -23,7 +23,7 @@ public function before(RequestInterface $request, $arguments = null)
         return $request;
     }
 
-    public function after(RequestInterface $request, ResponseInterface $response, $arguments = null)
+    public function after(RequestInterface $request, ResponseInterface $response, $arguments = null): void
     {
     }
 }

tests/_support/MigrationTestMigrations/Database/Migrations/2018-01-24-102301_Some_migration.php

@@ -13,7 +13,7 @@
 
 class Migration_some_migration extends \CodeIgniter\Database\Migration
 {
-    public function up()
+    public function up(): void
     {
         $this->forge->addField([
             'key' => [
@@ -28,7 +28,7 @@ public function up()
         ]);
     }
 
-    public function down()
+    public function down(): void
     {
         $this->forge->dropTable('foo', true);
     }

tests/_support/MigrationTestMigrations/Database/Migrations/2018-01-24-102302_Another_migration.php

@@ -13,7 +13,7 @@
 
 class Migration_another_migration extends \CodeIgniter\Database\Migration
 {
-    public function up()
+    public function up(): void
     {
         $fields = [
             'value' => [
@@ -29,7 +29,7 @@ public function up()
         ]);
     }
 
-    public function down()
+    public function down(): void
     {
         if ($this->db->tableExists('foo')) {
             $this->forge->dropColumn('foo', 'value');

tests/_support/Publishers/TestPublisher.php

@@ -39,7 +39,7 @@ final class TestPublisher extends Publisher
     /**
      * Fakes an error on the given file.
      */
-    public static function setResult(bool $result)
+    public static function setResult(bool $result): void
     {
         self::$result = $result;
     }

tests/_support/View/Cells/AdditionCell.php

@@ -17,7 +17,7 @@ class AdditionCell extends Cell
 {
     public int $value = 2;
 
-    public function mount(?int $number = null, bool $skipAddition = false)
+    public function mount(?int $number = null, bool $skipAddition = false): void
     {
         $this->value = ! $skipAddition
             ? $this->value + (int) $number

tests/_support/View/SampleClassWithInitController.php

@@ -25,7 +25,7 @@ class SampleClassWithInitController
 {
     private ResponseInterface $response;
 
-    public function initController(RequestInterface $request, ResponseInterface $response, LoggerInterface $logger)
+    public function initController(RequestInterface $request, ResponseInterface $response, LoggerInterface $logger): void
     {
         $this->response = $response;
     }