cross-solution / YAWIK

module/Auth/test/AuthTest/Factory/Controller/Plugin/UserSwitcherFactoryTest.php

Indentation +3 added lines, -3 removed lines

@@ -50,9 +50,9 @@
         $container = $this->getServiceManagerMock();
         $plugins   = $this->getPluginManagerMock($container);
         $this->target
-	        ->expects($this->once())
-	        ->method('__invoke')
-	        ->with($container, 'Auth/User/Switcher')
+            ->expects($this->once())
+            ->method('__invoke')
+            ->with($container, 'Auth/User/Switcher')
         ;
 
         $this->target->createService($container);

Spacing +3 added lines, -3 removed lines

@@ -39,11 +39,11 @@ 
     private $target = [
         UserSwitcherFactory::class,
         '@testCreateServiceInvokesItself' => [
-            'mock' => [ '__invoke' ],
+            'mock' => ['__invoke'],
         ],
     ];
 
-    private $inheritance = [ FactoryInterface::class ];
+    private $inheritance = [FactoryInterface::class];
 
     public function testCreateServiceInvokesItself()
     {
@@ -63,7 +63,7 @@ 
         $auth = $this->getMockBuilder(AuthenticationService::class)->disableOriginalConstructor()->getMock();
         $acl = $this->getMockBuilder(Acl::class)->disableOriginalConstructor()->getMock();
         $sm = $this->createServiceManagerMock();
-        $controllerPlugins = $this->getPluginManagerMock(['Acl' => ['service' => $acl, 'count_get' => 1]],$sm);
+        $controllerPlugins = $this->getPluginManagerMock(['Acl' => ['service' => $acl, 'count_get' => 1]], $sm);
         $container = $this->getServiceManagerMock([
                 'AuthenticationService' => ['service' => $auth, 'count_get' => 1],
                 'ControllerPluginManager' => ['service' => $controllerPlugins, 'count_get' => 1]

module/Auth/test/AuthTest/Factory/View/Helper/AuthFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
         
         $hm = new HelperPluginManager($sm);
         
-        $helper = $f->__invoke($sm,Auth::class);
+        $helper = $f->__invoke($sm, Auth::class);
         
         $this->assertInstanceOf('\Auth\View\Helper\Auth', $helper);
         $this->assertSame($auth, $helper->getService());

module/Auth/test/AuthTest/Controller/ManageControllerTest.php

Indentation +28 added lines, -29 removed lines

@@ -18,44 +18,43 @@
 
 /**
  * Class ManageControllerTest
- 
  * @package AuthTest\Controller
  */
 class ManageControllerTest extends AbstractFunctionalControllerTestCase
 {
-	const URL_MY_PROFILE = '/en/my/profile';
+    const URL_MY_PROFILE = '/en/my/profile';
 	
-	private $hybridAuthAdapter;
+    private $hybridAuthAdapter;
 	
-	public function setUp()
-	{
-		parent::setUp();
+    public function setUp()
+    {
+        parent::setUp();
 		
-		$this->hybridAuthAdapter = $this->getMockBuilder(HybridAuth::class)
-			->disableOriginalConstructor()
-			->getMock()
-		;
-		$this->setMockToServiceLocator('HybridAuthAdapter',$this->hybridAuthAdapter);
-	}
+        $this->hybridAuthAdapter = $this->getMockBuilder(HybridAuth::class)
+            ->disableOriginalConstructor()
+            ->getMock()
+        ;
+        $this->setMockToServiceLocator('HybridAuthAdapter',$this->hybridAuthAdapter);
+    }
 	
-	public function testAccessWhenNotLoggedIn()
-	{
-		$this->dispatch(self::URL_MY_PROFILE, Request::METHOD_GET);
+    public function testAccessWhenNotLoggedIn()
+    {
+        $this->dispatch(self::URL_MY_PROFILE, Request::METHOD_GET);
 		
-		$result = $this->getResponse()->getContent();
+        $result = $this->getResponse()->getContent();
 		
-		$this->assertNotRedirect();
-		$this->assertResponseStatusCode(Response::STATUS_CODE_401);
-		$this->assertContains('Please authenticate yourself to proceed', $result);
-	}
+        $this->assertNotRedirect();
+        $this->assertResponseStatusCode(Response::STATUS_CODE_401);
+        $this->assertContains('Please authenticate yourself to proceed', $result);
+    }
 	
-	public function testAccessWhenLoggedIn()
-	{
-		$this->authenticateUser();
-		$this->dispatch(self::URL_MY_PROFILE,Request::METHOD_GET);
-		$result = $this->getResponse()->getContent();
-		$this->assertNotRedirect();
-		$this->assertResponseStatusCode(Response::STATUS_CODE_200);
-		$this->assertContains('My profile - YAWIK', $result);
-	}
+    public function testAccessWhenLoggedIn()
+    {
+        $this->authenticateUser();
+        $this->dispatch(self::URL_MY_PROFILE,Request::METHOD_GET);
+        $result = $this->getResponse()->getContent();
+        $this->assertNotRedirect();
+        $this->assertResponseStatusCode(Response::STATUS_CODE_200);
+        $this->assertContains('My profile - YAWIK', $result);
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
 			->disableOriginalConstructor()
 			->getMock()
 		;
-		$this->setMockToServiceLocator('HybridAuthAdapter',$this->hybridAuthAdapter);
+		$this->setMockToServiceLocator('HybridAuthAdapter', $this->hybridAuthAdapter);
 	}
 	
 	public function testAccessWhenNotLoggedIn()
@@ -52,7 +52,7 @@ 
 	public function testAccessWhenLoggedIn()
 	{
 		$this->authenticateUser();
-		$this->dispatch(self::URL_MY_PROFILE,Request::METHOD_GET);
+		$this->dispatch(self::URL_MY_PROFILE, Request::METHOD_GET);
 		$result = $this->getResponse()->getContent();
 		$this->assertNotRedirect();
 		$this->assertResponseStatusCode(Response::STATUS_CODE_200);

module/Auth/test/AuthTest/Controller/RemoveControllerTest.php

Indentation +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
         
         $pluginManager = $this->controller->getPluginManager();
         $pluginManager->setService('redirect', $redirect);
-		$pluginManager->setService('params', $params);
+        $pluginManager->setService('params', $params);
         
         $user = $this->getMockBuilder(User::class)
             ->getMock();

module/Auth/test/AuthTest/Controller/RegisterControllerTest.php

Indentation +2 added lines, -2 removed lines

@@ -52,8 +52,8 @@
 
 
         $this->serviceMock = $this->getMockBuilder('Auth\Service\Register')
-                                  ->disableOriginalConstructor()
-                                  ->getMock();
+                                    ->disableOriginalConstructor()
+                                    ->getMock();
 
 
         $this->paramsMock = $this->getMockBuilder('Zend\Mvc\Controller\Plugin\Params')

Spacing +3 added lines, -3 removed lines

@@ -134,7 +134,7 @@ 
     {
         $postData = array(
             'name' => uniqid('name'),
-            'email' => uniqid('email') . '@' . uniqid('host') . '.com.pl'
+            'email' => uniqid('email').'@'.uniqid('host').'.com.pl'
         );
 
         $request = new Request();
@@ -181,7 +181,7 @@ 
     {
         $postData = array(
             'name' => uniqid('name'),
-            'email' => uniqid('email') . '@' . uniqid('host') . '.com.pl'
+            'email' => uniqid('email').'@'.uniqid('host').'.com.pl'
         );
 
         $request = new Request();
@@ -228,7 +228,7 @@ 
     {
         $postData = array(
             'name' => uniqid('name'),
-            'email' => uniqid('email') . '@' . uniqid('host') . '.com.pl'
+            'email' => uniqid('email').'@'.uniqid('host').'.com.pl'
         );
 
         $request = new Request();

module/Auth/test/AuthTest/Controller/Plugin/UserSwitcherTest.php

Indentation +8 added lines, -8 removed lines

@@ -205,13 +205,13 @@ 
         $_SESSION = $oldSession;
         $this->assertTrue($this->target->switchUser('switchedUser'));
         $this->assertEquals(
-             [
-                 'isSwitchedUser' => true,
-                 'originalUser' => 'originalUser',
-                 'params' => [], 'ref' => '/some/ref',
-                 'session' => serialize($oldSession)
-             ],
-             $_SESSION[UserSwitcher::SESSION_NAMESPACE]->getArrayCopy()
+                [
+                    'isSwitchedUser' => true,
+                    'originalUser' => 'originalUser',
+                    'params' => [], 'ref' => '/some/ref',
+                    'session' => serialize($oldSession)
+                ],
+                $_SESSION[UserSwitcher::SESSION_NAMESPACE]->getArrayCopy()
         );
         $_SESSION = [];
     }
@@ -235,7 +235,7 @@ 
         $acl->expects($this->once())->method('setUser')->with($this->isInstanceOf(User::class));
 
         $user = $this->getMockBuilder(User::class)->disableOriginalConstructor()
-                     ->setMethods(['getId'])->getMock();
+                        ->setMethods(['getId'])->getMock();
         $user->expects($this->any())->method('getId')->willReturn('switchedUser');
 
         $this->target->switchUser($user, ['ref' => 'ref']);

Spacing +7 added lines, -7 removed lines

@@ -43,10 +43,10 @@ 
     private $target = [
         UserSwitcher::class,
         'getSimpleAuthMock',
-        '@testInheritance' => [ 'as_reflection' => true ],
+        '@testInheritance' => ['as_reflection' => true],
         '@testInvokationProxiesToCorrectMethods' => [
             'args' => false,
-            'mock' => [ 'clear' => 1, 'switchUser' => ['with' => 'testUserId', 'count' => 1]],
+            'mock' => ['clear' => 1, 'switchUser' => ['with' => 'testUserId', 'count' => 1]],
         ],
         '@testClearRestoresOriginalUserAndClearsSession' => [
             'args' => 'getComplexAuthMock',
@@ -62,7 +62,7 @@ 
         ],
     ];
 
-    private $inheritance = [ AbstractPlugin::class ];
+    private $inheritance = [AbstractPlugin::class];
 
     public function propertiesProvider()
     {
@@ -107,7 +107,7 @@ 
             ->disableOriginalConstructor()
             ->getMock();
 
-        return [ $auth ];
+        return [$auth];
     }
 
     private function getComplexAuthMock()
@@ -132,7 +132,7 @@ 
 
         $auth->expects($this->any())->method('getUser')->willReturn(new User());
 
-        return [ $auth ];
+        return [$auth];
     }
 
     public function testInvokationProxiesToCorrectMethods()
@@ -149,7 +149,7 @@ 
     public function returnReferenceProvider()
     {
         return [
-            [ null ], [ 'some/ref/uri' ],
+            [null], ['some/ref/uri'],
         ];
     }
 
@@ -163,7 +163,7 @@ 
         $session = new Container(UserSwitcher::SESSION_NAMESPACE);
         $session->isSwitchedUser = true;
         $session->originalUser = 'switchedUser';
-        $oldSession      = [
+        $oldSession = [
             'oldSession' => true,
             'must'       => 'be same'
         ];

module/Auth/test/AuthTest/Controller/PasswordControllerFunctionalTest.php

Indentation +34 added lines, -34 removed lines

@@ -43,9 +43,9 @@ 
             ]));
 	    
         $manager = $this->getMockBuilder(Manager::class)
-	        ->disableOriginalConstructor()
-	        ->getMock()
-	    ;
+            ->disableOriginalConstructor()
+            ->getMock()
+        ;
         $hybridAuth = $this->getMockBuilder(\Hybrid_Auth::class)->disableOriginalConstructor()->getMock();
         $this->setMockToServiceLocator('repositories', $this->repositoriesMock);
         $this->setMockToServiceLocator('Organizations\ImageFileCache\Manager',$manager);
@@ -61,7 +61,7 @@ 
 
     public function testAccessWhenYouAreLogged()
     {
-	    /*$repository = $this->getMockBuilder(OrganizationImage::class)
+        /*$repository = $this->getMockBuilder(OrganizationImage::class)
 	                       ->disableOriginalConstructor()
 	                       ->getMock()
 	    ;
@@ -82,37 +82,37 @@ 
         $this->assertContains('My password', $result);
     }
 	
-	public function testAccessWhenYouAreNotLoggedIn()
-	{
-		$this->dispatch(self::URL_MY_PASSWORD, Request::METHOD_GET);
+    public function testAccessWhenYouAreNotLoggedIn()
+    {
+        $this->dispatch(self::URL_MY_PASSWORD, Request::METHOD_GET);
 		
-		$result = $this->getResponse()->getContent();
+        $result = $this->getResponse()->getContent();
 		
-		$this->assertNotRedirect();
-		$this->assertResponseStatusCode(Response::STATUS_CODE_401);
-		$this->assertContains('Please authenticate yourself to proceed', $result);
-	}
+        $this->assertNotRedirect();
+        $this->assertResponseStatusCode(Response::STATUS_CODE_401);
+        $this->assertContains('Please authenticate yourself to proceed', $result);
+    }
 	
-	/**
-	 * Assert response status code
-	 *
-	 * @param int $code
-	 */
-	public function assertResponseStatusCode($code)
-	{
-		if ($this->useConsoleRequest) {
-			if (! in_array($code, [0, 1])) {
-				throw new \PHPUnit_Framework_ExpectationFailedException($this->createFailureMessage(
-					'Console status code assert value must be O (valid) or 1 (error)'
-				));
-			}
-		}
-		$match = $this->getResponseStatusCode();
-		if ($code != $match) {
-			throw new \PHPUnit_Framework_ExpectationFailedException($this->createFailureMessage(
-				sprintf('Failed asserting response code "%s", actual status code is "%s"', $code, $match)
-			));
-		}
-		$this->assertEquals($code, $match);
-	}
+    /**
+     * Assert response status code
+     *
+     * @param int $code
+     */
+    public function assertResponseStatusCode($code)
+    {
+        if ($this->useConsoleRequest) {
+            if (! in_array($code, [0, 1])) {
+                throw new \PHPUnit_Framework_ExpectationFailedException($this->createFailureMessage(
+                    'Console status code assert value must be O (valid) or 1 (error)'
+                ));
+            }
+        }
+        $match = $this->getResponseStatusCode();
+        if ($code != $match) {
+            throw new \PHPUnit_Framework_ExpectationFailedException($this->createFailureMessage(
+                sprintf('Failed asserting response code "%s", actual status code is "%s"', $code, $match)
+            ));
+        }
+        $this->assertEquals($code, $match);
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -21,9 +21,9 @@ 
 
     public function setUp()
     {
-        $logDir = __DIR__ . '/../../../../../log/';
-        $errorLogFile = $logDir . 'error.log';
-        $yawikLogFile = $logDir . 'yawik.log';
+        $logDir = __DIR__.'/../../../../../log/';
+        $errorLogFile = $logDir.'error.log';
+        $yawikLogFile = $logDir.'yawik.log';
 
         if ((file_exists($errorLogFile) && !is_writable($errorLogFile))
             || (file_exists($yawikLogFile) && !is_writable($yawikLogFile))
@@ -39,7 +39,7 @@ 
             ->getMock();
         $this->repositoriesMock->expects($this->any())->method('get')
             ->will($this->returnValueMap([
-                [ 'Organizations/OrganizationImage', $orgImageRepo ]
+                ['Organizations/OrganizationImage', $orgImageRepo]
             ]));
 	    
         $manager = $this->getMockBuilder(Manager::class)
@@ -48,7 +48,7 @@ 
 	    ;
         $hybridAuth = $this->getMockBuilder(\Hybrid_Auth::class)->disableOriginalConstructor()->getMock();
         $this->setMockToServiceLocator('repositories', $this->repositoriesMock);
-        $this->setMockToServiceLocator('Organizations\ImageFileCache\Manager',$manager);
+        $this->setMockToServiceLocator('Organizations\ImageFileCache\Manager', $manager);
         $this->setMockToServiceLocator('HybridAuth', $hybridAuth);
     }
 
@@ -101,7 +101,7 @@ 
 	public function assertResponseStatusCode($code)
 	{
 		if ($this->useConsoleRequest) {
-			if (! in_array($code, [0, 1])) {
+			if (!in_array($code, [0, 1])) {
 				throw new \PHPUnit_Framework_ExpectationFailedException($this->createFailureMessage(
 					'Console status code assert value must be O (valid) or 1 (error)'
 				));

module/Auth/test/AuthTest/Entity/InfoTest.php

Spacing +3 added lines, -3 removed lines

@@ -242,10 +242,10 @@
     public function provideGetDisplayNameTestData()
     {
         return array(
-            array(null,        null,      "name@example.de", "name@example.de"),
+            array(null, null, "name@example.de", "name@example.de"),
             array(" Carsten ", " Bleek ", "name@example.de", "Carsten Bleek"),
-            array(null,        " Bleek ", "name@example.de", "Bleek"),
-            array(" Carsten ", null,      "name@example.de", "name@example.de"),
+            array(null, " Bleek ", "name@example.de", "Bleek"),
+            array(" Carsten ", null, "name@example.de", "name@example.de"),
         );
     }
 

module/Auth/test/AuthTest/Entity/AnonymousUserTest.php

Spacing +2 added lines, -2 removed lines

@@ -83,9 +83,9 @@
      */
     public function testGetId()
     {
-        $this->assertEquals('token:' . $this->target->getToken(), $this->target->getId());
+        $this->assertEquals('token:'.$this->target->getToken(), $this->target->getId());
 
-        $id=uniqid();
+        $id = uniqid();
         $this->target->setId($id);
         $this->assertEquals($id, $this->target->getId());
     }