consolidation / robo

tests/src/Traits/TestTasksTrait.php

Doc Comments +16 added lines

@@ -17,6 +17,9 @@ 
     protected $capturedOutput;
     protected $logger;
 
+    /**
+     * @return \League\Container\ContainerInterface
+     */
     public function initTestTasksTrait($commandClass = null, $container = null, $input = null)
     {
         if (!$container) {
@@ -47,6 +50,9 @@ 
         return $container;
     }
 
+    /**
+     * @return null|OutputInterface
+     */
     public function capturedOutputStream()
     {
         if (!$this->testPrinter) {
@@ -66,6 +72,9 @@ 
         return $this->capturedOutput;
     }
 
+    /**
+     * @param string $value
+     */
     public function assertOutputContains($value)
     {
         $output = $this->accumulate();
@@ -73,6 +82,9 @@ 
         $this->assertContains($value, $output);
     }
 
+    /**
+     * @param string $value
+     */
     public function assertOutputNotContains($value)
     {
         $output = $this->accumulate();
@@ -80,6 +92,9 @@ 
         $this->assertNotContains($value, $output);
     }
 
+    /**
+     * @param string $value
+     */
     public function assertOutputEquals($value)
     {
         $output = $this->accumulate();
@@ -90,6 +105,7 @@ 
     /**
      * Make our output comparisons more platform-agnostic by converting
      * CRLF (Windows) or raw CR (confused output) to a LF (unix/Mac).
+     * @param string $output
      */
     protected function simplify($output)
     {

tests/functional/ExecTest.php

Spacing +3 added lines, -3 removed lines

@@ -64,7 +64,7 @@ 
         // variables are present.
         $task = $this->taskExec('env | wc -l')->interactive(false);
         $result = $task->run();
-        $start_count = (int) $result->getMessage();
+        $start_count = (int)$result->getMessage();
         $this->assertGreaterThan(0, $start_count);
 
         // Verify that we get the same amount of environment variables with
@@ -73,13 +73,13 @@ 
         $result = $task->run();
         $this->assertEquals(
             $start_count,
-            (int) $result->getMessage());
+            (int)$result->getMessage());
 
         // Now run the same command, but this time add another environment
         // variable, and see if our count increases by one.
         $task = $this->taskExec('env | wc -l')->interactive(false);
         $task->env('FOO', 'BAR');
         $result = $task->run();
-        $this->assertEquals($start_count + 1, (int) $result->getMessage());
+        $this->assertEquals($start_count + 1, (int)$result->getMessage());
     }
 }

tests/src/Fixtures.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
      */
     public function mktmpdir($basedir = false)
     {
-        $tempfile = tempnam($basedir ?: $this->testDir ?: sys_get_temp_dir(),'robo-tests');
+        $tempfile = tempnam($basedir ?: $this->testDir ?: sys_get_temp_dir(), 'robo-tests');
         unlink($tempfile);
         mkdir($tempfile);
         $this->tmpDirs[] = $tempfile;

Braces +1 added lines, -2 removed lines

@@ -27,8 +27,7 @@
         foreach ($this->tmpDirs as $tmpDir) {
             try {
                 $fs->remove($tmpDir);
-            }
-            catch (\Exception $e) {
+            } catch (\Exception $e) {
                 // Ignore problems with removing fixtures.
             }
         }

Doc Comments +4 added lines, -1 removed lines

@@ -39,7 +39,7 @@ 
      * Create a new temporary directory.
      *
      * @param string|bool $basedir Where to store the temporary directory
-     * @return type
+     * @return string
      */
     public function mktmpdir($basedir = false)
     {
@@ -61,6 +61,9 @@ 
         return $targetSandbox;
     }
 
+    /**
+     * @param string $filename
+     */
     public function dataFile($filename)
     {
         return $this->fixturesDir() . '/' . $filename;

tests/functional/AssetsTest.php

Indentation +14 added lines, -14 removed lines

@@ -25,28 +25,28 @@
     public function testCssMinification()
     {
         if (version_compare(PHP_VERSION, '7.2.0') >= 0) {
-          $this->markTestSkipped('natxet/cssmin uses deprecated "each()" function (PHP 7.2+)');
+            $this->markTestSkipped('natxet/cssmin uses deprecated "each()" function (PHP 7.2+)');
         }
 
         $this->fixtures->createAndCdToSandbox();
 
-				$sampleCss = $this->fixtures->dataFile('sample.css');
-				$outputCss = 'minifiedSample.css';
+                $sampleCss = $this->fixtures->dataFile('sample.css');
+                $outputCss = 'minifiedSample.css';
 
-				$initialFileSize = filesize($sampleCss);
+                $initialFileSize = filesize($sampleCss);
 
-				$this->taskMinify($sampleCss)
-				  ->to('minifiedSample.css')
-				  ->run();
+                $this->taskMinify($sampleCss)
+                    ->to('minifiedSample.css')
+                    ->run();
 
-				$this->assertFileExists($outputCss);
-				$minifiedFileSize = filesize($outputCss);
-				$outputCssContents = file_get_contents($outputCss);
+                $this->assertFileExists($outputCss);
+                $minifiedFileSize = filesize($outputCss);
+                $outputCssContents = file_get_contents($outputCss);
 
-				$this->assertLessThan($initialFileSize, $minifiedFileSize, 'Minified file is smaller than the source file');
-				$this->assertGreaterThan(0, $minifiedFileSize, 'Minified file is not empty');
-				$this->assertContains('body', $outputCssContents, 'Minified file has some content from the source file');
-				$this->assertNotContains('Sample css file', $outputCssContents, 'Minified file does not contain comment from source file');
+                $this->assertLessThan($initialFileSize, $minifiedFileSize, 'Minified file is smaller than the source file');
+                $this->assertGreaterThan(0, $minifiedFileSize, 'Minified file is not empty');
+                $this->assertContains('body', $outputCssContents, 'Minified file has some content from the source file');
+                $this->assertNotContains('Sample css file', $outputCssContents, 'Minified file does not contain comment from source file');
     }
 
 }