phmLabs / Smoke

src/Extensions/SmokeMemory/MemoryExtension.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         foreach ($results as $result) {
             /** @var UriAwareResponse $response */
             if ($result->getStatus() == CheckResult::STATUS_FAILURE) {
-                $this->memory[] = (string)$response->getUri();
+                $this->memory[] = (string) $response->getUri();
             }
         }
     }

src/Extensions/SmokeReporter/Reporter/LeankoalaReporter.php

Spacing +14 added lines, -14 removed lines

@@ -126,7 +126,7 @@ 
         foreach ($this->results as $results) {
             foreach ($results as $result) {
                 /* @var CheckResult $result */
-                $tool = 'Smoke' . $result->getRuleName();
+                $tool = 'Smoke'.$result->getRuleName();
                 $checks[$tool][] = $result;
             }
         }
@@ -138,19 +138,19 @@ 
                 continue;
             }
 
-            $message = 'The smoke test for #system_name# failed (Rule: ' . $toolName . ').<ul>';
+            $message = 'The smoke test for #system_name# failed (Rule: '.$toolName.').<ul>';
             $status = Event::STATUS_SUCCESS;
             $failureCount = 0;
-            $identifier = $toolName . '_' . $this->system;
+            $identifier = $toolName.'_'.$this->system;
 
             foreach ($results as $result) {
                 /** @var CheckResult $result */
                 if ($result->getStatus() === CheckResult::STATUS_FAILURE) {
                     $comingFrom = '';
                     if ($this->addComingFrom && $this->retriever->getComingFrom($result->getResponse()->getUri())) {
-                        $comingFrom = ', coming from: ' . $this->retriever->getComingFrom($result->getResponse()->getUri());
+                        $comingFrom = ', coming from: '.$this->retriever->getComingFrom($result->getResponse()->getUri());
                     }
-                    $message .= '<li>' . $result->getMessage() . ' (url: ' . (string)$result->getResponse()->getUri() . $comingFrom . ')</li>';
+                    $message .= '<li>'.$result->getMessage().' (url: '.(string) $result->getResponse()->getUri().$comingFrom.')</li>';
                     ++$failureCount;
                 }
             }
@@ -158,9 +158,9 @@ 
                 $status = Event::STATUS_FAILURE;
                 $message .= '</ul>';
                 $firstResult = array_pop($results);
-                $attributes[] = new Attribute('html-content', (string)$firstResult->getResponse()->getBody(), true);
+                $attributes[] = new Attribute('html-content', (string) $firstResult->getResponse()->getBody(), true);
             } else {
-                $message = 'All checks for system "#system_name#" succeeded [SmokeBasic:' . $toolName . '].';
+                $message = 'All checks for system "#system_name#" succeeded [SmokeBasic:'.$toolName.'].';
             }
 
             $this->send($identifier, $this->system, $message, $status, $failureCount, $this->tool, $this->system, $attributes);
