Completed
Push — output_parsers_refactor ( 755930...b5c5df )
by Alessandro
07:17
created
src/Paraunit/Process/ParaunitProcessAbstract.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -147,6 +147,6 @@
 block discarded – undo
147 147
      */
148 148
     public function isEmpty()
149 149
     {
150
-        return (bool) count($this->filename);
150
+        return (bool)count($this->filename);
151 151
     }
152 152
 }
Please login to merge, or discard this patch.
src/Paraunit/Printer/ProcessPrinter.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@
 block discarded – undo
75 75
             case 'S':
76 76
                 $this->printWithCounter('<skipped>S</skipped>');
77 77
                 break;
78
-           case 'R':
78
+            case 'R':
79 79
                 $this->printWithCounter('<risky>R</risky>');
80 80
                 break;
81 81
             case '.':
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -24,13 +24,13 @@
 block discarded – undo
24 24
     {
25 25
         $process = $processEvent->getProcess();
26 26
 
27
-        if ( ! $processEvent->has('output_interface')) {
27
+        if (!$processEvent->has('output_interface')) {
28 28
             throw new \BadMethodCallException('missing output_interface');
29 29
         }
30 30
 
31 31
         $this->output = $processEvent->get('output_interface');
32 32
 
33
-        if ( ! $this->output instanceof OutputInterface) {
33
+        if (!$this->output instanceof OutputInterface) {
34 34
             throw new \BadMethodCallException('output_interface, unexpected type: ' . get_class($this->output));
35 35
         }
36 36
 
Please login to merge, or discard this patch.
src/Paraunit/Parser/JSONLogFetcher.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@
 block discarded – undo
33 33
     {
34 34
         $filePath = $this->fileName->generate($process);
35 35
 
36
-        if ( ! file_exists($filePath)) {
36
+        if (!file_exists($filePath)) {
37 37
             throw new JSONLogNotFoundException($process);
38 38
         }
39 39
 
Please login to merge, or discard this patch.
src/Paraunit/Parser/ProcessOutputParser.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
     public function onProcessTerminated(ProcessEvent $processEvent)
40 40
     {
41 41
         foreach ($this->parsers as $parser) {
42
-            if ( ! $parser->parsingFoundResult($processEvent->getProcess())) {
42
+            if (!$parser->parsingFoundResult($processEvent->getProcess())) {
43 43
                 return;
44 44
             }
45 45
         }
Please login to merge, or discard this patch.
src/Paraunit/Tests/Stub/PHPUnitOutput/JSONLogs/JSONLogStub.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -25,8 +25,8 @@
 block discarded – undo
25 25
      */
26 26
     public static function getCleanOutputFileContent($filename)
27 27
     {
28
-        $fullFilename =  __DIR__ . DIRECTORY_SEPARATOR . $filename . '.json';
29
-        if ( ! file_exists($fullFilename)) {
28
+        $fullFilename = __DIR__ . DIRECTORY_SEPARATOR . $filename . '.json';
29
+        if (!file_exists($fullFilename)) {
30 30
             throw new \Exception('Unknown file stub: ' . $filename);
31 31
         }
32 32
 
Please login to merge, or discard this patch.
src/Paraunit/Tests/BaseFunctionalTestCase.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -111,7 +111,7 @@
 block discarded – undo
111 111
      */
112 112
     public function getTestWithSegFault()
113 113
     {
114
-        if ( ! extension_loaded('sigsegv')) {
114
+        if (!extension_loaded('sigsegv')) {
115 115
             $this->markTestIncomplete('The segfault cannot be reproduced in this environment');
116 116
         }
117 117
 
Please login to merge, or discard this patch.
src/Paraunit/Tests/Functional/Runner/RunnerTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -134,7 +134,7 @@
 block discarded – undo
134 134
     {
135 135
         $phpunitVersion = new \PHPUnit_Runner_Version();
136 136
 
137
-        if ( ! preg_match('/^5\./', $phpunitVersion->id())) {
137
+        if (!preg_match('/^5\./', $phpunitVersion->id())) {
138 138
             $this->markTestSkipped('PHPUnit < 5 in this env, warnings are not present.');
139 139
         }
140 140
 
Please login to merge, or discard this patch.
src/Paraunit/Parser/JSONLogParser.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace Paraunit\Parser;
4 4
 
5 5
 use Paraunit\Exception\JSONLogNotFoundException;
6
-use Paraunit\Exception\RecoverableTestErrorException;
7 6
 use Paraunit\Lifecycle\ProcessEvent;
8 7
 use Paraunit\Process\ParaunitProcessAbstract;
9 8
 use Paraunit\Process\ProcessResultInterface;
Please login to merge, or discard this patch.
src/Paraunit/Printer/OutputContainer.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
     }
40 40
 
41 41
     /**
42
-     * @return \string[]
42
+     * @return integer[]
43 43
      */
44 44
     public function getFileNames()
45 45
     {
Please login to merge, or discard this patch.