phmLabs / Smoke

src/Rules/Seo/RobotsDisallowAllRule.php

Unused Use Statements -2 removed lines

@@ -4,8 +4,6 @@
 
 use Ivory\HttpAdapter\HttpAdapterInterface;
 use Psr\Http\Message\ResponseInterface;
-use whm\Smoke\Http\ClientAware;
-use whm\Smoke\Http\Response;
 use whm\Smoke\Rules\Rule;
 use whm\Smoke\Rules\ValidationFailedException;
 

Spacing +4 added lines, -4 removed lines

@@ -21,12 +21,12 @@ 
 
     public function validate(ResponseInterface $response)
     {
-        $url = $response->getUri()->getScheme() . '://' . $response->getUri()->getHost();
+        $url = $response->getUri()->getScheme().'://'.$response->getUri()->getHost();
 
         if (substr_count($url, '/') === 2) {
-            $filename = $robotsUrl = $url . '/robots.txt';
+            $filename = $robotsUrl = $url.'/robots.txt';
         } elseif (substr_count($url, '/') === 3) {
-            $filename = $robotsUrl = $url . 'robots.txt';
+            $filename = $robotsUrl = $url.'robots.txt';
         } else {
             return;
         }
@@ -39,7 +39,7 @@ 
 
         $normalizedContent = $this->normalizeContent($content);
 
-        if (strpos($normalizedContent, 'user-agent:* disallow:/' . PHP_EOL) !== false) {
+        if (strpos($normalizedContent, 'user-agent:* disallow:/'.PHP_EOL) !== false) {
             throw new ValidationFailedException('The robots.txt contains disallow all (Disallow: /)');
         }
 

src/Rules/StandardRule.php

Doc Comments +4 added lines

@@ -28,6 +28,10 @@
 
     abstract protected function doValidation(ResponseInterface $response);
 
+    /**
+     * @param boolean $valueToBeTrue
+     * @param string $errorMessage
+     */
     protected function assert($valueToBeTrue, $errorMessage)
     {
         if (!$valueToBeTrue) {

src/Rules/Xml/Sitemap/ValidRule.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace whm\Smoke\Rules\Xml\Sitemap;
 
 use Psr\Http\Message\ResponseInterface;
-use whm\Smoke\Http\Response;
 use whm\Smoke\Rules\StandardRule;
 use whm\Smoke\Rules\ValidationFailedException;
 

src/Extensions/SmokeFilter/FilterExtension.php

Spacing +4 added lines, -4 removed lines

@@ -41,12 +41,12 @@ 
     public function init($filters = array(), $filterFile = '', $exclusive = array())
     {
         if (count($exclusive) > 0 && (count($filters) > 0 || $filterFile !== '')) {
-            throw new \RuntimeException("It's not possible to define filter lists and an exclusive list at the same time [Extension: " . get_class($this) . '].');
+            throw new \RuntimeException("It's not possible to define filter lists and an exclusive list at the same time [Extension: ".get_class($this).'].');
         }
 
         if ($filterFile !== '') {
             if (!file_exists($filterFile)) {
-                throw new \RuntimeException('Filter file not found: ' . $filterFile);
+                throw new \RuntimeException('Filter file not found: '.$filterFile);
             }
 
             $filterElements = EnvAwareYaml::parse(file_get_contents($filterFile));
@@ -85,7 +85,7 @@ 
      */
     public function isFiltered(Event $event, $ruleName, UriAwareResponse $response)
     {
-        $uri = (string)$this->retriever->getOriginUri($response->getUri());
+        $uri = (string) $this->retriever->getOriginUri($response->getUri());
 
         if ($this->currentModus === self::MODUS_FILTER) {
             $isFiltered = $this->isFilteredByFilter($ruleName, $uri);
@@ -104,7 +104,7 @@ 
     private function isFilteredByFilter($ruleName, $uri)
     {
         foreach ($this->filters as $filter) {
-            if ($ruleName === $filter['rule'] && 0 < preg_match('$' . $filter['uri'] . '$', $uri)) {
+            if ($ruleName === $filter['rule'] && 0 < preg_match('$'.$filter['uri'].'$', $uri)) {
                 return true;
             }
         }

src/Extensions/SmokeResponseRetriever/Retriever/ListRetriever/Retriever.php

Spacing +10 added lines, -10 removed lines

@@ -109,20 +109,20 @@ 
                         /* @var \Ivory\HttpAdapter\HttpAdapterException $exception */
 
                         $mainUri = $request->getUri();
-                        $this->redirects[(string)$mainUri->getScheme() . '://' . $mainUri->getHost() . $corruptUrl] = (string)$mainUri;
+                        $this->redirects[(string) $mainUri->getScheme().'://'.$mainUri->getHost().$corruptUrl] = (string) $mainUri;
 
-                        $this->urls[] = ['url' => $mainUri->getScheme() . '://' . $mainUri->getHost() . $corruptUrl, 'system' => $url['system']];
-                        $this->urlStack[] = ['url' => $mainUri->getScheme() . '://' . $mainUri->getHost() . $corruptUrl, 'system' => $url['system']];
+                        $this->urls[] = ['url' => $mainUri->getScheme().'://'.$mainUri->getHost().$corruptUrl, 'system' => $url['system']];
+                        $this->urlStack[] = ['url' => $mainUri->getScheme().'://'.$mainUri->getHost().$corruptUrl, 'system' => $url['system']];
 
                         return $this->next();
                     }
 
                     // the error handling should be done withing the calling class
-                    echo "\n   " . $exception->getMessage() . "\n";
+                    echo "\n   ".$exception->getMessage()."\n";
 
                     return $this->next();
                 } else {
-                    $errorMessages .= $exception->getMessage() . "\n";
+                    $errorMessages .= $exception->getMessage()."\n";
                 }
             }
             if ($errorMessages !== '') {
@@ -135,8 +135,8 @@ 
 
     public function getOriginUri(UriInterface $uri)
     {
-        if (array_key_exists((string)$uri, $this->redirects)) {
-            return $this->urls[$this->redirects[(string)$uri]]['url'];
+        if (array_key_exists((string) $uri, $this->redirects)) {
+            return $this->urls[$this->redirects[(string) $uri]]['url'];
         }
 
         return $uri;
@@ -149,11 +149,11 @@ 
 
     public function getSystem(UriInterface $uri)
     {
-        if (array_key_exists((string)$uri, $this->redirects)) {
-            return $this->urls[$this->redirects[(string)$uri]]['system'];
+        if (array_key_exists((string) $uri, $this->redirects)) {
+            return $this->urls[$this->redirects[(string) $uri]]['system'];
         }
 
-        return $this->urls[(string)$uri]['system'];
+        return $this->urls[(string) $uri]['system'];
     }
 
     public function getSystems()

src/Extensions/SmokeReporter/Reporter/XUnitReporter.php

Spacing +4 added lines, -4 removed lines

@@ -59,7 +59,7 @@ 
         $failures = 0;
 
         if ($this->retriever instanceof CrawlingRetriever) {
-            $startPage = (string)$this->retriever->getStartPage();
+            $startPage = (string) $this->retriever->getStartPage();
         } else {
             $startPage = '';
         }
@@ -80,8 +80,8 @@ 
                     $testCase->addFailure(new Failure($ruleName, $message));
 
                     if ($this->retriever instanceof CrawlingRetriever) {
-                        $stackTrace = $result->getUrl() . ' coming from ' . (string)$this->retriever->getComingFrom($result->getUrl()) . PHP_EOL;
-                        $stackTrace .= '    - ' . $message . " [rule: $ruleName]";
+                        $stackTrace = $result->getUrl().' coming from '.(string) $this->retriever->getComingFrom($result->getUrl()).PHP_EOL;
+                        $stackTrace .= '    - '.$message." [rule: $ruleName]";
                         $testCase->setSystemOut($stackTrace);
                     }
                 }
@@ -92,7 +92,7 @@ 
 
         file_put_contents($this->filename, $xUnitReport->toXml());
 
-        $this->output->writeln('    <info>Writing XUnit Output to file:</info> ' . $this->filename);
+        $this->output->writeln('    <info>Writing XUnit Output to file:</info> '.$this->filename);
     }
 
     public function processResults($results)

src/Rules/Xml/Rss/ValidRule.php

Spacing +6 added lines, -6 removed lines

@@ -15,7 +15,7 @@ 
 
     private function getSchema()
     {
-        return __DIR__ . '/' . self::SCHEMA;
+        return __DIR__.'/'.self::SCHEMA;
     }
 
     public function validate(ResponseInterface $response)
@@ -32,17 +32,17 @@ 
             $lastError = libxml_get_last_error();
             if ($lastError) {
                 throw new ValidationFailedException(
-                    'The given xml file is not well formed (last error: ' .
-                    str_replace("\n", '', $lastError->message) . ').');
+                    'The given xml file is not well formed (last error: '.
+                    str_replace("\n", '', $lastError->message).').');
             }
             $valid = @$dom->schemaValidate($this->getSchema());
             if (!$valid) {
                 $lastError = libxml_get_last_error();
                 $lastErrorMessage = str_replace("\n", '', $lastError->message);
                 throw new ValidationFailedException(
-                    'The given xml file did not Validate vs. ' .
-                    $this->getSchema() . ' (last error: ' .
-                    $lastErrorMessage . ').');
+                    'The given xml file did not Validate vs. '.
+                    $this->getSchema().' (last error: '.
+                    $lastErrorMessage.').');
             }
         }
     }

src/Rules/Image/SizeRule.php

Spacing +1 added lines, -1 removed lines

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

src/Rules/Http/Header/GZipRule.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
         if ($response instanceof ContentTypeAwareResponse) {
             if (strpos($response->getContentType(), 'image') === false
                 && strpos($response->getContentType(), 'pdf') === false
-                && strlen((string)$response->getBody()) >= $this->minFileSize
+                && strlen((string) $response->getBody()) >= $this->minFileSize
             ) {
                 if (!$response->hasHeader('Content-Encoding') || $response->getHeader('Content-Encoding')[0] !== 'gzip') {
                     throw new ValidationFailedException('gzip compression not active');