maciejslawik / freshmail

tests/Subscriber/SubscriberAddHandlerTest.php

Spacing +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
     public function getSubscriberAddHandlerMock($sendRequestReturnValue)
     {
         $subscriberAddHandler = $this->getMockBuilder('\MSlwk\FreshMail\Handler\Subscriber\SubscriberAddHandler')
-            ->setConstructorArgs([new ErrorHandler(), '', ''])
-            ->setMethods(['sendRequest'])
+            ->setConstructorArgs([ new ErrorHandler(), '', '' ])
+            ->setMethods([ 'sendRequest' ])
             ->getMock();
 
         $subscriberAddHandler->expects($this->once())

tests/Subscriber/SubscriberEditHandlerTest.php

Spacing +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
     public function getSubscriberEditHandlerMock($sendRequestReturnValue)
     {
         $subscriberEditHandler = $this->getMockBuilder('\MSlwk\FreshMail\Handler\Subscriber\SubscriberEditHandler')
-            ->setConstructorArgs([new ErrorHandler(), '', ''])
-            ->setMethods(['sendRequest'])
+            ->setConstructorArgs([ new ErrorHandler(), '', '' ])
+            ->setMethods([ 'sendRequest' ])
             ->getMock();
 
         $subscriberEditHandler->expects($this->once())

tests/Base/AuthorizationTest.php

Spacing +2 added lines, -2 removed lines

@@ -27,8 +27,8 @@
     private function getPingHandlerMock($sendRequestReturnValue)
     {
         $pingHandler = $this->getMockBuilder('\MSlwk\FreshMail\Handler\Ping\PingHandler')
-            ->setConstructorArgs([new ErrorHandler(), '', ''])
-            ->setMethods(['sendRequest'])
+            ->setConstructorArgs([ new ErrorHandler(), '', '' ])
+            ->setMethods([ 'sendRequest' ])
             ->getMock();
 
         $pingHandler->expects($this->once())

tests/Base/AbstractHandlerTest.php

Spacing +7 added lines, -7 removed lines

@@ -41,8 +41,8 @@ 
     public function providerTestGenerateGetParametersString()
     {
         return [
-            [['user', '123'], '/user/123'],
-            [[], '']
+            [ [ 'user', '123' ], '/user/123' ],
+            [ [ ], '' ]
         ];
     }
 
@@ -52,8 +52,8 @@ 
     public function providerTestGeneratePostParametersString()
     {
         return [
-            [['hash' => '124e12', 'email' => 'test@test.com'], '{"hash":"124e12","email":"test@test.com"}'],
-            [[], '']
+            [ [ 'hash' => '124e12', 'email' => 'test@test.com' ], '{"hash":"124e12","email":"test@test.com"}' ],
+            [ [ ], '' ]
         ];
     }
 
@@ -65,7 +65,7 @@ 
         $method = $reflection->getMethod('getAuthorizationHash');
         $method->setAccessible(true);
 
-        $generatedHash = $method->invokeArgs($handler, ['/rest/ping', '']);
+        $generatedHash = $method->invokeArgs($handler, [ '/rest/ping', '' ]);
         self::assertEquals('fabe17d9e6096a32b6c57cfcef8af070e007e774', $generatedHash);
     }
 
@@ -83,7 +83,7 @@ 
         $method = $reflection->getMethod('generateGetParamsString');
         $method->setAccessible(true);
 
-        $generatedResultString = $method->invokeArgs($handler, [$parametersArray]);
+        $generatedResultString = $method->invokeArgs($handler, [ $parametersArray ]);
         self::assertEquals($expectedResultString, $generatedResultString);
     }
 
@@ -101,7 +101,7 @@ 
         $method = $reflection->getMethod('generatePostParamsString');
         $method->setAccessible(true);
 
-        $generatedResultString = $method->invokeArgs($handler, [$parametersArray]);
+        $generatedResultString = $method->invokeArgs($handler, [ $parametersArray ]);
         self::assertEquals($expectedResultString, $generatedResultString);
     }
 }

tests/Campaign/CampaignCreateHandlerTest.php

