phmLabs / Smoke

src/Extensions/SmokeStop/StopExtension.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         if (array_key_exists($name, $this->stopStrategies)) {
             return $this->stopStrategies[$name];
         } else {
-            throw new \RuntimeException("Strategy ('" . $name . "') not found. Available strategies are " . implode(', ', array_keys($this->stopStrategies)));
+            throw new \RuntimeException("Strategy ('".$name."') not found. Available strategies are ".implode(', ', array_keys($this->stopStrategies)));
         }
     }
 }

src/Extensions/SmokeFilter/FilterExtension.php

Spacing +3 added lines, -3 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));
@@ -105,7 +105,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/SmokeReporter/Reporter/StandardCliReporter.php

Spacing +6 added lines, -6 removed lines

@@ -55,11 +55,11 @@ 
             if ($result->isFailure()) {
                 if (array_key_exists($ruleKey, $result->getMessages())) {
                     $messages = $result->getMessages();
-                    $failedUrls[] = (string) $result->getUrl() . ' - ' . $messages[$ruleKey];
+                    $failedUrls[] = (string) $result->getUrl().' - '.$messages[$ruleKey];
                     ++$count;
                 }
                 if ($count > $this->maxResults) {
-                    $failedUrls[] = '... only the first ' . $this->maxResults . ' elements are shown.';
+                    $failedUrls[] = '... only the first '.$this->maxResults.' elements are shown.';
                     break;
                 }
             }
@@ -77,15 +77,15 @@ 
             $failedUrls = $this->getFailedUrls($ruleKey);
 
             if (count($failedUrls) > 0) {
-                $this->output->writeln('  <error> ' . get_class($rule) . ' </error>');
+                $this->output->writeln('  <error> '.get_class($rule).' </error>');
             } else {
-                $this->output->writeln('  <info> ' . get_class($rule) . ' </info>');
+                $this->output->writeln('  <info> '.get_class($rule).' </info>');
             }
 
-            $this->output->writeln('   ' . str_replace("\n", "\n   ", $info['documentation']) . "\n");
+            $this->output->writeln('   '.str_replace("\n", "\n   ", $info['documentation'])."\n");
 
             foreach ($failedUrls as $failedUrl) {
-                $this->output->writeln('   - ' . $failedUrl);
+                $this->output->writeln('   - '.$failedUrl);
             }
 
             $this->output->writeln('');

src/Extensions/SmokeReporter/Reporter/WarmUpLiveReporter.php

Spacing +2 added lines, -2 removed lines

@@ -18,13 +18,13 @@
     public function processResult(Result $result)
     {
         ++$this->urlCount;
-        $this->output->writeln('   ' . $result->getUrl());
+        $this->output->writeln('   '.$result->getUrl());
         $this->output->writeln('');
     }
 
     public function finish()
     {
-        $this->output->writeln('   <comment>Warm up finished. ' . $this->urlCount . ' urls visited.</comment>');
+        $this->output->writeln('   <comment>Warm up finished. '.$this->urlCount.' urls visited.</comment>');
         $this->output->writeln('');
     }
 }

src/Extensions/SmokeReporter/Reporter/XUnitReporter.php

Spacing +3 added lines, -3 removed lines

@@ -82,8 +82,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);
                     }
                 }
@@ -94,6 +94,6 @@ 
 
         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);
     }
 }

src/Extensions/SmokeReporter/Reporter/CliReporter.php

Spacing +3 added lines, -3 removed lines

@@ -30,15 +30,15 @@
 
     protected function renderFailure(Result $result)
     {
-        $this->output->writeln('   <error> ' . $result->getUrl() . ' </error> coming from ' . (string) $this->retriever->getComingFrom($result->getUrl()));
+        $this->output->writeln('   <error> '.$result->getUrl().' </error> coming from '.(string) $this->retriever->getComingFrom($result->getUrl()));
         foreach ($result->getMessages() as $ruleName => $message) {
-            $this->output->writeln('    - ' . $message . " [rule: $ruleName]");
+            $this->output->writeln('    - '.$message." [rule: $ruleName]");
         }
         $this->output->writeln('');
     }
 
     protected function renderSuccess(Result $result)
     {
-        $this->output->writeln('   <info> ' . $result->getUrl() . ' </info> all tests passed');
+        $this->output->writeln('   <info> '.$result->getUrl().' </info> all tests passed');
     }
 }