phmLabs / Smoke

src/Rules/Http/Header/HttpStatusRule.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     public function validate(Response $response)
     {
         if ($response->getStatus() !== $this->expectedStatus) {
-            throw new ValidationFailedException('Status code ' . $response->getStatus() . ' found, ' . $this->expectedStatus . ' expected.');
+            throw new ValidationFailedException('Status code '.$response->getStatus().' found, '.$this->expectedStatus.' expected.');
         }
     }
 }

src/Rules/Html/SizeRule.php

Spacing +1 added lines, -1 removed lines

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

src/Rules/Html/ForeignDomainImageRule.php

Spacing +1 added lines, -1 removed lines

@@ -41,6 +41,6 @@
             }
         }
 
-        $this->assert(count($foreignImages) === 0, 'Images from a foreign domain where found (' . implode(', ', $foreignImages) . ')');
+        $this->assert(count($foreignImages) === 0, 'Images from a foreign domain where found ('.implode(', ', $foreignImages).')');
     }
 }

src/Rules/Html/XPathExistsRule.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
         $domXPath = new \DOMXPath($domDocument);
 
         foreach ($this->xPaths as $xpath) {
-            $this->assert($domXPath->query($xpath)->length > 0, 'The xpath ' . $xpath . ' was not found.');
+            $this->assert($domXPath->query($xpath)->length > 0, 'The xpath '.$xpath.' was not found.');
         }
     }
 }

src/Rules/Html/InsecureContentRule.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
 
         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/StringNotExistsRule.php

Spacing +1 added lines, -1 removed lines

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

src/Rules/Html/InvalidUrlsRule.php

Spacing +1 added lines, -1 removed lines

@@ -30,6 +30,6 @@
             }
         }
 
-        $this->assert(count($invalidUrls) === 0, 'Invalid urls found (' . implode(', ', $invalidUrls) . ').');
+        $this->assert(count($invalidUrls) === 0, 'Invalid urls found ('.implode(', ', $invalidUrls).').');
     }
 }

src/Rules/Html/StringExistsRule.php

Spacing +1 added lines, -1 removed lines

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

src/Rules/Xml/DuplicateIdRule.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
         if (count($duplicatedIds) > 0) {
             unset($duplicatedIds['']);
-            throw new ValidationFailedException('Duplicate ids found (' . implode(', ', array_keys($duplicatedIds)) . ')');
+            throw new ValidationFailedException('Duplicate ids found ('.implode(', ', array_keys($duplicatedIds)).')');
         }
     }
 }