cross-solution / YAWIK

module/Core/test/CoreTest/Factory/Paginator/RepositoryAbstractFactoryTest.php

Spacing +10 added lines, -10 removed lines

@@ -44,10 +44,10 @@ 
     public function serviceNamesProvider()
     {
         return [
-            [ 'Repository/Core/FileEntity', true ],
-            [ 'Core/FileEntity', true ],
-            [ 'Repository/NoModuleWillBeEverCalledThat/SomeEntity', false ],
-            [ 'ThisIsAHighlyUnCommonModuleName/ObscureEntity', false ],
+            ['Repository/Core/FileEntity', true],
+            ['Core/FileEntity', true],
+            ['Repository/NoModuleWillBeEverCalledThat/SomeEntity', false],
+            ['ThisIsAHighlyUnCommonModuleName/ObscureEntity', false],
         ];
     }
 
@@ -65,7 +65,7 @@ 
             ->disableOriginalConstructor()
             ->getMock()
         ;
-        $method = "assert" . ($expected ? 'True' : 'False');
+        $method = "assert".($expected ? 'True' : 'False');
 
         $this->{$method}($this->target->canCreate($sm, $serviceName));
     }
@@ -73,8 +73,8 @@ 
     public function servicesProvider()
     {
         return [
-            [ 'Repository/Core/TestEntity', '\Core\Entity\TestEntity', [], false ],
-            [ 'Core/TestEntity', '\Core\Entity\TestEntity', [ 'testOption' => 'testValue' ], true ]
+            ['Repository/Core/TestEntity', '\Core\Entity\TestEntity', [], false],
+            ['Core/TestEntity', '\Core\Entity\TestEntity', ['testOption' => 'testValue'], true]
         ];
     }
 
@@ -108,19 +108,19 @@ 
                         ->disableOriginalConstructor()
                         ->setMethods(['has', 'get'])->getMockForAbstractClass();
 
-        $filters->expects($this->once())->method('has')->with('PaginationQuery/' . $serviceName)->willReturn($hasFilter);
+        $filters->expects($this->once())->method('has')->with('PaginationQuery/'.$serviceName)->willReturn($hasFilter);
 
         if ($hasFilter) {
             $filter = $this->getMockBuilder('\Zend\Filter\FilterInterface')->getMockForAbstractClass();
             $filter->expects($this->once())->method('filter')->with($options, $qb)->willReturn($qb);
 
-            $filters->expects($this->once())->method('get')->with('PaginationQuery/' . $serviceName)->willReturn($filter);
+            $filters->expects($this->once())->method('get')->with('PaginationQuery/'.$serviceName)->willReturn($filter);
         }
 
         $sm = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')->disableOriginalConstructor()->getMock();
 
         $sm->expects($this->exactly(2))->method('get')
-                                       ->withConsecutive([ 'repositories' ], [ 'FilterManager'])
+                                       ->withConsecutive(['repositories'], ['FilterManager'])
                                        ->will($this->onConsecutiveCalls($repositories, $filters));
 
         $target = $this->target;

module/Core/test/CoreTest/Collection/IdentityWrapperTest.php

Spacing +16 added lines, -16 removed lines