Spacing +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
     public function getCampaignCreateHandlerMock($sendRequestReturnValue)
     {
         $campaignCreateHandler = $this->getMockBuilder('\MSlwk\FreshMail\Handler\Campaign\CampaignCreateHandler')
-            ->setConstructorArgs([new ErrorHandler(), '', ''])
-            ->setMethods(['sendRequest'])
+            ->setConstructorArgs([ new ErrorHandler(), '', '' ])
+            ->setMethods([ 'sendRequest' ])
             ->getMock();
 
         $campaignCreateHandler->expects($this->once())

tests/Campaign/CampaignDeleteHandlerTest.php

Spacing +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
     public function getCampaignDeleteHandlerMock($sendRequestReturnValue)
     {
         $campaignDeleteHandler = $this->getMockBuilder('\MSlwk\FreshMail\Handler\Campaign\CampaignDeleteHandler')
-            ->setConstructorArgs([new ErrorHandler(), '', ''])
-            ->setMethods(['sendRequest'])
+            ->setConstructorArgs([ new ErrorHandler(), '', '' ])
+            ->setMethods([ 'sendRequest' ])
             ->getMock();
 
         $campaignDeleteHandler->expects($this->once())

tests/Campaign/CampaignEditHandlerTest.php

Spacing +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
     public function getCampaignDeleteHandlerMock($sendRequestReturnValue)
     {
         $campaignEditHandler = $this->getMockBuilder('\MSlwk\FreshMail\Handler\Campaign\CampaignEditHandler')
-            ->setConstructorArgs([new ErrorHandler(), '', ''])
-            ->setMethods(['sendRequest'])
+            ->setConstructorArgs([ new ErrorHandler(), '', '' ])
+            ->setMethods([ 'sendRequest' ])
             ->getMock();
 
         $campaignEditHandler->expects($this->once())

tests/Campaign/CampaignSendHandlerTest.php

Spacing +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
     public function getCampaignSendHandlerMock($sendRequestReturnValue)
     {
         $campaignSendHandler = $this->getMockBuilder('\MSlwk\FreshMail\Handler\Campaign\CampaignSendHandler')
-            ->setConstructorArgs([new ErrorHandler(), '', ''])
-            ->setMethods(['sendRequest'])
+            ->setConstructorArgs([ new ErrorHandler(), '', '' ])
+            ->setMethods([ 'sendRequest' ])
             ->getMock();
 
         $campaignSendHandler->expects($this->once())

tests/Campaign/CampaignTestHandlerTest.php

Spacing +9 added lines, -9 removed lines

@@ -30,8 +30,8 @@ 
     public function getCampaignTestHandlerMock($sendRequestReturnValue)
     {
         $campaignTestHandler = $this->getMockBuilder('\MSlwk\FreshMail\Handler\Campaign\CampaignTestHandler')
-            ->setConstructorArgs([new ErrorHandler(), '', ''])
-            ->setMethods(['sendRequest'])
+            ->setConstructorArgs([ new ErrorHandler(), '', '' ])
+            ->setMethods([ 'sendRequest' ])
             ->getMock();
 
         $campaignTestHandler->expects($this->once())
@@ -44,7 +44,7 @@ 
     public function testCampaignTestSuccessfullySent()
     {
         $campaignTestHandler = $this->getCampaignTestHandlerMock('{"status":"OK"}');
-        $returnValue = $campaignTestHandler->testCampaign('id_hash', ['test@test.com']);
+        $returnValue = $campaignTestHandler->testCampaign('id_hash', [ 'test@test.com' ]);
         self::assertNull($returnValue);
     }
 
@@ -62,7 +62,7 @@ 
             '{"errors":[{ "message":"Incorrect campaign hash", "code":"1721" }], "status":"errors"}'
         );
         $this->expectException(FreshMailCampaignException::class);
-        $campaignTestHandler->testCampaign('id_hash', ['test@test.com']);
+        $campaignTestHandler->testCampaign('id_hash', [ 'test@test.com' ]);
     }
 
     public function testEmptyEmail()
@@ -71,7 +71,7 @@ 
             '{"errors":[{ "message":"Missing email address", "code":"1722" }], "status":"errors"}'
         );
         $this->expectException(FreshMailCampaignException::class);
-        $campaignTestHandler->testCampaign('id_hash', ['test@test.com']);
+        $campaignTestHandler->testCampaign('id_hash', [ 'test@test.com' ]);
     }
 
     public function testAtLeastOneEmailInvalid()
@@ -80,7 +80,7 @@ 
             '{"errors":[{ "message":"At least one email is invalid", "code":"1723" }], "status":"errors"}'
         );
         $this->expectException(FreshMailCampaignException::class);
-        $campaignTestHandler->testCampaign('id_hash', ['test@test.com']);
+        $campaignTestHandler->testCampaign('id_hash', [ 'test@test.com' ]);
     }
 
     public function testCampaignDoesntExist()
@@ -89,7 +89,7 @@ 
             '{"errors":[{ "message":"Requested campaign doesnt exist", "code":"1724" }], "status":"errors"}'
         );
         $this->expectException(FreshMailCampaignException::class);
-        $campaignTestHandler->testCampaign('id_hash', ['test@test.com']);
+        $campaignTestHandler->testCampaign('id_hash', [ 'test@test.com' ]);
     }
 
     public function testCampaignNotReadyToSend()
@@ -98,7 +98,7 @@ 
             '{"errors":[{ "message":"Campaign not ready to send", "code":"1725" }], "status":"errors"}'
         );
         $this->expectException(FreshMailCampaignException::class);
-        $campaignTestHandler->testCampaign('id_hash', ['test@test.com']);
+        $campaignTestHandler->testCampaign('id_hash', [ 'test@test.com' ]);
     }
 
     public function testSendingFailed()
@@ -107,6 +107,6 @@ 
             '{"errors":[{ "message":"Sending the campaign failed", "code":"1726" }], "status":"errors"}'
         );
         $this->expectException(FreshMailCampaignException::class);
-        $campaignTestHandler->testCampaign('id_hash', ['test@test.com']);
+        $campaignTestHandler->testCampaign('id_hash', [ 'test@test.com' ]);
     }
 }