@@ -173,7 +173,7 @@ 
             foreach ($results as $result) {
                 /* @var CheckResult $result */
 
-                $identifier = '_' . $this->getIdentifier($result);
+                $identifier = '_'.$this->getIdentifier($result);
                 $tool = $this->getPrefix($result->getRuleName());
 
                 $component = $this->getComponent($result->getRuleName());
@@ -181,7 +181,7 @@ 
 
                 $attributes = array();
                 if ($result->getStatus() == CheckResult::STATUS_FAILURE) {
-                    $body = (string)$result->getResponse()->getBody();
+                    $body = (string) $result->getResponse()->getBody();
                     if ($body == "") {
                         $attributes[] = new Attribute('html content', '<empty>');
                     } else {
@@ -205,7 +205,7 @@ 
                 $this->send(
                     $identifier,
                     $currentSystem,
-                    $result->getMessage() . ' (url: ' . (string)$result->getResponse()->getUri() . ')',
+                    $result->getMessage().' (url: '.(string) $result->getResponse()->getUri().')',
                     $result->getStatus(),
                     $result->getValue(),
                     $tool,
@@ -219,12 +219,12 @@ 
     private function getIdentifier(CheckResult $result)
     {
         if ($this->useUrlAsIdentifier) {
-            $suffix = '_' . md5((string)$result->getResponse()->getUri());
+            $suffix = '_'.md5((string) $result->getResponse()->getUri());
         } else {
             $suffix = '';
         }
 
-        return $this->tool . '_' . $result->getRuleName() . $suffix;
+        return $this->tool.'_'.$result->getRuleName().$suffix;
     }
 
     private function getPrefix($string)
@@ -255,8 +255,8 @@ 
                 $this->reporter->sendEvent($event);
             } catch (KoalamonException $e) {
                 $this->output->writeln("\n  <error> Error sending result to leankoala. </error>");
-                $this->output->writeln('   Url: ' . $e->getUrl());
-                $this->output->writeln('   Payload: ' . $e->getPayload());
+                $this->output->writeln('   Url: '.$e->getUrl());
+                $this->output->writeln('   Payload: '.$e->getPayload());
                 $this->output->writeln("");
             } catch (\Exception $e) {
                 $this->output->writeln($e->getMessage());

src/Rules/Http/RedirectRule.php

Spacing +6 added lines, -6 removed lines

@@ -31,23 +31,23 @@
             $uri = new Uri($url['url']);
 
             $urlResponses = $client->sendRequests([new Request('GET', $uri)]);
-            $effectiveUrl = (string)$urlResponses[0]->getEffectiveUri();
+            $effectiveUrl = (string) $urlResponses[0]->getEffectiveUri();
 
-            if ($effectiveUrl != (string)$response->getUri()) {
-                $errors[] = 'The url "' . $url['url'] . '" gets redirected to "' . $effectiveUrl . '".';
+            if ($effectiveUrl != (string) $response->getUri()) {
+                $errors[] = 'The url "'.$url['url'].'" gets redirected to "'.$effectiveUrl.'".';
             }
         }
 
         if (count($errors) > 0) {
-            $message = 'Not all given urls do redirect to "' . (string)$response->getUri() . '"".<ul>';
+            $message = 'Not all given urls do redirect to "'.(string) $response->getUri().'"".<ul>';
             foreach ($errors as $error) {
-                $message .= '<li>' . $error . '</li>';
+                $message .= '<li>'.$error.'</li>';
             }
             $message .= "</ul>";
 
             return new CheckResult(CheckResult::STATUS_FAILURE, $message);
         } else {
-            return new CheckResult(CheckResult::STATUS_SUCCESS, 'All given urls redirect to ' . (string)$response->getUri());
+            return new CheckResult(CheckResult::STATUS_SUCCESS, 'All given urls redirect to '.(string) $response->getUri());
         }
     }
 }

src/Rules/Html/ClosingHtmlTagRule.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
         if ($response instanceof ContentTypeAwareResponse) {
 
             // @todo this could be part of the StandardRule class
-            $body = (string)$response->getBody();
+            $body = (string) $response->getBody();
             $body = preg_replace('/[\x00-\x09\x0B\x0C\x0E-\x1F\x7F,\xFF,\x8B]/', '', $body);
 
             if (($response->getStatusCode() < 300 || $response->getStatusCode() >= 400)
@@ -25,7 +25,7 @@ 
                 && strlen($body) > 0
             ) {
                 if (stripos($body, '</html>') === false) {
-                    throw new ValidationFailedException('Closing html tag is missing (document length: ' . strlen($body) . ').');
+                    throw new ValidationFailedException('Closing html tag is missing (document length: '.strlen($body).').');
                 }
             }
         }

src/Rules/Html/InvalidUrlsRule.php

Spacing +4 added lines, -4 removed lines

@@ -18,19 +18,19 @@
      */
     protected function doValidation(ResponseInterface $response)
     {
-        $document = new Document((string)$response->getBody(), false);
+        $document = new Document((string) $response->getBody(), false);
         $urls = $document->getDependencies($response->getUri());
         $invalidUrls = array();
 
         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/InsecureContentRule.php

Spacing +3 added lines, -3 removed lines

@@ -44,7 +44,7 @@ 
             return;
         }
 
-        $htmlDocument = new Document((string)$response->getBody());
+        $htmlDocument = new Document((string) $response->getBody());
 
         $resources = $htmlDocument->getDependencies($uri, false);
 
@@ -54,7 +54,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;
                     }
@@ -68,7 +68,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

Spacing +6 added lines, -6 removed lines

@@ -21,12 +21,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();
@@ -35,8 +35,8 @@ 
             $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) {
@@ -44,11 +44,11 @@ 
                 $lastErrorMessage = str_replace("\n", '', $lastError->message);
 
                 if ($response instanceof UriAwareResponse) {
-                    $toolUrl = sprintf(self::PUBLIC_SERVICE, urlencode((string)$response->getUri()));
+                    $toolUrl = sprintf(self::PUBLIC_SERVICE, urlencode((string) $response->getUri()));
                 } else {
                     $toolUrl = '';
                 }
-                return new CheckResult(CheckResult::STATUS_FAILURE, 'The given xml file is not a valid rss file (last error: ' . $lastErrorMessage . ').', null, $toolUrl);
+                return new CheckResult(CheckResult::STATUS_FAILURE, 'The given xml file is not a valid rss file (last error: '.$lastErrorMessage.').', null, $toolUrl);
             }
         }
         return new CheckResult(CheckResult::STATUS_SUCCESS, 'The given rss file is valid.');