cross-solution / YAWIK

module/Core/test/CoreTest/Controller/Plugin/NotificationTest.php

Spacing +17 added lines, -17 removed lines

@@ -75,7 +75,7 @@ 
 
         $mock->expects($this->once())
             ->method('addMessage')
-            ->with('some message',Notification::NAMESPACE_SUCCESS)
+            ->with('some message', Notification::NAMESPACE_SUCCESS)
             ->willReturn($mock)
         ;
 
@@ -87,7 +87,7 @@ 
         );
         $this->assertSame(
             $mock,
-            $mock('some message',Notification::NAMESPACE_SUCCESS)
+            $mock('some message', Notification::NAMESPACE_SUCCESS)
         );
     }
 
@@ -100,12 +100,12 @@ 
         $translator->expects($this->once())
             ->method('translate')
             ->willReturnMap([
-                ['some message','default','translated some message'],
+                ['some message', 'default', 'translated some message'],
             ])
         ;
         $listener->expects($this->exactly(2))
             ->method('trigger')
-            ->with(NotificationEvent::EVENT_NOTIFICATION_ADD,$this->isInstanceOf(NotificationEvent::class))
+            ->with(NotificationEvent::EVENT_NOTIFICATION_ADD, $this->isInstanceOf(NotificationEvent::class))
         ;
 
         $target->setTranslator($translator);
@@ -122,7 +122,7 @@ 
      * @param string $message
      * @dataProvider getTestCreateNamespacedMessages
      */
-    public function testCreateNamespacedMessages($method,$message)
+    public function testCreateNamespacedMessages($method, $message)
     {
         $mock = $this->getMockBuilder(Notification::class)
             ->disableOriginalConstructor()
@@ -136,21 +136,21 @@ 
             ->willReturn($mock)
         ;
 
-        $callback = array($mock,$method);
+        $callback = array($mock, $method);
         $this->assertSame(
             $mock,
-            call_user_func($callback,$message)
+            call_user_func($callback, $message)
         );
     }
 
     public function getTestCreateNamespacedMessages()
     {
         return [
-            ['info','info message'],
-            ['warning','warning message'],
-            ['success','success message'],
-            ['danger','danger message'],
-            ['error','error message']
+            ['info', 'info message'],
+            ['warning', 'warning message'],
+            ['success', 'success message'],
+            ['danger', 'danger message'],
+            ['error', 'error message']
         ];
     }
 
@@ -173,8 +173,8 @@ 
         $events->expects($this->exactly(2))
             ->method('getNotifications')
             ->willReturnOnConsecutiveCalls(
-                [],// setup for empty array test
-                [$entity1,$entity2] // setup for non empty array tests
+                [], // setup for empty array test
+                [$entity1, $entity2] // setup for non empty array tests
             )
         ;
 
@@ -191,8 +191,8 @@ 
         $mock->expects($this->exactly(2))
             ->method('renderMessage')
             ->withConsecutive(
-                ['some notification',Notification::NAMESPACE_INFO],
-                ['some notification',Notification::NAMESPACE_DANGER]
+                ['some notification', Notification::NAMESPACE_INFO],
+                ['some notification', Notification::NAMESPACE_DANGER]
             )
         ;
         $mock->createOutput($events);
@@ -222,6 +222,6 @@ 
             ->willReturn($flash)
         ;
 
-        $target->renderMessage('some message',Notification::NAMESPACE_INFO);
+        $target->renderMessage('some message', Notification::NAMESPACE_INFO);
     }
 }

module/Core/test/CoreTest/Controller/Plugin/CreatePaginatorTest.php

Indentation +7 added lines, -7 removed lines

@@ -92,13 +92,13 @@
             ->disableOriginalConstructor()
             ->getMock()
         ;
-	    $container = $this->getMockBuilder(ContainerInterface::class)
-	                      ->getMock();
-	    $container->expects($this->once())
-		    ->method('get')
-		    ->with('ServiceManager')
-		    ->willReturn($sm)
-		;
+        $container = $this->getMockBuilder(ContainerInterface::class)
+                            ->getMock();
+        $container->expects($this->once())
+            ->method('get')
+            ->with('ServiceManager')
+            ->willReturn($sm)
+        ;
 		    
         $em = $this->getMockBuilder(EventManager::class)
             ->disableOriginalConstructor()

