phmLabs / Smoke

src/Rules/Xml/XmlValidXsdRule.php

Spacing +4 added lines, -4 removed lines

@@ -22,12 +22,12 @@ 
 
     protected function doValidation(ResponseInterface $response)
     {
-        $body = (string)$response->getBody();
+        $body = (string) $response->getBody();
 
         $dom = new \DOMDocument();
         @$dom->loadXML($body);
 
-        $filename = (string)$response->getUri();
+        $filename = (string) $response->getUri();
 
         $error = false;
         $messageParts = array();
@@ -39,12 +39,12 @@ 
                 $error = true;
                 $lastError = libxml_get_last_error();
 
-                $messageParts[] = $xsdFile['xsdfilename'] . ' - ' . $xsdFile['xsdfileurl'] . '(last error: ' . str_replace("\n", '', $lastError->message) . ').';
+                $messageParts[] = $xsdFile['xsdfilename'].' - '.$xsdFile['xsdfileurl'].'(last error: '.str_replace("\n", '', $lastError->message).').';
             }
         }
 
         if ($error === true) {
-            $message = 'XML file (' . $filename . ')  does not validate against the following XSD files: ' . implode(', ', $messageParts);
+            $message = 'XML file ('.$filename.')  does not validate against the following XSD files: '.implode(', ', $messageParts);
             throw new ValidationFailedException($message);
         }
     }

src/Rules/Xml/DuplicateIdRule.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
             return;
         }
 
-        $crawler = new Crawler((string)$response->getBody());
+        $crawler = new Crawler((string) $response->getBody());
 
         $idList = $crawler->filterXPath('//*[@id!=""]');
 
@@ -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)).')');
         }
     }
 }

src/Rules/Json/ValidRule.php

Spacing +2 added lines, -2 removed lines

@@ -26,9 +26,9 @@
 
     public function doValidation(ResponseInterface $response)
     {
-        $result = json_decode((string)$response->getBody());
+        $result = json_decode((string) $response->getBody());
         if ($result === null) {
-            throw new ValidationFailedException("The given JSON data can not be validated (last error: '" . $this->json_errors[json_last_error()] . "').");
+            throw new ValidationFailedException("The given JSON data can not be validated (last error: '".$this->json_errors[json_last_error()]."').");
         }
     }
 }

src/Rules/Json/JsonPathExistsRule.php

Spacing +4 added lines, -4 removed lines

@@ -54,7 +54,7 @@ 
 
     public function doValidation(ResponseInterface $response)
     {
-        $body = (string)$response->getBody();
+        $body = (string) $response->getBody();
 
         $json = json_decode($body);
 
@@ -73,17 +73,17 @@ 
 
             if ($jsonValue === false || (is_array($jsonValue) && empty($jsonValue))) {
                 $error = true;
-                $noCorrectJsonPaths[] = $path['pattern'] . ' (JSON Path not found)';
+                $noCorrectJsonPaths[] = $path['pattern'].' (JSON Path not found)';
             }
             if ($this->checkRelation($path['relation'], $path['value'], $count) === false) {
                 $error = true;
-                $noCorrectJsonPaths[] = $path['pattern'] . ' (number of JSONPaths is not correct corresponding to the given relation/value)';
+                $noCorrectJsonPaths[] = $path['pattern'].' (number of JSONPaths is not correct corresponding to the given relation/value)';
             }
         }
 
         if ($error === true) {
             $allNoCorrectJsonPaths = implode('", "', $noCorrectJsonPaths);
-            throw new ValidationFailedException('Disonances with JSON Paths "' . $allNoCorrectJsonPaths . '!');
+            throw new ValidationFailedException('Disonances with JSON Paths "'.$allNoCorrectJsonPaths.'!');
         }
     }
 }

src/Rules/Html/ForeignDomainImageRule.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
 
     protected function doValidation(ResponseInterface $response)
     {
-        $document = new Document((string)$response->getBody());
+        $document = new Document((string) $response->getBody());
         $images = $document->getImages($response->getUri());
 
         $foreignImages = array();
@@ -37,10 +37,10 @@ 
         foreach ($images as $image) {
             /* @var $image Uri */
             if ($currentUri->getHost($this->depth) !== $image->getHost($this->depth)) {
-                $foreignImages[] = (string)$image;
+                $foreignImages[] = (string) $image;
             }
         }
 
-        $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/ClosingHtmlTagRule.php

Spacing +2 added lines, -2 removed lines

@@ -16,8 +16,8 @@
     {
         if ($response instanceof ContentTypeAwareResponse) {
             if (($response->getStatusCode() < 300 || $response->getStatusCode() >= 400) && $response->getContentType() === 'text/html') {
-                if (stripos((string)$response->getBody(), '</html>') === false) {
-                    throw new ValidationFailedException('Closing html tag is missing (document lengh: ' . strlen((string)$response->getBody()) . ').');
+                if (stripos((string) $response->getBody(), '</html>') === false) {
+                    throw new ValidationFailedException('Closing html tag is missing (document lengh: '.strlen((string) $response->getBody()).').');
                 }
             }
         }

src/Rules/Html/InvalidUrlsRule.php

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
      */
     protected function doValidation(ResponseInterface $response)
     {
-        $document = new Document((string)$response->getBody(), false);
+        $document = new Document((string) $response->getBody(), false);
 
         $urls = $document->getDependencies($response->getUri());
 
@@ -27,13 +27,13 @@ 
 
         foreach ($urls as $url) {
 
-            $idnUrl = $url->getScheme() . '://' . idn_to_ascii($url->getHost()) .  $url->getPath();
+            $idnUrl = $url->getScheme().'://'.idn_to_ascii($url->getHost()).$url->getPath();
 
             if (!filter_var($idnUrl, FILTER_VALIDATE_URL)) {
-                $invalidUrls[] = (string)$url;
+                $invalidUrls[] = (string) $url;
             }
         }
 
-        $this->assert(count($invalidUrls) === 0, 'Invalid urls found (' . implode(', ', $invalidUrls) . ').');
+        $this->assert(count($invalidUrls) === 0, 'Invalid urls found ('.implode(', ', $invalidUrls).').');
     }
 }

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.');
     }
 }