Merge pull request #203

Added void typo

Author: andrey-helldar

tests/Commands/EventsTest.php

@@ -15,7 +15,7 @@
 
 class EventsTest extends TestCase
 {
-    public function testSuccess()
+    public function testSuccess(): void
     {
         $this->copyFiles();
 
@@ -29,7 +29,7 @@ public function testSuccess()
         Event::assertNotDispatched(DeployOperationFailed::class);
     }
 
-    public function testFailed()
+    public function testFailed(): void
     {
         $this->expectException(Exception::class);
         $this->expectExceptionMessage('Custom exception');

tests/Commands/MakeTest.php

@@ -10,7 +10,7 @@
 
 class MakeTest extends TestCase
 {
-    public function testMakingFiles()
+    public function testMakingFiles(): void
     {
         $name = 'MakeExample';
 
@@ -28,7 +28,7 @@ public function testMakingFiles()
         );
     }
 
-    public function testAskedName()
+    public function testAskedName(): void
     {
         $path = $this->getOperationsPath() . '/' . date('Y_m_d_His') . '_some_name.php';
 
@@ -41,7 +41,7 @@ public function testAskedName()
         $this->assertFileExists($path);
     }
 
-    public function testAutoName()
+    public function testAutoName(): void
     {
         $path = $this->getOperationsPath() . '/' . date('Y_m_d_His') . '_auto.php';
 
@@ -54,7 +54,7 @@ public function testAutoName()
         $this->assertFileExists($path);
     }
 
-    public function testNestedRightSlashWithoutExtension()
+    public function testNestedRightSlashWithoutExtension(): void
     {
         $name = 'Foo/bar/QweRty';
 
@@ -72,7 +72,7 @@ public function testNestedRightSlashWithoutExtension()
         );
     }
 
-    public function testNestedRightSlashWithExtension()
+    public function testNestedRightSlashWithExtension(): void
     {
         $name = 'Foo/bar/QweRty.php';
 
@@ -90,7 +90,7 @@ public function testNestedRightSlashWithExtension()
         );
     }
 
-    public function testNestedLeftSlashWithoutExtension()
+    public function testNestedLeftSlashWithoutExtension(): void
     {
         $name = 'Foo\\bar\\QweRty';
 
@@ -108,7 +108,7 @@ public function testNestedLeftSlashWithoutExtension()
         );
     }
 
-    public function testNestedLeftSlashWithExtension()
+    public function testNestedLeftSlashWithExtension(): void
     {
         $name = 'Foo\\bar\\QweRty.php';
 
@@ -126,7 +126,7 @@ public function testNestedLeftSlashWithExtension()
         );
     }
 
-    public function testFromCustomizedStub()
+    public function testFromCustomizedStub(): void
     {
         $name = 'MakeExample';
 

tests/Commands/MutexTest.php

@@ -39,7 +39,7 @@ public function handle(): int
         $this->command->setLaravel($container);
     }
 
-    public function testCanRunIsolatedCommandIfNotBlocked()
+    public function testCanRunIsolatedCommandIfNotBlocked(): void
     {
         $this->mutex->shouldReceive('create')
             ->andReturn(true)
@@ -53,7 +53,7 @@ public function testCanRunIsolatedCommandIfNotBlocked()
         $this->assertSame(1, $this->command->ran);
     }
 
-    public function testCannotRunIsolatedCommandIfBlocked()
+    public function testCannotRunIsolatedCommandIfBlocked(): void
     {
         $this->mutex->shouldReceive('create')
             ->andReturn(false)
@@ -67,7 +67,7 @@ public function testCannotRunIsolatedCommandIfBlocked()
         $this->assertSame(0, $this->command->ran);
     }
 
-    public function testCanRunCommandAgainAfterOtherCommandFinished()
+    public function testCanRunCommandAgainAfterOtherCommandFinished(): void
     {
         $this->mutex->shouldReceive('create')
             ->andReturn(true)
@@ -82,7 +82,7 @@ public function testCanRunCommandAgainAfterOtherCommandFinished()
         $this->assertEquals(2, $this->command->ran);
     }
 