Spacing +6 added lines, -6 removed lines

@@ -49,10 +49,10 @@ 
     public function providePaginatorCreationData()
     {
         return [
-            [ 'Test/Paginator', ['test' => 'value'], ['merged' => 'yes'], false, ['page' => 1, 'count' => 10, 'range' => 5] ],
-            [ 'Test2/YetAnotherPager', ['page' => 2], [], true, ['page' => 2, 'count' => 10, 'range' => 5] ],
-            [ 'Yet/Another', ['page' => 3, 'count' => 90, 'range' => 2], false, false, ['page' => 3, 'count' => 90, 'range' => 2] ],
-            [ 'Even/Another', ['test' => 'value'], [], new \ArrayObject(['test' => 'value']), ['page' => 1, 'count' => 10, 'range' => 5]],
+            ['Test/Paginator', ['test' => 'value'], ['merged' => 'yes'], false, ['page' => 1, 'count' => 10, 'range' => 5]],
+            ['Test2/YetAnotherPager', ['page' => 2], [], true, ['page' => 2, 'count' => 10, 'range' => 5]],
+            ['Yet/Another', ['page' => 3, 'count' => 90, 'range' => 2], false, false, ['page' => 3, 'count' => 90, 'range' => 2]],
+            ['Even/Another', ['test' => 'value'], [], new \ArrayObject(['test' => 'value']), ['page' => 1, 'count' => 10, 'range' => 5]],
         ];
     }
 
