cross-solution / YAWIK

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

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()
         ;
 	    

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

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);
 

module/Core/test/CoreTest/Controller/Plugin/PaginationBuilder/BaseTest.php

Spacing +15 added lines, -15 removed lines

@@ -45,7 +45,7 @@ 
         ],
     ];
 
-    protected $inheritance = [ '\Zend\Mvc\Controller\Plugin\AbstractPlugin' ];
+    protected $inheritance = ['\Zend\Mvc\Controller\Plugin\AbstractPlugin'];
 
     public function testInvokationWithoutArgumentsReturnsSelf()
     {
@@ -91,16 +91,16 @@ 
     public function argumentsStackProvider()
     {
         return [
-            [ 'paginator', ['paginator'], ['as' => 'paginator', 'paginator', []] ],
-            [ 'paginator', ['name', 'alias'], ['as' => 'alias', 'name', []] ],
-            [ 'paginator', ['name', ['param' => 'value'], 'alias'], ['as' => 'alias', 'name', ['param' => 'value']] ],
-            [ 'paginator', ['name', []], ['as' => 'paginator', 'name', []]],
-            [ 'form', ['formName'], ['as' => 'searchform', 'formName', null]],
-            [ 'form', ['formName', ['testOpt' => 'testVal']], ['as' => 'searchform', 'formName', ['testOpt' => 'testVal']]],
-            [ 'form', ['formName', null, 'alias'], ['as' => 'alias', 'formName', null]],
-            [ 'form', ['formName', 'alias'], ['as' => 'alias', 'formName', null]],
-            [ 'params', ['namespace'], ['namespace', ['page' => 1]]],
-            [ 'params', ['namespace', ['param' => 'value']], ['namespace', ['param' => 'value']]],
+            ['paginator', ['paginator'], ['as' => 'paginator', 'paginator', []]],
+            ['paginator', ['name', 'alias'], ['as' => 'alias', 'name', []]],
+            ['paginator', ['name', ['param' => 'value'], 'alias'], ['as' => 'alias', 'name', ['param' => 'value']]],
+            ['paginator', ['name', []], ['as' => 'paginator', 'name', []]],
+            ['form', ['formName'], ['as' => 'searchform', 'formName', null]],
+            ['form', ['formName', ['testOpt' => 'testVal']], ['as' => 'searchform', 'formName', ['testOpt' => 'testVal']]],
+            ['form', ['formName', null, 'alias'], ['as' => 'alias', 'formName', null]],
+            ['form', ['formName', 'alias'], ['as' => 'alias', 'formName', null]],
+            ['params', ['namespace'], ['namespace', ['page' => 1]]],
+            ['params', ['namespace', ['param' => 'value']], ['namespace', ['param' => 'value']]],
         ];
     }
 
@@ -124,14 +124,14 @@ 
     {
         $this->target->expects($this->exactly(2))->method('paginator')
             ->withConsecutive(
-                [ 'Name' ],
-                [ 'Name', 'Alias']
+                ['Name'],
+                ['Name', 'Alias']
             )->willReturn($this->returnSelf());
 
         $this->target->expects($this->exactly(2))->method('form')
             ->withConsecutive(
-                [ 'Name' ],
-                [ 'Name', 'Alias']
+                ['Name'],
+                ['Name', 'Alias']
             )->willReturn($this->returnSelf());
 
         $stack = [