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/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/PackExtractTest.php

Indentation +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     public function testPackExtract($archiveType)
     {
         if ((version_compare(PHP_VERSION, '7.4.0') >= 0) && (getenv('TRAVIS'))) {
-          $this->markTestSkipped('Zip libraries apparently not available on Travis CI with PHP 7.4 image.');
+            $this->markTestSkipped('Zip libraries apparently not available on Travis CI with PHP 7.4 image.');
         }
 
         // Archive directory and then extract it again with Archive and Extract tasks

src/Task/File/loadTasks.php

Doc Comments +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 trait loadTasks
 {
     /**
-     * @param array|\Iterator $files
+     * @param string[] $files
      *
      * @return \Robo\Task\File\Concat|\Robo\Collection\CollectionBuilder
      */

src/Task/Filesystem/FlattenDir.php

Doc Comments +2 added lines, -2 removed lines

@@ -136,7 +136,7 @@ 
      * Sets the value from which direction and how much parent dirs should be included.
      * Accepts a positive or negative integer or an array with two integer values.
      *
-     * @param int|int[] $parents
+     * @param integer[] $parents
      *
      * @return $this
      *
@@ -198,7 +198,7 @@ 
     }
 
     /**
-     * @param array $dirs
+     * @param string[] $dirs
      *
      * @return array|\Robo\Result
      *

Spacing +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
      */
     public function dirPermissions($permission)
     {
-        $this->chmod = (int) $permission;
+        $this->chmod = (int)$permission;
 
         return $this;
     }

src/Task/Filesystem/loadTasks.php

Doc Comments +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 trait loadTasks
 {
     /**
-     * @param string|string[] $dirs
+     * @param string[] $dirs
      *
      * @return \Robo\Task\Filesystem\CleanDir|\Robo\Collection\CollectionBuilder
      */

tests/functional/FlattenDirTest.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         $this->fixtures->createAndCdToSandbox();
 
         $result = $this->taskFlattenDir('some/deeply/nested/*.re')
-            ->includeParents([1,1])
+            ->includeParents([1, 1])
             ->parentDir('some')
             ->to('flattened')
             ->run();

tests/functional/AssetsTest.php

Indentation +14 added lines, -14 removed lines

@@ -25,29 +25,29 @@
     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);
 
-				$result = $this->taskMinify($sampleCss)
-				  ->to('minifiedSample.css')
-				  ->run();
+                $result = $this->taskMinify($sampleCss)
+                    ->to('minifiedSample.css')
+                    ->run();
         $this->assertTrue($result->wasSuccessful());
 
-				$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');
     }
 
 }

tests/functional/ExecTest.php

Spacing +3 added lines, -3 removed lines

@@ -68,7 +68,7 @@ 
         $task = $this->taskExec('env | wc -l')->interactive(false);
         $result = $task->run();
         $this->assertTrue($result->wasSuccessful());
-        $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
@@ -78,7 +78,7 @@ 
         $this->assertTrue($result->wasSuccessful());
         $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.
@@ -86,6 +86,6 @@ 
         $task->env('FOO', 'BAR');
         $result = $task->run();
         $this->assertTrue($result->wasSuccessful());
-        $this->assertEquals($start_count + 1, (int) $result->getMessage());
+        $this->assertEquals($start_count + 1, (int)$result->getMessage());
     }
 }