@@ -70,7 +70,7 @@ 
     public function testPaginatorCreation($paginatorName, $params, $defaultParams, $usePostParams, $expect)
     {
         if ($defaultParams) { $options = array_merge($params, $defaultParams); }
-        else                { $options = $params; }
+        else { $options = $params; }
         $request = new Request();
         if ($usePostParams) {
             $request->setPost(new Parameters($params));
@@ -102,7 +102,7 @@ 
 		    
         $em = $this->getMockBuilder(EventManager::class)
             ->disableOriginalConstructor()
-            ->setMethods(['getEvent','triggerEvent'])
+            ->setMethods(['getEvent', 'triggerEvent'])
             ->getMock()
         ;
 	    

Braces +1 added lines, -2 removed lines

@@ -69,8 +69,7 @@
      */
     public function testPaginatorCreation($paginatorName, $params, $defaultParams, $usePostParams, $expect)
     {
-        if ($defaultParams) { $options = array_merge($params, $defaultParams); }
-        else                { $options = $params; }
+        if ($defaultParams) { $options = array_merge($params, $defaultParams); } else                { $options = $params; }
         $request = new Request();
         if ($usePostParams) {
             $request->setPost(new Parameters($params));

module/Core/test/CoreTest/Controller/Plugin/FileSenderTest.php

Spacing +9 added lines, -9 removed lines

@@ -37,7 +37,7 @@ 
             ->willReturn($repositories)
         ;
         $ob = FileSender::factory($container);
-        $this->assertInstanceOf(FileSender::class,$ob);
+        $this->assertInstanceOf(FileSender::class, $ob);
     }
 
     public function testSendFile()
@@ -61,34 +61,34 @@ 
         ;
         $repo->expects($this->any())
             ->method('find')
-            ->willReturn(null,$file)
+            ->willReturn(null, $file)
         ;
 
         $sender = new FileSender($repositories);
         $sender->setController($controller);
 
         // fist test: response will be 404 if file is not found
-        $sender('someRepository',$fileId);
-        $this->assertEquals(404,$response->getStatusCode());
+        $sender('someRepository', $fileId);
+        $this->assertEquals(404, $response->getStatusCode());
 
         // second test: will handle send file properly
         $file->expects($this->any())
             ->method('__get')
             ->willReturnMap([
-                ['type','type'],
-                ['size','size']
+                ['type', 'type'],
+                ['size', 'size']
             ])
         ;
-        $resource = fopen(__FILE__,'r');
+        $resource = fopen(__FILE__, 'r');
         $file->expects($this->once())
             ->method('getResource')
             ->willReturn($resource)
         ;
         ob_start();
-        $sender('someRepository',$fileId);
+        $sender('someRepository', $fileId);
         $output = ob_get_contents();
         ob_end_clean();
 
-        $this->assertStringEqualsFile(__FILE__,$output);
+        $this->assertStringEqualsFile(__FILE__, $output);
     }
 }

module/Core/test/CoreTest/Controller/Plugin/MailTest.php

Spacing +23 added lines, -23 removed lines

@@ -90,20 +90,20 @@ 
         $this->viewResolver->expects($this->any())
             ->method('resolve')
             ->willReturnMap([
-                ['mail/null-template',null,null],
-                ['coretest/mail/null-template',null,__DIR__.'/fixtures/mail-template.phtml'],
-                ['mail/template',null,__DIR__.'/fixtures/mail-template.phtml'],
-                ['mail/exception',null,__DIR__.'/fixtures/error.phtml']
+                ['mail/null-template', null, null],
+                ['coretest/mail/null-template', null, __DIR__.'/fixtures/mail-template.phtml'],
+                ['mail/template', null, __DIR__.'/fixtures/mail-template.phtml'],
+                ['mail/exception', null, __DIR__.'/fixtures/error.phtml']
             ])
         ;
 
         $container->expects($this->any())
             ->method('get')
             ->willReturnMap([
-                ['Log/Core/Mail',$this->mailLog],
-                ['ViewResolver',$this->viewResolver],
-                ['EventManager',$this->events],
-                ['ModuleManager',$this->moduleManager]
+                ['Log/Core/Mail', $this->mailLog],
+                ['ViewResolver', $this->viewResolver],
+                ['EventManager', $this->events],
+                ['ModuleManager', $this->moduleManager]
             ])
         ;
 
@@ -131,15 +131,15 @@ 
     public function testTemplate()
     {
         $target = $this->target;
-        $params = ['Hello' => 'World','this'=>'this should not rendered'];
+        $params = ['Hello' => 'World', 'this'=>'this should not rendered'];
 
         $target($params);
         $target->template('null-template');
 
-        $this->assertInstanceOf(Sendmail::class,$target->getTransport());
-        $this->assertSame($this->controller,$target->getController());
-        $this->assertContains('Hello World!',$target->getBody());
-        $this->assertNotContains($params['this'],$target->getBody());
+        $this->assertInstanceOf(Sendmail::class, $target->getTransport());
+        $this->assertSame($this->controller, $target->getController());
+        $this->assertContains('Hello World!', $target->getBody());
+        $this->assertNotContains($params['this'], $target->getBody());
     }
 
     public function testTemplateThrowExceptionWhenTemplateFileError()
@@ -175,24 +175,24 @@ 
     /**
      * @dataProvider getTestInformationCompleteException
      */
-    public function testInformationCompleteException($remove,$message)
+    public function testInformationCompleteException($remove, $message)
     {
         $srcFile = __DIR__.'/fixtures/mail-template.phtml';
         $tplFile = sys_get_temp_dir().'/yawik/mail-template.phtml';
 
         $contents = file_get_contents($srcFile);
         $pattern = "/^.*".preg_quote($remove).".*$/m";
-        $contents = preg_replace($pattern,'',$contents);
-        if(!is_dir($dir = dirname($tplFile))){
-            mkdir($dir,0777,true);
+        $contents = preg_replace($pattern, '', $contents);
+        if (!is_dir($dir = dirname($tplFile))) {
+            mkdir($dir, 0777, true);
         }
-        file_put_contents($tplFile,$contents,LOCK_EX);
+        file_put_contents($tplFile, $contents, LOCK_EX);
 
         $resolver = $this->createMock(ResolverInterface::class);
         $resolver->expects($this->any())
             ->method('resolve')
             ->willReturnMap([
-                ['mail/exception',null,$tplFile]
+                ['mail/exception', null, $tplFile]
             ])
         ;
 
@@ -220,9 +220,9 @@ 
     public function getTestInformationCompleteException()
     {
         return [
-            ['$from','A from email address'],
-            ['$fromName','A from name'],
-            ['$subject','A subject must be'],
+            ['$from', 'A from email address'],
+            ['$fromName', 'A from name'],
+            ['$subject', 'A subject must be'],
         ];
     }
 
@@ -245,7 +245,7 @@ 
             'Hello' => 'World'
         ]);
         $target->setTransport($transport);
-        $target->getHeaders()->addHeaderLine('to','to@example.com');
+        $target->getHeaders()->addHeaderLine('to', 'to@example.com');
         $target->template('template');
         $this->assertTrue($target->send());
         $this->assertContains(

module/Core/test/CoreTest/Controller/Plugin/ContentCollectorTest.php

Spacing +4 added lines, -4 removed lines

@@ -45,9 +45,9 @@ 
         $viewModel = new ViewModel();
         $events->expects($this->any())
             ->method('trigger')
-            ->with($event,'some_target')
+            ->with($event, 'some_target')
             ->willReturn([
-                'test_template',$viewModel
+                'test_template', $viewModel
             ])
         ;
 
@@ -58,9 +58,9 @@ 
         $target->captureTo('some_path');
 
         /* @var \Zend\View\Model\ViewModel[] $childs */
-        $output = $target->trigger($event,'some_target');
+        $output = $target->trigger($event, 'some_target');
         $childs = $output->getChildren();
-        $this->assertInstanceOf(ViewModel::class,$output);
+        $this->assertInstanceOf(ViewModel::class, $output);
         $this->assertEquals(
             'test_template',
             $childs[0]->getTemplate()

module/Core/test/CoreTest/Controller/Plugin/ConfigFactoryTest.php

Spacing +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
             ->willReturn([])
         ;
         $factory = new ConfigFactory();
-        $service = $factory($container,'some');
+        $service = $factory($container, 'some');
 
-        $this->assertInstanceOf(Config::class,$service);
+        $this->assertInstanceOf(Config::class, $service);
     }
 }

module/Core/test/CoreTest/Controller/Plugin/ListQueryTest.php

Spacing +7 added lines, -7 removed lines

@@ -41,7 +41,7 @@ 
     {
         $container = $this->createMock(ContainerInterface::class);
         $ob = ListQuery::factory($container);
-        $this->assertInstanceOf(ListQuery::class,$ob);
+        $this->assertInstanceOf(ListQuery::class, $ob);
     }
 
     public function propertiesProvider()
@@ -49,23 +49,23 @@ 
         return [
             [
                 'pageParamName',
-                ['value' => 'somePage','setter_value'=>null,'default'=>'page']
+                ['value' => 'somePage', 'setter_value'=>null, 'default'=>'page']
             ],
             [
-                'propertiesMap' ,
-                ['value'=>'bar','setter_value'=>null,'default'=>array()]
+                'propertiesMap',
+                ['value'=>'bar', 'setter_value'=>null, 'default'=>array()]
             ],
             [
                 'itemsPerPage',
-                ['value' => 10,'setter_value'=>null,'default' => 25]
+                ['value' => 10, 'setter_value'=>null, 'default' => 25]
             ],
             [
                 'queryKeysLowercased',
-                ['value' => false,'setter_value'=>null,'default' => true]
+                ['value' => false, 'setter_value'=>null, 'default' => true]
             ],
             [
                 'sortParamName',
-                ['value' => 's','setter_value'=>null,'default' => 'sort']
+                ['value' => 's', 'setter_value'=>null, 'default' => 'sort']
             ],
         ];
     }

