cross-solution / YAWIK

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

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

Indentation +14 added lines, -14 removed lines

@@ -45,17 +45,17 @@ 
         $this->pluginMock = $pluginMock;
 
         $controller = $this->getMockBuilder('\Zend\Mvc\Controller\AbstractController')
-                           ->setMethods(['getRequest', 'plugin'])
-                           ->getMockForAbstractClass();
+                            ->setMethods(['getRequest', 'plugin'])
+                            ->getMockForAbstractClass();
 
         $controller->expects($this->any())->method('getRequest')->willReturn($this->request);
         $controller->expects($this->any())->method('plugin')->will($this->returnValueMap(
-                                                                       [
-                                                                           ['paginator', null, $pluginMock],
-                                                                           ['paginationParams', null, $pluginMock],
-                                                                           ['searchform', null, $pluginMock],
-                                                                       ]
-                                                                   ));
+                                                                        [
+                                                                            ['paginator', null, $pluginMock],
+                                                                            ['paginationParams', null, $pluginMock],
+                                                                            ['searchform', null, $pluginMock],
+                                                                        ]
+                                                                    ));
 
         $this->target->setController($controller);
 
@@ -117,12 +117,12 @@ 
         $actual = $query->toArray();
 
         $this->assertAttributeEquals(
-             new Parameters([
-                 'a' => 'test', 'b' => ['test1', 'test2'],
-                 'c' => ['test1', 'test2'],
-                 'd' => ['test1' => 1, 'test2' => 1]
-             ]),
-             'parameters', $this->target);
+                new Parameters([
+                    'a' => 'test', 'b' => ['test1', 'test2'],
+                    'c' => ['test1', 'test2'],
+                    'd' => ['test1' => 1, 'test2' => 1]
+                ]),
+                'parameters', $this->target);
     }
 }
 

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 = [

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

Spacing +18 added lines, -18 removed lines

@@ -60,7 +60,7 @@ 
     {
         $mock = $this->getMockBuilder(PaginationParams::class)
             ->disableOriginalConstructor()
-            ->setMethods(['getParams','getList'])
+            ->setMethods(['getParams', 'getList'])
             ->getMock()
         ;
         $repository = $this->createMock(RepositoryInterface::class);
@@ -68,7 +68,7 @@ 
         // setup getParams call
         $mock->expects($this->once())
             ->method('getParams')
-            ->with('some/namespace',['page'=>1],null)
+            ->with('some/namespace', ['page'=>1], null)
             ->willReturn('getParams called')
         ;
 
@@ -76,8 +76,8 @@ 
         $mock->expects($this->exactly(2))
             ->method('getList')
             ->withConsecutive(
-                ['some/namespace',[$this,'methodToTest']],
-                ['some/namespace',$repository]
+                ['some/namespace', [$this, 'methodToTest']],
+                ['some/namespace', $repository]
             )
             ->willReturn('getList called')
         ;
@@ -95,12 +95,12 @@ 
         );
         $this->assertEquals(
             'getList called',
-            $mock('some/namespace',[$this,'methodToTest']),
+            $mock('some/namespace', [$this, 'methodToTest']),
             '__invoke() should call ::getList when defaults is callable'
         );
         $this->assertEquals(
             'getList called',
-            $mock('some/namespace',$repository),
+            $mock('some/namespace', $repository),
             '__invoke() should call ::getList when $defaults is a Repository'
         );
     }
@@ -114,7 +114,7 @@ 
     {
         $mock = new PaginationParams();
         $params = ['key' => 'value'];
-        $mock->setParams('namespace',['key'=>'value']);
+        $mock->setParams('namespace', ['key'=>'value']);
         $container = new Container('namespace');
         $this->assertEquals(
             $params,
@@ -132,11 +132,11 @@ 
             ->method('getQuery')
             ->willReturn($params)
         ;
-        $params->set('clear',true);
-        $output = $mock->getParams('namespace',[]);
+        $params->set('clear', true);
+        $output = $mock->getParams('namespace', []);
         $this->assertNull($output->get('clear'));
         // unset clear value to be used for next test
-        $params->set('clear',false);
+        $params->set('clear', false);
 
 
         $session = new Container('namespace');
@@ -148,8 +148,8 @@ 
             1 => 'session',
             2 => 'undefined'
         ];
-        $params->set('existing','existing value');
-        $output = $mock->getParams('namespace',$defaults,$params);
+        $params->set('existing', 'existing value');
+        $output = $mock->getParams('namespace', $defaults, $params);
         $this->assertEquals(
             'value',
             $output->get('key'),
@@ -177,8 +177,8 @@ 
         $session->list = null;
         $mock = $this->target;
 
-        $callback = [$this,'methodToTest'];
-        $output = $mock->getList('namespace',$callback);
+        $callback = [$this, 'methodToTest'];
+        $output = $mock->getList('namespace', $callback);
         $this->assertEquals(
             'methodToTest called',
             $output,
@@ -188,7 +188,7 @@ 
         $session->list = 'some value';
         $this->assertEquals(
             'some value',
-            $mock->getList('namespace',$callback)
+            $mock->getList('namespace', $callback)
         );
     }
 
@@ -203,7 +203,7 @@ 
 
         $mock->expects($this->once())
             ->method('getList')
-            ->with('namespace','callback')
+            ->with('namespace', 'callback')
             ->willReturn($list)
         ;
 
@@ -220,8 +220,8 @@ 
             ->willReturn('next')
         ;
         $this->assertEquals(
-            ['previous','next'],
-            $mock->getNeighbours('namespace','callback','id')
+            ['previous', 'next'],
+            $mock->getNeighbours('namespace', 'callback', 'id')
         );
     }
 }