phmLabs / Smoke

src/Rules/Html/BigFilesRule.php

Spacing +2 added lines, -2 removed lines

@@ -38,9 +38,9 @@
         }
 
         if (count($bigFiles) > 0) {
-            $message = "Some files were found that are too big (max: " . $this->maxElementSize . " KB):<ul>";
+            $message = "Some files were found that are too big (max: ".$this->maxElementSize." KB):<ul>";
             foreach ($bigFiles as $bigFile) {
-                $message .= '<li>File: ' . $bigFile['name'] . ', Size: ' . $bigFile['size'] . ' KB</li>';
+                $message .= '<li>File: '.$bigFile['name'].', Size: '.$bigFile['size'].' KB</li>';
             }
             $message .= "</ul>";
             $result = new CheckResult(CheckResult::STATUS_FAILURE, $message, count($bigFiles));

src/Extensions/SmokeOnFailure/OnFailureExtension.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 
     public function init($command)
     {
-        $this->callback = function () use ($command) {
+        $this->callback = function() use ($command) {
             return eval($command);
         };
     }

src/Rules/Html/ResourcesCountRule.php

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
 
     public function init($maxElements = 200)
     {
-        $this->maxElements = (int)$maxElements;
+        $this->maxElements = (int) $maxElements;
     }
 
     protected function doValidation(ResponseInterface $response)
@@ -28,12 +28,12 @@ 
         if ($response instanceof ResourcesAwareResponse) {
             $resourceCount = count($response->getResources());
             if ($resourceCount > $this->maxElements) {
-                $message = 'Too many resources were loaded. ' . $resourceCount . ' resources loaded, maximum was ' . $this->maxElements . '.';
+                $message = 'Too many resources were loaded. '.$resourceCount.' resources loaded, maximum was '.$this->maxElements.'.';
                 $result = new CheckResult(CheckResult::STATUS_FAILURE, $message, $resourceCount);
                 $result->addAttribute(new Attribute('resources', $response->getResources(), true));
                 return $result;
             } else {
-                $message = $resourceCount . ' resources were loaded.';
+                $message = $resourceCount.' resources were loaded.';
                 return new CheckResult(CheckResult::STATUS_SUCCESS, $message, $resourceCount);
             }
         } else {

src/Rules/Xml/XmlValidXsdRule.php

Spacing +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
 
     protected function doValidation(ResponseInterface $response)
     {
-        $body = (string)$response->getBody();
+        $body = (string) $response->getBody();
 
         $dom = new \DOMDocument();
         @$dom->loadXML($body);
@@ -37,12 +37,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 (' . (string)$response->getUri() . ')  does not validate against the following XSD files: ' . implode(', ', $messageParts);
+            $message = 'XML file ('.(string) $response->getUri().')  does not validate against the following XSD files: '.implode(', ', $messageParts);
             throw new ValidationFailedException($message);
         }
     }

src/Rules/Json/JsonPathExistsRule.php

Spacing +5 added lines, -5 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'], (int)$path['value'], $count) === false) {
+            if ($this->checkRelation($path['relation'], (int) $path['value'], $count) === false) {
                 $error = true;
-                $noCorrectJsonPaths[] = $path['pattern'] . ' (' . $count . ' elements found, expected ' . $path['relation'] . ' ' . $path['value'] . ')';
+                $noCorrectJsonPaths[] = $path['pattern'].' ('.$count.' elements found, expected '.$path['relation'].' '.$path['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/InsecureContentRule.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
             return;
         }
 
-        $htmlDocument = new Document((string)$response->getBody());
+        $htmlDocument = new Document((string) $response->getBody());
 
         $resources = $htmlDocument->getDependencies($uri, false);
 
@@ -46,7 +46,7 @@ 
             if ($resource->getScheme() && 'https' !== $resource->getScheme()) {
                 $excluded = false;
                 foreach ($this->excludedFiles as $excludedFile) {
-                    if (preg_match('*' . $excludedFile . '*', (string)$resource)) {
+                    if (preg_match('*'.$excludedFile.'*', (string) $resource)) {
                         $excluded = true;
                         break;
                     }
@@ -60,7 +60,7 @@ 
         if (count($unsecures) > 0) {
             $message = 'At least one dependency was found on a secure url, that was transfered insecure.<ul>';
             foreach ($unsecures as $unsecure) {
-                $message .= '<li>' . (string)$unsecure . '</li>';
+                $message .= '<li>'.(string) $unsecure.'</li>';
             }
             $message .= '</ul>';
             return new CheckResult(CheckResult::STATUS_FAILURE, $message, count($unsecures));

src/Rules/Xml/Rss/ValidRule.php

Unused Use Statements -1 removed lines

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

Spacing +6 added lines, -6 removed lines

@@ -18,12 +18,12 @@ 
 
     private function getSchema()
     {
-        return __DIR__ . '/' . self::SCHEMA;
+        return __DIR__.'/'.self::SCHEMA;
     }
 
     public function doValidation(ResponseInterface $response)
     {
-        $body = (string)$response->getBody();
+        $body = (string) $response->getBody();
         if (preg_match('/<rss/', $body)) {
             libxml_clear_errors();
             $dom = new \DOMDocument();
@@ -31,16 +31,16 @@ 
             $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 is not a valid rss file (last error: ' .
-                    $lastErrorMessage . ').');
+                    'The given xml file is not a valid rss file (last error: '.
+                    $lastErrorMessage.').');
             }
         }
     }

src/Rules/CheckResult.php

Doc Comments +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
     }
 
     /**
-     * @param array $attributes
+     * @param array $attribute
      */
     public function addAttribute(Attribute $attribute)
     {
@@ -68,7 +68,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getMessage()
     {

src/Rules/Http/HttpsCertificateExpireRule.php

Spacing +4 added lines, -4 removed lines

@@ -31,7 +31,7 @@ 
         } else {
 
             if ($certInfo['validFrom_time_t'] > time() || $certInfo['validTo_time_t'] < time()) {
-                $errorMessage = 'Certificate is expired. [' . $validFrom . ' - ' . $validTo . ']';
+                $errorMessage = 'Certificate is expired. ['.$validFrom.' - '.$validTo.']';
 
                 $result = new CheckResult(CheckResult::STATUS_FAILURE, $errorMessage);
                 $infoJson = json_encode($certInfo);
@@ -41,8 +41,8 @@ 
                     $result->addAttribute(new Attribute('certificate information', $infoJson, true));
                 }
                 return $result;
-            } elseif ($certInfo['validTo_time_t'] < strtotime('+' . $this->expireWarningTime . 'days')) {
-                $errorMessage = 'Certificate warning, expires in less than ' . $this->expireWarningTime . ' days. Certificate expires at: ' . $validTo;
+            } elseif ($certInfo['validTo_time_t'] < strtotime('+'.$this->expireWarningTime.'days')) {
+                $errorMessage = 'Certificate warning, expires in less than '.$this->expireWarningTime.' days. Certificate expires at: '.$validTo;
 
                 $result = new CheckResult(CheckResult::STATUS_FAILURE, $errorMessage);
                 $result->addAttribute(new Attribute('certificate information', json_encode($certInfo), true));
@@ -50,6 +50,6 @@ 
             }
         }
 
-        return new CheckResult(CheckResult::STATUS_SUCCESS, 'The certificate does not expire within the next ' . $this->expireWarningTime . ' days. Expire date: ' . $validTo . '.');
+        return new CheckResult(CheckResult::STATUS_SUCCESS, 'The certificate does not expire within the next '.$this->expireWarningTime.' days. Expire date: '.$validTo.'.');
     }
 }