Completed
Push — master ( b7508b...b025bf )
by Nils
03:16
created
src/Extensions/SmokeReporter/Reporter/KoalamonReporter.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 
93 93
     public function finish()
94 94
     {
95
-        $this->output->writeln('Sending results to ' . $this->server . " ... \n");
95
+        $this->output->writeln('Sending results to '.$this->server." ... \n");
96 96
 
97 97
         if ($this->groupBy === 'prefix') {
98 98
             $this->sendGroupedByPrefix();
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
 
124 124
         foreach ($this->getRuleKeys() as $rule) {
125 125
             foreach ($systems as $system) {
126
-                $identifier = $this->tool . '_' . $this->getPrefix($rule) . '_' . $system;
126
+                $identifier = $this->tool.'_'.$this->getPrefix($rule).'_'.$system;
127 127
                 $failureMessages[$identifier]['message'] = '';
128 128
                 $failureMessages[$identifier]['system'] = $system;
129 129
                 $failureMessages[$identifier]['tool'] = $this->getPrefix($rule);
@@ -140,22 +140,22 @@  discard block
 block discarded – undo
140 140
                         $system = $this->retriever->getSystem(($result->getUrl()));
141 141
                     }
142 142
 
143
-                    $identifer = $this->tool . '_' . $this->getPrefix($ruleLKey) . '_' . $system;
143
+                    $identifer = $this->tool.'_'.$this->getPrefix($ruleLKey).'_'.$system;
144 144
 
145 145
                     if ($failureMessages[$identifer]['message'] === '') {
146
-                        $failureMessages[$identifer]['message'] = 'The ' . $this->getPrefix($ruleLKey) . ' test for #system_name# failed.<ul>';
146
+                        $failureMessages[$identifer]['message'] = 'The '.$this->getPrefix($ruleLKey).' test for #system_name# failed.<ul>';
147 147
                     }
148 148
                     ++$counter[$identifer];
149
-                    $failureMessages[$identifer]['message'] .= '<li>' . $message . '<br>url: ' . $result->getUrl() . ', coming from: ' . $this->retriever->getComingFrom($result->getUrl()) . '</li>';
149
+                    $failureMessages[$identifer]['message'] .= '<li>'.$message.'<br>url: '.$result->getUrl().', coming from: '.$this->retriever->getComingFrom($result->getUrl()).'</li>';
150 150
                 }
151 151
             }
152 152
         }
153 153
 
154 154
         foreach ($failureMessages as $key => $failureMessage) {
155 155
             if ($failureMessage['message'] !== '') {
156
-                $this->send($this->identifier . '_' . $key, $failureMessage['system'], $failureMessage['message'] . '</ul>', self::STATUS_FAILURE, '', $counter[$key], $failureMessage['tool']);
156
+                $this->send($this->identifier.'_'.$key, $failureMessage['system'], $failureMessage['message'].'</ul>', self::STATUS_FAILURE, '', $counter[$key], $failureMessage['tool']);
157 157
             } else {
158
-                $this->send($this->identifier . '_' . $key, $failureMessage['system'], '', self::STATUS_SUCCESS, '', 0, $failureMessage['tool']);
158
+                $this->send($this->identifier.'_'.$key, $failureMessage['system'], '', self::STATUS_SUCCESS, '', 0, $failureMessage['tool']);
159 159
             }
160 160
         }
161 161
     }
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
             $failedTests = array();
168 168
             if ($result->isFailure()) {
169 169
                 foreach ($result->getMessages() as $ruleLKey => $message) {
170
-                    $identifier = 'smoke_' . $ruleLKey . '_' . $result->getUrl();
170
+                    $identifier = 'smoke_'.$ruleLKey.'_'.$result->getUrl();
171 171
 
172 172
                     if ($this->system === '') {
173 173
                         $system = str_replace('http://', '', $result->getUrl());
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
             }
181 181
             foreach ($rules as $rule) {
182 182
                 if (!in_array($rule, $failedTests, true)) {
183
-                    $identifier = 'smoke_' . $rule . '_' . $result->getUrl();
183
+                    $identifier = 'smoke_'.$rule.'_'.$result->getUrl();
184 184
 
185 185
                     if ($this->systemUseRetriever) {
186 186
                         $system = $this->retriever->getSystem($result->getUrl());
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
                     } else {
190 190
                         $system = $this->system;
191 191
                     }
192
-                    $this->send($identifier, $system, 'smoke_' . $rule . '_' . $result->getUrl(), self::STATUS_SUCCESS, (string) $result->getUrl());
192
+                    $this->send($identifier, $system, 'smoke_'.$rule.'_'.$result->getUrl(), self::STATUS_SUCCESS, (string) $result->getUrl());
193 193
                 }
194 194
             }
195 195
         }
@@ -213,16 +213,16 @@  discard block
 block discarded – undo
213 213
                         $system = $this->retriever->getSystem(new Uri($result->getUrl()));
214 214
                     }
215 215
                     if ($failureMessages[$ruleLKey] === '') {
216
-                        $failureMessages[$ruleLKey]['message'] = '    The smoke test for #system_name# failed (Rule: ' . $ruleLKey . ').<ul>';
216
+                        $failureMessages[$ruleLKey]['message'] = '    The smoke test for #system_name# failed (Rule: '.$ruleLKey.').<ul>';
217 217
                     }
218 218
                     ++$counter[$ruleLKey];
219 219
 
220 220
                     $comingFrom = '';
221 221
                     if ($this->retriever->getComingFrom($result->getUrl())) {
222
-                        $comingFrom = ', coming from: ' . $this->retriever->getComingFrom($result->getUrl());
222
+                        $comingFrom = ', coming from: '.$this->retriever->getComingFrom($result->getUrl());
223 223
                     }
224 224
 
225
-                    $failureMessages[$ruleLKey]['message'] .= '<li>' . $message . ' (url: ' . $result->getUrl() . $comingFrom . ')</li > ';
225
+                    $failureMessages[$ruleLKey]['message'] .= '<li>'.$message.' (url: '.$result->getUrl().$comingFrom.')</li > ';
226 226
                     $failureMessages[$ruleLKey]['system'] = $system;
227 227
                 }
228 228
             }
@@ -230,9 +230,9 @@  discard block
 block discarded – undo
230 230
 
231 231
         foreach ($failureMessages as $key => $failureMessage) {
232 232
             if ($failureMessage !== '') {
233
-                $this->send($this->identifier . '_' . $key, $this->system, $failureMessage['message'] . ' </ul > ', self::STATUS_FAILURE, '', $counter[$key]);
233
+                $this->send($this->identifier.'_'.$key, $this->system, $failureMessage['message'].' </ul > ', self::STATUS_FAILURE, '', $counter[$key]);
234 234
             } else {
235
-                $this->send($this->identifier . '_' . $key, $this->system, '', self::STATUS_SUCCESS, '', 0);
235
+                $this->send($this->identifier.'_'.$key, $this->system, '', self::STATUS_SUCCESS, '', 0);
236 236
             }
237 237
         }
238 238
     }
Please login to merge, or discard this patch.