Fix code styling [ci skip]

Author: aarondfrancis

tests/Unit/Commands/ActivateTest.php

@@ -66,7 +66,8 @@ public function assertEvents($deployed = true, $activated = true)
             Event::assertNotDispatched(AfterFunctionsActivated::class);
         }
     }
-public function test_it_should_activate_functions()
+
+    public function test_it_should_activate_functions()
     {
         $this->mockActivating();
 
@@ -85,7 +86,8 @@ protected function mockInvokeAsync($with = [], $return = null)
     {
         return $this->mockMethod('invokeAsync', $with, $return);
     }
-public function test_it_should_pre_warm_functions_if_the_latest_version_is_different()
+
+    public function test_it_should_pre_warm_functions_if_the_latest_version_is_different()
     {
         // The latest version is not the active version.
         $this->lambda->shouldReceive('latestVersionHasAlias')
@@ -123,7 +125,8 @@ public function test_it_should_pre_warm_functions_if_the_latest_version_is_diffe
 
         $this->artisan('sidecar:activate --pre-warm');
     }
-public function test_it_should_not_pre_warm_functions_if_the_latest_version_is_the_same()
+
+    public function test_it_should_not_pre_warm_functions_if_the_latest_version_is_the_same()
     {
         $this->lambda->shouldReceive('latestVersionHasAlias')
             ->andReturn(true);
@@ -145,7 +148,8 @@ public function test_it_should_not_pre_warm_functions_if_the_latest_version_is_t
 
         $this->artisan('sidecar:activate --pre-warm');
     }
-public function test_it_should_activate_functions_with_env()
+
+    public function test_it_should_activate_functions_with_env()
     {
         $this->mockActivating();
 

tests/Unit/Commands/ConfigureTest.php

@@ -156,7 +156,8 @@ protected function createRolePayload()
             ]),
         ];
     }
-public function test_basic_happy_path()
+
+    public function test_basic_happy_path()
     {
         $this->mockS3(function ($mock) {
             $this->mockHeadBucketNotFound($mock);

tests/Unit/Commands/DeployTest.php

@@ -11,7 +11,7 @@
 
 class DeployTest extends DeploymentTest
 {
-public function test_it_deploys_the_functions_in_the_config()
+    public function test_it_deploys_the_functions_in_the_config()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(false);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -25,7 +25,8 @@ public function test_it_deploys_the_functions_in_the_config()
 
         $this->assertEvents($deployed = true, $activated = false);
     }
-public function test_it_deploys_and_activates_the_functions_in_the_config()
+
+    public function test_it_deploys_and_activates_the_functions_in_the_config()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(false);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -42,7 +43,8 @@ public function test_it_deploys_and_activates_the_functions_in_the_config()
 
         $this->assertEvents($deployed = true, $activated = true);
     }
-public function test_it_uses_a_fake_environment()
+
+    public function test_it_uses_a_fake_environment()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(false);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);

tests/Unit/DeploymentTest.php

@@ -110,7 +110,8 @@ public function assertEvents($deployed = true, $activated = true)
             Event::assertNotDispatched(AfterFunctionsActivated::class);
         }
     }
-public function test_it_deploys_a_function_that_doesnt_exist()
+
+    public function test_it_deploys_a_function_that_doesnt_exist()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(false);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -120,7 +121,8 @@ public function test_it_deploys_a_function_that_doesnt_exist()
 
         $this->assertEvents($deployed = true, $activated = false);
     }
-public function test_it_deploys_a_function_that_doesnt_exist_from_the_deployment_class()
+
+    public function test_it_deploys_a_function_that_doesnt_exist_from_the_deployment_class()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(false);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -130,7 +132,8 @@ public function test_it_deploys_a_function_that_doesnt_exist_from_the_deployment
 
         $this->assertEvents($deployed = true, $activated = false);
     }
-public function test_it_deploys_an_array_of_functions()
+
+    public function test_it_deploys_an_array_of_functions()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(false);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -140,7 +143,8 @@ public function test_it_deploys_an_array_of_functions()
 
         $this->assertEvents($deployed = true, $activated = false);
     }
-public function test_it_deploys_the_functions_in_the_config()
+
+    public function test_it_deploys_the_functions_in_the_config()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(false);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -154,7 +158,8 @@ public function test_it_deploys_the_functions_in_the_config()
 
         $this->assertEvents($deployed = true, $activated = false);
     }
-public function test_it_deploys_and_activates_a_function_that_doesnt_exist()
+
+    public function test_it_deploys_and_activates_a_function_that_doesnt_exist()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(false);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -165,7 +170,8 @@ public function test_it_deploys_and_activates_a_function_that_doesnt_exist()
 
         $this->assertEvents($deployed = true, $activated = true);
     }