@@ -34,7 +34,7 @@ 
     
     protected function setUp()
     {
-        $getEntryMock = function ($id) {
+        $getEntryMock = function($id) {
             $entry = $this->getMockBuilder(\stdClass::class)
                 ->setMethods(['getId'])
                 ->getMock();
@@ -291,11 +291,11 @@ 
     public function testExistsWithKey()
     {
         $searchKey = key($this->entries);
-        $this->assertTrue($this->identityWrapper->exists(function ($key) use ($searchKey) {
+        $this->assertTrue($this->identityWrapper->exists(function($key) use ($searchKey) {
             return $key === $searchKey;
         }));
         $searchKey = 'non-existent';
-        $this->assertFalse($this->identityWrapper->exists(function ($key) use ($searchKey) {
+        $this->assertFalse($this->identityWrapper->exists(function($key) use ($searchKey) {
             return $key === $searchKey;
         }));
     }
@@ -306,11 +306,11 @@ 
     public function testExistsWithValue()
     {
         $searchValue = current($this->entries);
-        $this->assertTrue($this->identityWrapper->exists(function ($key, $value) use ($searchValue) {
+        $this->assertTrue($this->identityWrapper->exists(function($key, $value) use ($searchValue) {
             return $value === $searchValue;
         }));
         $searchValue = 'non-existent';
-        $this->assertFalse($this->identityWrapper->exists(function ($key, $value) use ($searchValue) {
+        $this->assertFalse($this->identityWrapper->exists(function($key, $value) use ($searchValue) {
             return $value === $searchValue;
         }));
     }
@@ -321,7 +321,7 @@ 
     public function testFilter()
     {
         $searchValue = current($this->entries);
-        $filter = function ($value) use ($searchValue) {
+        $filter = function($value) use ($searchValue) {
             return $value === $searchValue;
         };
         $filtered = $this->identityWrapper->filter($filter);
@@ -336,11 +336,11 @@ 
     public function testForAllWithKey()
     {
         $keys = array_keys($this->entries);
-        $this->assertTrue($this->identityWrapper->forAll(function ($key) use ($keys) {
+        $this->assertTrue($this->identityWrapper->forAll(function($key) use ($keys) {
             return in_array($key, $keys);
         }));
         $keys = ['non-existent'];
-        $this->assertFalse($this->identityWrapper->forAll(function ($key) use ($keys) {
+        $this->assertFalse($this->identityWrapper->forAll(function($key) use ($keys) {
             return in_array($key, $keys);
         }));
     }
@@ -351,11 +351,11 @@ 
     public function testForAllWithValue()
     {
         $values = array_values($this->entries);
-        $this->assertTrue($this->identityWrapper->forAll(function ($key, $value) use ($values) {
+        $this->assertTrue($this->identityWrapper->forAll(function($key, $value) use ($values) {
             return in_array($value, $values, true);
         }));
         $values = ['non-existent'];
-        $this->assertFalse($this->identityWrapper->forAll(function ($key, $value) use ($values) {
+        $this->assertFalse($this->identityWrapper->forAll(function($key, $value) use ($values) {
             return in_array($value, $values, true);
         }));
     }
@@ -365,7 +365,7 @@ 
      */
     public function testMapValidOperation()
     {
-        $map = function ($value) {
+        $map = function($value) {
             $value->random = rand();
             return $value;
         };
@@ -381,7 +381,7 @@ 
      */
     public function testMapInvalidOperation()
     {
-        $map = function ($value) {
+        $map = function($value) {
             return sprintf('id: %s', $value->getId());
         };
         $this->identityWrapper->map($map);
@@ -393,7 +393,7 @@ 
     public function testPartitionWithKey()
     {
         $searchKey = key($this->entries);
-        $partitions = $this->identityWrapper->partition(function ($key) use ($searchKey) {
+        $partitions = $this->identityWrapper->partition(function($key) use ($searchKey) {
             return $key === $searchKey;
         });
         $expectedFirstPartiton = array_slice($this->entries, 0, 1, true);
@@ -412,7 +412,7 @@ 
     public function testPartitionWithValue()
     {
         $searchValue = reset($this->entries);
-        $partitions = $this->identityWrapper->partition(function ($key, $value) use ($searchValue) {
+        $partitions = $this->identityWrapper->partition(function($key, $value) use ($searchValue) {
             return $value === $searchValue;
         });
         $expectedFirstPartiton = array_slice($this->entries, 0, 1, true);
@@ -554,8 +554,8 @@ 
     public function testSetIdentityExtractor()
     {
         $suffix = '.suffix';
-        $identityExtractor = function ($entry) use ($suffix) {
-            return $entry->getId() . $suffix;
+        $identityExtractor = function($entry) use ($suffix) {
+            return $entry->getId().$suffix;
         };
         
         $this->assertSame($this->identityWrapper, $this->identityWrapper->setIdentityExtractor($identityExtractor));

module/Core/test/CoreTest/Options/ImageSetOptionsTest.php

Spacing +2 added lines, -2 removed lines

@@ -32,12 +32,12 @@
 
     private $target = ImageSetOptions::class;
 
-    private $inheritance = [ AbstractOptions::class ];
+    private $inheritance = [AbstractOptions::class];
 
     private $properties = [
         ['entityClass', ['default' => Image::class, '@value' => Image::class]],
         ['formElementName', ['default' => ImageSetInterface::ORIGINAL, 'value' => 'formElementNameValue']],
-        ['images', ['default'=>[ImageSetInterface::THUMBNAIL => [100,100]], 'value' => ['a' => [1,1]]]],
+        ['images', ['default'=>[ImageSetInterface::THUMBNAIL => [100, 100]], 'value' => ['a' => [1, 1]]]],
         ['entity', ['ignore_setter' => true, 'value@' => Image::class]],
     ];
 }

module/Core/test/CoreTest/Mail/MailService/BaseTest.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
             'stringtemplate' => '\Core\Mail\StringTemplateMessage',
         );
         $factories = array(
-            'htmltemplate'   => [HTMLTemplateMessage::class,'factory']
+            'htmltemplate'   => [HTMLTemplateMessage::class, 'factory']
         );
 
         $this->assertAttributeEquals(false, 'shareByDefault', $target, 'shareByDefault is not set to FALSE');

module/Core/test/CoreTest/Mail/MailService/SendMailTest.php

Spacing +6 added lines, -6 removed lines

@@ -40,7 +40,7 @@ 
     protected function setUp()
     {
         $test = $this;
-        $sendCallback = function ($value) use ($test) {
+        $sendCallback = function($value) use ($test) {
             return $value === $test->expectedMail;
         };
         $transport = $this->getMockForAbstractClass('\Zend\Mail\Transport\TransportInterface');
@@ -109,9 +109,9 @@ 
     public function testOverrideRecipient()
     {
         $overrideEmail = 'overidden@email';
-        $ccEmail="cc@email";
-        $bccEmail="bcc@email";
-        $toEmail="to@email";
+        $ccEmail = "cc@email";
+        $bccEmail = "bcc@email";
+        $toEmail = "to@email";
         
         $recipients = new AddressList();
         $recipients->add($overrideEmail);
@@ -128,13 +128,13 @@ 
         $this->target->send($mail);
 
         $headers = $mail->getHeaders();
-        $expectedTo = 'To: ' . $overrideEmail;
+        $expectedTo = 'To: '.$overrideEmail;
         $this->assertFalse($headers->has('cc'));
         $this->assertFalse($headers->has('bcc'));
         $this->assertTrue($headers->has('X-Original-Recipients'));
 
         $this->assertEquals($expectedTo, $headers->get('to')->toString());
-        $this->assertEquals('X-Original-Recipients: To: ' . $toEmail . '; Cc: ' . $ccEmail . '; Bcc: ' . $bccEmail, $headers->get('X-Original-Recipients')->toString());
+        $this->assertEquals('X-Original-Recipients: To: '.$toEmail.'; Cc: '.$ccEmail.'; Bcc: '.$bccEmail, $headers->get('X-Original-Recipients')->toString());
     }
 
     public function testSetsXMailerHeader()

module/Core/test/CoreTest/Mail/StringTemplateMessageTest.php

Spacing +11 added lines, -11 removed lines

@@ -67,11 +67,11 @@ 
         $addressList = new AddressList();
         $addressValue = ArrayUtils::iteratorToArray($addressList);
         return [
-            ['variables',[
+            ['variables', [
                 'value' => ['some' => 'value'],
                 'default' => [],
             ]],
-            ['variables',[
+            ['variables', [
                 'value' => $this,
                 'setter_exception' => [
                     \InvalidArgumentException::class,
@@ -79,30 +79,30 @@ 
                 ],
 
             ]],
-            ['variables',[
+            ['variables', [
                 'value' => $addressList,
                 'expect' => $addressValue
             ]],
-            ['template',[
+            ['template', [
                 'value' => 'some-template',
                 'default' => null
             ]],
-            ['callbacks',[
+            ['callbacks', [
                 'value' => [
-                    'some' => [$this,'someCallback']
+                    'some' => [$this, 'someCallback']
                 ],
                 'default' => array(),
             ]],
-            ['callbacks',[
+            ['callbacks', [
                 'value' => $this,
                 'setter_exception' => [
                     \InvalidArgumentException::class,
                     'Expect an array or an instance of \Traversable, but received'
                 ]
             ]],
-            ['callbacks',[
+            ['callbacks', [
                 'value' => [
-                    'uncallable' => [$this,'notCallable']
+                    'uncallable' => [$this, 'notCallable']
                 ],
                 'setter_exception' => [
                     \InvalidArgumentException::class,
@@ -115,7 +115,7 @@ 
     public function testGetBodyText()
     {
         $translator = $this->createMock(TranslatorInterface::class);
-        $target =  new StringTemplateMessage();
+        $target = new StringTemplateMessage();
         $target->setTranslator($translator);
 
         $translator->expects($this->any())
@@ -155,7 +155,7 @@ 
             trim($target->getSubject())
         );
 
-        $target->setCallback('callback', function () {
+        $target->setCallback('callback', function() {
             return 'Hello From Callback!';
         });
         $target->setCallback('object', 'getSubject');

module/Core/test/CoreTest/Mail/FileTransportTest.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         $options = $this->createMock(FileOptions::class);
         $options->expects($this->once())
             ->method('getCallback')
-            ->willReturn([$this,'getFileName'])
+            ->willReturn([$this, 'getFileName'])
         ;
         $options->expects($this->exactly(2))
             ->method('getPath')

module/Core/test/CoreTest/Mail/TranslatorAwareMessageTest.php

Spacing +2 added lines, -2 removed lines

@@ -38,11 +38,11 @@
      */
     private $target = TranslatorAwareMessage::class;
 
-    private $inheritance = [ Message::class, TranslatorAwareInterface::class ];
+    private $inheritance = [Message::class, TranslatorAwareInterface::class];
 
     public function propertiesProvider()
     {
-        $setTranslator = function () {
+        $setTranslator = function() {
             $this->target->setTranslator(new Translator());
         };
 

module/Core/test/CoreTest/Mail/MailServiceFactoryTest.php

Spacing +3 added lines, -3 removed lines

@@ -48,9 +48,9 @@
             ->method('get')
             ->willReturnMap(
                 [
-                    ['Config',[]],
-                    ['Auth/Options',$authOptions],
-                    ['Core/MailServiceOptions',$mailOptions]
+                    ['Config', []],
+                    ['Auth/Options', $authOptions],
+                    ['Core/MailServiceOptions', $mailOptions]
             ]
         );