module/Core/test/CoreTest/Controller/Plugin/ConfigTest.php

Spacing +10 added lines, -10 removed lines

@@ -57,7 +57,7 @@ 
     {
         $config = include __DIR__.'/fixtures/config-dump.php';
         $moduleManager = $this->createMock(ModuleManagerInterface::class);
-        return new IndexController($moduleManager,$config);
+        return new IndexController($moduleManager, $config);
     }
 
     /**
@@ -65,13 +65,13 @@ 
      * @param string $module
      * @dataProvider getTestInvoke
      */
-    public function testInvoke($config,$key=null,$module=null,$expected)
+    public function testInvoke($config, $key = null, $module = null, $expected)
     {
         $plugin = new Config($config);
         $plugin->setController($this->getController());
-        $value = $plugin($key,$module);
+        $value = $plugin($key, $module);
         $this
-            ->assertEquals($expected,$value)
+            ->assertEquals($expected, $value)
         ;
     }
 
@@ -87,11 +87,11 @@ 
         ];
 
         return [
-            [   $config, 'dashboard', 'Jobs', $config['Jobs']['dashboard']  ],
-            [   $config, 'dashboard', ['Jobs','Applications'], $combinedOutput  ],
-            [   $config, 'settings', null, $config['Core']['settings']  ],
-            [   $config, 'Core', true, $config['Core']  ],
-            [   $config, ['dashboard','form'],['Jobs','Application'], ['Jobs' =>$config['Jobs']]    ],
+            [$config, 'dashboard', 'Jobs', $config['Jobs']['dashboard']],
+            [$config, 'dashboard', ['Jobs', 'Applications'], $combinedOutput],
+            [$config, 'settings', null, $config['Core']['settings']],
+            [$config, 'Core', true, $config['Core']],
+            [$config, ['dashboard', 'form'], ['Jobs', 'Application'], ['Jobs' =>$config['Jobs']]],
         ];
     }
 
