zsturgess / overwatch

src/UserBundle/Security/ApiAuthenticator.php

Spacing +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
         
         $apiToken = hash_hmac(
             'sha256',
-            'timestamp=' . $credentials[self::TIMESTAMP],
+            'timestamp='.$credentials[self::TIMESTAMP],
             $user->getApiKey()
         );
         

src/UserBundle/Tests/Controller/ApiControllerTest.php

Spacing +12 added lines, -12 removed lines

@@ -20,7 +20,7 @@ 
         $this->loginAs(UserFixtures::$users['user-1'], 'overwatchApi');
         $this->client = $this->makeClient(); //When using loginAs, we must re-create the client
         $this->client->enableProfiler();
-        $this->client->request('POST', '/api/users/' . $email);
+        $this->client->request('POST', '/api/users/'.$email);
         
         $this->assertJsonResponse($this->client->getResponse());
         
@@ -65,7 +65,7 @@ 
     public function testFindUser()
     {
         $this->logIn('ROLE_SUPER_ADMIN');
-        $this->client->request('GET', '/api/users/' . UserFixtures::$users['user-1']);
+        $this->client->request('GET', '/api/users/'.UserFixtures::$users['user-1']);
         
         $this->assertJsonResponse($this->client->getResponse());
         $this->assertJsonStringEqualsJsonString(
@@ -80,7 +80,7 @@ 
     public function testFindUserInsufficentPerms()
     {
         $this->logIn('ROLE_ADMIN');
-        $this->client->request('GET', '/api/users/' . UserFixtures::$users['user-1']);
+        $this->client->request('GET', '/api/users/'.UserFixtures::$users['user-1']);
         
         $this->assertForbidden($this->client->getResponse());
     }
@@ -151,7 +151,7 @@ 
     {
         $this->loginAs(UserFixtures::$users['user-1'], 'overwatchApi');
         $this->client = $this->makeClient(); //When using loginAs, we must re-create the client
-        $this->client->request('POST', '/api/users/' . UserFixtures::$users['user-2']->getId() . '/lock');
+        $this->client->request('POST', '/api/users/'.UserFixtures::$users['user-2']->getId().'/lock');
         
         $user = $this->em->find('Overwatch\UserBundle\Entity\User', UserFixtures::$users['user-2']->getId());
         $this->assertTrue($user->isLocked());
@@ -166,7 +166,7 @@ 
     public function testToggleLockUserInsufficentPerms()
     {
         $this->logIn('ROLE_ADMIN');
-        $this->client->request('POST', '/api/users/' . UserFixtures::$users['user-1']->getId() . '/lock');
+        $this->client->request('POST', '/api/users/'.UserFixtures::$users['user-1']->getId().'/lock');
         
         $this->assertForbidden($this->client->getResponse());
     }
@@ -175,7 +175,7 @@ 
     {
         $this->loginAs(UserFixtures::$users['user-1'], 'overwatchApi');
         $this->client = $this->makeClient(); //When using loginAs, we must re-create the client
-        $this->client->request('POST', '/api/users/' . UserFixtures::$users['user-1']->getId() . '/lock');
+        $this->client->request('POST', '/api/users/'.UserFixtures::$users['user-1']->getId().'/lock');
         
         $this->assertForbidden($this->client->getResponse());
     }
@@ -184,7 +184,7 @@ 
     {
         $this->loginAs(UserFixtures::$users['user-1'], 'overwatchApi');
         $this->client = $this->makeClient(); //When using loginAs, we must re-create the client
-        $this->client->request('POST', '/api/users/' . UserFixtures::$users['user-2']->getId() . '/role/ROLE_ADMIN');
+        $this->client->request('POST', '/api/users/'.UserFixtures::$users['user-2']->getId().'/role/ROLE_ADMIN');
         
         $user = $this->em->find('Overwatch\UserBundle\Entity\User', UserFixtures::$users['user-2']->getId());
         $this->assertTrue($user->hasRole('ROLE_ADMIN'));
@@ -199,7 +199,7 @@ 
     public function testSetUserRoleInsufficentPerms()
     {
         $this->logIn('ROLE_ADMIN');
-        $this->client->request('POST', '/api/users/' . UserFixtures::$users['user-1']->getId() . '/role/ROLE_USER');
+        $this->client->request('POST', '/api/users/'.UserFixtures::$users['user-1']->getId().'/role/ROLE_USER');
         
         $this->assertForbidden($this->client->getResponse());
     }
@@ -208,7 +208,7 @@ 
     {
         $this->loginAs(UserFixtures::$users['user-1'], 'overwatchApi');
         $this->client = $this->makeClient(); //When using loginAs, we must re-create the client
-        $this->client->request('POST', '/api/users/' . UserFixtures::$users['user-1']->getId() . '/role/ROLE_USER');
+        $this->client->request('POST', '/api/users/'.UserFixtures::$users['user-1']->getId().'/role/ROLE_USER');
         
         $this->assertForbidden($this->client->getResponse());
     }
@@ -217,7 +217,7 @@ 
     {
         $this->loginAs(UserFixtures::$users['user-1'], 'overwatchApi');
         $this->client = $this->makeClient(); //When using loginAs, we must re-create the client
-        $this->client->request('DELETE', '/api/users/' . UserFixtures::$users['user-2']->getId());
+        $this->client->request('DELETE', '/api/users/'.UserFixtures::$users['user-2']->getId());
         
         $user = $this->em->find('Overwatch\UserBundle\Entity\User', UserFixtures::$users['user-2']->getId());
         $this->assertNull($user);
@@ -228,7 +228,7 @@ 
     public function testDeleteUserInsufficentPerms()
     {
         $this->logIn('ROLE_ADMIN');
-        $this->client->request('DELETE', '/api/users/' . UserFixtures::$users['user-1']->getId());
+        $this->client->request('DELETE', '/api/users/'.UserFixtures::$users['user-1']->getId());
         
         $this->assertForbidden($this->client->getResponse());
     }
@@ -237,7 +237,7 @@ 
     {
         $this->loginAs(UserFixtures::$users['user-1'], 'overwatchApi');
         $this->client = $this->makeClient(); //When using loginAs, we must re-create the client
-        $this->client->request('DELETE', '/api/users/' . UserFixtures::$users['user-1']->getId());
+        $this->client->request('DELETE', '/api/users/'.UserFixtures::$users['user-1']->getId());
         
         $this->assertForbidden($this->client->getResponse());
     }

src/UserBundle/Tests/Controller/AppControllerTest.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         $firewall = 'overwatch';
         
         $token = new UsernamePasswordToken('testUser', null, $firewall, [$role]);
-        $session->set('_security_' . $firewall, serialize($token));
+        $session->set('_security_'.$firewall, serialize($token));
         $session->save();
         $cookie = new Cookie($session->getName(), $session->getId());
         $this->client->getCookieJar()->set($cookie);

src/UserBundle/Tests/Security/ApiAuthenticatorTest.php

Spacing +1 added lines, -1 removed lines

@@ -177,7 +177,7 @@
         $timestamp = time();
         $apiToken = hash_hmac(
             'sha256',
-            'timestamp=' . $timestamp,
+            'timestamp='.$timestamp,
             $this->user->getApiKey()
         );
         

src/UserBundle/Tests/Base/WebDriverTestCase.php

Spacing +3 added lines, -3 removed lines

@@ -85,9 +85,9 @@
     {
         if ($this->webDriver !== null) {
             if (in_array($this->getStatus(), [\PHPUnit_Runner_BaseTestRunner::STATUS_FAILURE, \PHPUnit_Runner_BaseTestRunner::STATUS_ERROR])) {
-                echo PHP_EOL . PHP_EOL . PHP_EOL;
-                echo 'data:image/png;base64,' . base64_encode($this->webDriver->takeScreenshot());
-                echo PHP_EOL . PHP_EOL . PHP_EOL;
+                echo PHP_EOL.PHP_EOL.PHP_EOL;
+                echo 'data:image/png;base64,'.base64_encode($this->webDriver->takeScreenshot());
+                echo PHP_EOL.PHP_EOL.PHP_EOL;
             }
         }
         

src/ServiceBundle/Reporter/EmailReporter.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     private function sendEmail(TestResult $result, array $users)
     {
         $message = \Swift_Message::newInstance()
-            ->setSubject($result->getTest()->getName() . ' ' . $result->getStatus())
+            ->setSubject($result->getTest()->getName().' '.$result->getStatus())
             ->setFrom($this->config['report_from'])
             ->setTo($users)
             ->setBody(

src/ServiceBundle/DependencyInjection/OverwatchServiceExtension.php

Spacing +2 added lines, -2 removed lines

@@ -22,10 +22,10 @@
         $configuration = $this->processConfiguration(new Configuration(), $configs);
         
         foreach ($configuration as $service => $config) {
-            $container->setParameter('overwatch_service.' . $service, $config);
+            $container->setParameter('overwatch_service.'.$service, $config);
         }
         
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.yml');
     }
 }

src/ServiceBundle/Tests/Reporter/EmailReporterTest.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         
         $message = $this->mailerSpy->getInvocations()[0]->parameters[0];
         $this->assertEquals(
-            $result->getTest()->getName() . ' ' . $result->getStatus(),
+            $result->getTest()->getName().' '.$result->getStatus(),
             $message->getSubject()
         );
         

src/ServiceBundle/Tests/Reporter/SmsReporterTest.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
         
         $message = $this->messageSpy->getInvocations()[0]->parameters[2];
         $this->assertEquals(
-            $result->getTest()->getName() . ' ' . $result->getStatus() . ': Me gusta success kid upvoting Obama first world problems.' . PHP_EOL,
+            $result->getTest()->getName().' '.$result->getStatus().': Me gusta success kid upvoting Obama first world problems.'.PHP_EOL,
             $message
         );