-    public function testCanRunCommandAgainNonAutomated()
+    public function testCanRunCommandAgainNonAutomated(): void
     {
         $this->mutex->shouldNotHaveBeenCalled();
 

tests/Commands/OperationsTest.php

@@ -16,7 +16,7 @@
 
 class OperationsTest extends TestCase
 {
-    public function testSuccess()
+    public function testSuccess(): void
     {
         $this->assertDatabaseDoesntTable($this->table);
 
@@ -32,7 +32,7 @@ public function testSuccess()
         $this->assertDatabaseOperationHas($this->table, 'test_migration');
     }
 
-    public function testSameName()
+    public function testSameName(): void
     {
         $this->assertDatabaseDoesntTable($this->table);
 
@@ -52,7 +52,7 @@ public function testSameName()
         $this->assertDatabaseOperationHas($this->table, 'test_migration');
     }
 
-    public function testOnce()
+    public function testOnce(): void
     {
         $this->copyFiles();
 
@@ -85,7 +85,7 @@ public function testOnce()
         $this->assertDatabaseOperationDoesntLike($this->table, $table);
     }
 
-    public function testSuccessTransaction()
+    public function testSuccessTransaction(): void
     {
         $this->copySuccessTransaction();
 
@@ -103,7 +103,7 @@ public function testSuccessTransaction()
         $this->assertDatabaseOperationHas($this->table, $table);
     }
 
-    public function testFailedTransaction()
+    public function testFailedTransaction(): void
     {
         $this->copyFailedTransaction();
 
@@ -128,7 +128,7 @@ public function testFailedTransaction()
         $this->assertDatabaseOperationDoesntLike($this->table, $table);
     }
 
-    public function testSingleEnvironment()
+    public function testSingleEnvironment(): void
     {
         $this->copyFiles();
 
@@ -163,7 +163,7 @@ public function testSingleEnvironment()
         $this->assertDatabaseOperationDoesntLike($this->table, 'run_except_testing');
     }
 
-    public function testManyEnvironments()
+    public function testManyEnvironments(): void
     {
         $this->copyFiles();
 
@@ -204,7 +204,7 @@ public function testManyEnvironments()
         $this->assertDatabaseOperationDoesntLike($this->table, 'run_except_many_environments');
     }
 
-    public function testAllow()
+    public function testAllow(): void
     {
         $this->copyFiles();
 
@@ -230,7 +230,7 @@ public function testAllow()
         $this->assertDatabaseOperationDoesntLike($this->table, 'run_disallow');
     }
 
-    public function testUpSuccess()
+    public function testUpSuccess(): void
     {
         $this->copyFiles();
 
@@ -248,7 +248,7 @@ public function testUpSuccess()
         $this->assertDatabaseOperationHas($this->table, 'run_success');
     }
 
-    public function testUpSuccessOnFailed()
+    public function testUpSuccessOnFailed(): void
     {
         $this->copyFiles();
 
@@ -283,7 +283,7 @@ public function testUpSuccessOnFailed()
         $this->assertDatabaseOperationDoesntLike($this->table, 'run_success_on_failed');
     }
 
-    public function testUpFailed()
+    public function testUpFailed(): void
     {
         $this->copyFiles();
 
@@ -301,7 +301,7 @@ public function testUpFailed()
         $this->assertDatabaseOperationHas($this->table, 'run_failed');
     }
 
-    public function testUpFailedOnException()
+    public function testUpFailedOnException(): void
     {
         $this->copyFiles();
 
@@ -336,7 +336,7 @@ public function testUpFailedOnException()
         $this->assertDatabaseOperationDoesntLike($this->table, 'run_failed_failure');
     }
 
-    public function testPathAsFileWithExtension()
+    public function testPathAsFileWithExtension(): void
     {
         $this->copyFiles();
 
@@ -358,7 +358,7 @@ public function testPathAsFileWithExtension()
         $this->assertSame('sub_path/2021_12_15_205804_baz', $this->table()->first()->operation);
     }
 
-    public function testPathAsFileWithoutExtension()
+    public function testPathAsFileWithoutExtension(): void
     {
         $this->copyFiles();
 
@@ -380,7 +380,7 @@ public function testPathAsFileWithoutExtension()
         $this->assertSame($path, $this->table()->first()->operation);
     }
 
-    public function testPathAsDirectory()
+    public function testPathAsDirectory(): void
     {
         $this->copyFiles();
 
@@ -402,7 +402,7 @@ public function testPathAsDirectory()
         $this->assertSame('sub_path/2021_12_15_205804_baz', $this->table()->first()->operation);
     }
 
-    public function testOperationsNotFound()
+    public function testOperationsNotFound(): void
     {
         $this->assertDatabaseDoesntTable($this->table);
 
@@ -413,7 +413,7 @@ public function testOperationsNotFound()
         $this->artisan(Names::Status)->assertExitCode(0);
     }
 
-    public function testDisabledBefore()
+    public function testDisabledBefore(): void
     {
         $this->copyFiles();
 
@@ -439,7 +439,7 @@ public function testDisabledBefore()
         $this->assertDatabaseOperationHas($this->table, 'test_before_disabled');
     }
 
-    public function testEnabledBefore()
+    public function testEnabledBefore(): void
     {
         $this->copyFiles();
 
@@ -465,7 +465,7 @@ public function testEnabledBefore()
         $this->assertDatabaseOperationDoesntLike($this->table, 'test_before_disabled');
     }
 
-    public function testMixedBefore()
+    public function testMixedBefore(): void
     {
         $this->copyFiles();
 
@@ -563,7 +563,7 @@ public function testSorting(): void
         $this->assertSame($files, $records);
     }
 
-    public function testDirectoryExclusion()
+    public function testDirectoryExclusion(): void
     {
         $this->copyFiles();
 
@@ -613,7 +613,7 @@ public function testDirectoryExclusion()
         $this->assertDatabaseOperationDoesntLike($this->table, 'sub_path/2022_10_27_230732_foo');
     }
 
-    public function testFileExclusion()
+    public function testFileExclusion(): void
     {
         $this->copyFiles();
 
@@ -658,7 +658,7 @@ public function testFileExclusion()
         $this->assertDatabaseOperationHas($this->table, 'sub_path/2022_10_27_230732_foo');
     }
 
-    public function testEmptyDirectory()
+    public function testEmptyDirectory(): void
     {
         $this->copyEmptyDirectory();
 
@@ -675,7 +675,7 @@ public function testEmptyDirectory()
         $this->assertDatabaseCount($this->table, 0);
     }
 
-    public function testAsync()
+    public function testAsync(): void
     {
         $this->copyAsync();
 
@@ -711,7 +711,7 @@ public function testAsync()
         ]));
     }
 
-    public function testSync()
+    public function testSync(): void
     {
         $this->copyAsync();
 
@@ -748,7 +748,7 @@ public function testSync()
         $this->assertDatabaseOperationDoesntLike($this->table, 'every_time');
     }
 
-    public function testViaMigrationMethod()
+    public function testViaMigrationMethod(): void
     {
         $this->copyViaMigrations();
 

tests/Commands/RefreshTest.php

@@ -9,7 +9,7 @@
 
 class RefreshTest extends TestCase
 {
-    public function testRefreshCommand()
+    public function testRefreshCommand(): void
     {
         $this->assertDatabaseDoesntTable($this->table);
 

tests/Commands/ResetTest.php

@@ -9,7 +9,7 @@
 
 class ResetTest extends TestCase
 {
-    public function testResetCommand()
+    public function testResetCommand(): void
     {
         $this->assertDatabaseDoesntTable($this->table);