-public function test_it_updates_an_existing_function()
+
+    public function test_it_updates_an_existing_function()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(true);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -177,7 +183,8 @@ public function test_it_updates_an_existing_function()
 
         $this->assertEvents($deployed = true, $activated = false);
     }
-public function test_it_updates_and_activates_an_existing_function()
+
+    public function test_it_updates_and_activates_an_existing_function()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(true);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -190,7 +197,8 @@ public function test_it_updates_and_activates_an_existing_function()
 
         $this->assertEvents($deployed = true, $activated = true);
     }
-public function test_it_sets_environment_variables()
+
+    public function test_it_sets_environment_variables()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(true);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -229,7 +237,8 @@ public function test_it_sets_environment_variables()
 
         $this->assertEvents($deployed = true, $activated = true);
     }
-public function test_it_doesnt_change_variables_that_havent_changed()
+
+    public function test_it_doesnt_change_variables_that_havent_changed()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(true);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -253,7 +262,8 @@ public function test_it_doesnt_change_variables_that_havent_changed()
 
         $this->assertEvents($deployed = true, $activated = true);
     }
-public function test_it_sets_function_tags()
+
+    public function test_it_sets_function_tags()
     {
         $this->lambda->shouldReceive('functionExists')->andReturn(false);
         $this->lambda->shouldReceive('getVersions')->andReturn([]);
@@ -304,7 +314,8 @@ public function test_it_sets_function_tags()
 
         $this->assertEvents($deployed = true, $activated = true);
     }
-public function test_it_throws_an_exception_if_there_are_no_functions()
+
+    public function test_it_throws_an_exception_if_there_are_no_functions()
     {
         config()->set('sidecar.functions', [
 

tests/Unit/EnvironmentMismatchTest.php

@@ -40,7 +40,8 @@ protected function mockInvokeNonExistent()
             ->once()
             ->andThrow($this->notFoundException());
     }
-public function test_it_throws_the_right_exception()
+
+    public function test_it_throws_the_right_exception()
     {
         $this->mockInvokeNonExistent();
 

tests/Unit/EnvironmentTest.php

@@ -10,7 +10,7 @@
 
 class EnvironmentTest extends Base
 {
-public function test_it_can_be_overridden()
+    public function test_it_can_be_overridden()
     {
         $this->assertEquals('testing', Sidecar::getEnvironment());
 
@@ -22,7 +22,8 @@ public function test_it_can_be_overridden()
 
         $this->assertEquals('testing', Sidecar::getEnvironment());
     }
-public function test_precedence_is_correct()
+
+    public function test_precedence_is_correct()
     {
         Sidecar::clearEnvironment();
 

tests/Unit/ExecuteMultipleTest.php

@@ -77,7 +77,8 @@ protected function mockMultiple()
             ]))
             ->andReturn($result);
     }
-public function test_execute_many_by_function()
+
+    public function test_execute_many_by_function()
     {
         $this->mockMultiple();
 
@@ -91,7 +92,8 @@ public function test_execute_many_by_function()
 
         $this->assertEvents(3);
     }
-public function test_execute_many_by_facade()
+
+    public function test_execute_many_by_facade()
     {
         $this->mockMultiple();
 
@@ -105,7 +107,8 @@ public function test_execute_many_by_facade()
 
         $this->assertEvents(3);
     }
-public function test_execute_many_by_facade_with_instantiated_class()
+
+    public function test_execute_many_by_facade_with_instantiated_class()
     {
         $this->mockMultiple();
 
@@ -119,7 +122,8 @@ public function test_execute_many_by_facade_with_instantiated_class()
 
         $this->assertEvents(3);
     }
-public function test_execute_many_by_function_int()
+
+    public function test_execute_many_by_function_int()
     {
         $result = Mockery::mock(PromiseInterface::class)
             ->shouldReceive('wait')
@@ -135,7 +139,8 @@ public function test_execute_many_by_function_int()
 
         $this->assertEvents(5);
     }
-public function test_execute_many_by_facade_int()
+
+    public function test_execute_many_by_facade_int()
     {
         $result = Mockery::mock(PromiseInterface::class)
             ->shouldReceive('wait')
@@ -151,7 +156,8 @@ public function test_execute_many_by_facade_int()
 
         $this->assertEvents(5);
     }
-public function test_execute_many_by_facade_int_with_instantiated_class()
+
+    public function test_execute_many_by_facade_int_with_instantiated_class()
     {
         $result = Mockery::mock(PromiseInterface::class)
             ->shouldReceive('wait')
@@ -167,7 +173,8 @@ public function test_execute_many_by_facade_int_with_instantiated_class()
 
         $this->assertEvents(5);
     }
-public function test_execute_many_async_by_function()
+
+    public function test_execute_many_async_by_function()
     {
         $result = Mockery::mock(PromiseInterface::class)
             ->shouldNotReceive('wait')