phmLabs / Smoke

src/Rules/Html/SizeRule.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@
 
     protected function doValidation(ResponseInterface $response)
     {
-        $size = strlen((string)$response->getBody()) / 1000;
-        $this->assert($size <= $this->maxSize, 'The size of this html file is too big (' . $size . ' KB)');
+        $size = strlen((string) $response->getBody()) / 1000;
+        $this->assert($size <= $this->maxSize, 'The size of this html file is too big ('.$size.' KB)');
     }
 }

src/Rules/Html/StringNotExistsRule.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@
 
     public function doValidation(ResponseInterface $response)
     {
-        $this->assert(strpos((string)$response->getBody(), $this->string) !== false,
-            'The given string (' . $this->string . ') was found in this document.');
+        $this->assert(strpos((string) $response->getBody(), $this->string) !== false,
+            'The given string ('.$this->string.') was found in this document.');
     }
 }

src/Rules/Html/StringExistsRule.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@
 
     protected function doValidation(ResponseInterface $response)
     {
-        $this->assert(strpos((string)$response->getBody(), $this->string) !== false,
-            'The given string (' . $this->string . ') was not found in this document.');
+        $this->assert(strpos((string) $response->getBody(), $this->string) !== false,
+            'The given string ('.$this->string.') was not found in this document.');
     }
 }

src/Rules/Html/InsecureContentRule.php

Spacing +2 added lines, -2 removed lines

@@ -27,13 +27,13 @@
             return;
         }
 
-        $htmlDocument = new Document((string)$response->getBody());
+        $htmlDocument = new Document((string) $response->getBody());
 
         $resources = $htmlDocument->getDependencies($uri, false);
 
         foreach ($resources as $resource) {
             if ($resource->getScheme() && 'https' !== $resource->getScheme()) {
-                throw new ValidationFailedException('At least one dependency was found on a secure url, that was transfered insecure (' . (string)$resource . ')');
+                throw new ValidationFailedException('At least one dependency was found on a secure url, that was transfered insecure ('.(string) $resource.')');
             }
         }
     }

src/Rules/Html/NoIndexRule.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
             return;
         }
 
-        $crawler = new Crawler((string)$response->getBody());
+        $crawler = new Crawler((string) $response->getBody());
         $metaTags = $crawler->filterXPath("//meta[@name='robots']/@content");
 
         foreach ($metaTags as $metaTag) {

src/Rules/Js/SyntaxRule.php

Spacing +6 added lines, -6 removed lines

@@ -30,18 +30,18 @@
             return;
         }
 
-        $filename = $this->tmpDir . DIRECTORY_SEPARATOR . md5((string)$response->getBody()) . '.js';
-        file_put_contents($filename, (string)$response->getBody());
-        $conf = __DIR__ . DIRECTORY_SEPARATOR . 'jsHint.conf';
+        $filename = $this->tmpDir.DIRECTORY_SEPARATOR.md5((string) $response->getBody()).'.js';
+        file_put_contents($filename, (string) $response->getBody());
+        $conf = __DIR__.DIRECTORY_SEPARATOR.'jsHint.conf';
 
-        $command = $this->jsHintExecutable . ' --config ' . $conf . ' --verbose ' . $filename . ' | grep -E E[0-9]+.$';
+        $command = $this->jsHintExecutable.' --config '.$conf.' --verbose '.$filename.' | grep -E E[0-9]+.$';
         $validationResult = shell_exec($command);
 
         unlink($filename);
 
         if (!is_null($validationResult)) {
-            $errorMsg = str_replace($filename . ':', '', $validationResult);
-            throw new ValidationFailedException('JavaScript error found: ' . $errorMsg);
+            $errorMsg = str_replace($filename.':', '', $validationResult);
+            throw new ValidationFailedException('JavaScript error found: '.$errorMsg);
         }
     }
 }

src/Rules/Image/FavIconRule.php

Spacing +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
             return;
         }
 
-        $imageHash = md5((string)$response->getBody());
+        $imageHash = md5((string) $response->getBody());
 
-        $this->assert(!array_key_exists($imageHash, $this->favicons), 'Seems like you use the standard favicon of your framework (' . $this->favicons[$imageHash] . ').');
+        $this->assert(!array_key_exists($imageHash, $this->favicons), 'Seems like you use the standard favicon of your framework ('.$this->favicons[$imageHash].').');
     }
 }

src/Rules/Image/SizeRule.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@
 
     protected function doValidation(ResponseInterface $response)
     {
-        $size = strlen((string)$response->getBody()) / 1000;
-        $this->assert($size <= $this->maxSize, 'The size of the file is too big (' . $size . ' KB)');
+        $size = strlen((string) $response->getBody()) / 1000;
+        $this->assert($size <= $this->maxSize, 'The size of the file is too big ('.$size.' KB)');
     }
 }

src/Rules/Security/PasswordSecureTransferRule.php

Spacing +3 added lines, -3 removed lines

@@ -19,10 +19,10 @@ 
 
     protected function doValidation(ResponseInterface $response)
     {
-        $crawler = new Crawler((string)$response->getBody());
+        $crawler = new Crawler((string) $response->getBody());
         $actionNodes = $crawler->filterXPath('//form[//input[@type="password"]]');
 
-        $url = (string)$response->getUri();
+        $url = (string) $response->getUri();
 
         foreach ($actionNodes as $node) {
             $action = $node->getAttribute('action');
@@ -36,7 +36,7 @@ 
 
             while ($parent = $parent->parentNode) {
                 if (property_exists($parent, 'tagName')) {
-                    $fullPath = $parent->tagName . '/' . $fullPath;
+                    $fullPath = $parent->tagName.'/'.$fullPath;
                 } else {
                     break;
                 }