@@ -102,7 +102,7 @@ 
         $plugin->setController($this->getController());
 
         $this
-            ->assertEquals($config['Core']['settings'],$plugin->settings)
+            ->assertEquals($config['Core']['settings'], $plugin->settings)
         ;
     }
 }

module/Core/test/CoreTest/Controller/Plugin/SearchFormTest.php

Indentation +5 added lines, -5 removed lines

@@ -31,7 +31,7 @@ 
  */
 class SearchFormTest extends \PHPUnit_Framework_TestCase
 {
-	use TestInheritanceTrait;
+    use TestInheritanceTrait;
 	
     protected $formElementManagerMock;
 
@@ -127,14 +127,14 @@ 
 
         $formData = ['data' => 'value'];
         $hydrator = $this->getMockBuilder(HydratorInterface::class)
-                         ->setMethods(['hydrate', 'extract'])
-                         ->getMockForAbstractClass();
+                            ->setMethods(['hydrate', 'extract'])
+                            ->getMockForAbstractClass();
         $hydrator->expects($this->once())->method('extract')->with($this->isInstanceOf(Parameters::class))->willReturn($formData);
         $hydrator->expects($this->once())->method('hydrate')->with($formData, $this->isInstanceOf(Parameters::class));
 
         $form = $this->getMockBuilder(Form::class)
-                     ->setMethods(['getHydrator', 'setData'])
-                     ->getMock();
+                        ->setMethods(['getHydrator', 'setData'])
+                        ->getMock();
         $form->expects($this->once())->method('gethydrator')->willReturn($hydrator);
         $form->expects($this->once())->method('setData')->with($formData);
         $this->formElementManagerMock

Spacing +3 added lines, -3 removed lines

@@ -39,7 +39,7 @@ 
         'class' => '\Core\Controller\Plugin\SearchForm',
         'args' => 'getTargetArgs',
         '@testInvokationProxiesToGet' => [
-            'mock' => [ 'get' ],
+            'mock' => ['get'],
             'args' => false,
         ],
         '@testInheritance' => [
@@ -47,7 +47,7 @@ 
         ],
     ];
 
-    protected $inheritance = [ AbstractPlugin::class ];
+    protected $inheritance = [AbstractPlugin::class];
 
     protected function getTargetArgs()
     {
@@ -63,7 +63,7 @@ 
     public function testInvokationProxiesToGet()
     {
         $form = 'TestForm';
-        $options  = ['test' => 'test'];
+        $options = ['test' => 'test'];
 
         $this->target->expects($this->once())->method('get')->with($form, $options);