consolidation / robo

tests/_helpers/SeeInOutputTrait.php

Doc Comments +4 added lines

@@ -11,6 +11,9 @@ 
     protected $capturedOutput;
     protected $logger;
 
+    /**
+     * @param \League\Container\Container $container
+     */
     public function initSeeInOutputTrait($container, $input = null)
     {
         $this->capturedOutput = '';
@@ -65,6 +68,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/unit/Common/ResultDataTest.php

Spacing +2 added lines, -2 removed lines

@@ -33,8 +33,8 @@
         $a = new ResultData(ResultData::EXITCODE_OK, '', ['one' => 'first', 'two' => 'second']);
 
         $to_be_merged = [
-            ['one' => 'ignored',],
-            ['three' => 'new',],
+            ['one' => 'ignored', ],
+            ['three' => 'new', ],
         ];
 
         foreach ($to_be_merged as $mergeThis) {

src/Collection/Collection.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use Robo\Exception\TaskException;
 use Robo\Exception\TaskExitException;
 use Robo\Contract\CommandInterface;
-
 use Robo\Contract\InflectionInterface;
 use Robo\State\StateAwareInterface;
 use Robo\State\StateAwareTrait;

Spacing +5 added lines, -5 removed lines

@@ -135,7 +135,7 @@ 
     {
         $collection = $this;
         $completionRegistrationTask = new CallableTask(
-            function () use ($collection, $completionTask) {
+            function() use ($collection, $completionTask) {
 
                 $collection->registerCompletion($completionTask);
             },
@@ -178,7 +178,7 @@ 
         $context += ['name' => 'Progress'];
         $context += TaskInfo::getTaskContext($this);
         return $this->addCode(
-            function () use ($level, $text, $context) {
+            function() use ($level, $text, $context) {
                 $context += $this->getState()->getData();
                 $this->printTaskOutput($level, $text, $context);
             }
@@ -194,7 +194,7 @@ 
     {
         $collection = $this;
         $rollbackRegistrationTask = new CallableTask(
-            function () use ($collection, $rollbackTask) {
+            function() use ($collection, $rollbackTask) {
                 $collection->registerRollback($rollbackTask);
             },
             $this
@@ -247,7 +247,7 @@ 
         if ($task instanceof StackBasedTask) {
             $task->stopOnFail(false);
         }
-        $ignoreErrorsInTask = function () use ($task) {
+        $ignoreErrorsInTask = function() use ($task) {
             $data = [];
             try {
                 $result = $this->runSubtask($task);
@@ -679,7 +679,7 @@ 
     {
         return $this->defer(
             $task,
-            function ($task, $state) use ($functionName, $stateKey) {
+            function($task, $state) use ($functionName, $stateKey) {
                 $fn = [$task, $functionName];
                 $value = $state[$stateKey];
                 $fn($value);

Doc Comments +4 added lines, -4 removed lines

@@ -212,7 +212,7 @@ 
      *
      * @param string $method
      * @param string $name
-     * @param callable|\Robo\Contract\TaskInterface $task
+     * @param callable $task
      * @param string $nameOfTaskToAdd
      *
      * @return $this
@@ -283,7 +283,7 @@ 
     /**
      * Return the list of task names added to this collection.
      *
-     * @return string[]
+     * @return integer[]
      */
     public function taskNames()
     {
@@ -356,7 +356,7 @@ 
     }
 
     /**
-     * @param int|string $name
+     * @param string $name
      * @param \Robo\Collection\Element $taskGroup
      *
      * @return $this
@@ -648,7 +648,7 @@ 
     }
 
     /**
-     * @param \Robo\Contract\TaskInterface|\Robo\Collection\NestedCollectionInterface|\Robo\Contract\WrappedTaskInterface $task
+     * @param TaskInterface $task
      *
      * @return \Robo\Result
      */

tests/unit/Task/CollectionTest.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use Robo\Robo;
 use Robo\Result;
 use Robo\State\Data;
-use Robo\Task\BaseTask;
 use Robo\Collection\Collection;
 use Robo\Task\ValueProviderTask;
 use Robo\Task\CollectionTestTask;

Spacing +8 added lines, -8 removed lines

@@ -110,11 +110,11 @@ 
             ->completion($completion1)
             ->rollbackCode(function() use($rollback1) { $rollback1->run(); } )
             ->completionCode(function() use($completion1) { $completion1->run(); } )
-            ->addCode(function () { return 42; })
+            ->addCode(function() { return 42; })
             ->progressMessage("not reached")
             ->rollback($rollback2)
             ->completion($completion2)
-            ->addCode(function () { return 13; });
+            ->addCode(function() { return 13; });
 
         $collection->setLogger($this->guy->logger());
 
@@ -137,15 +137,15 @@ 
 
         $result = $collection
             ->addCode(
-                function (Data $state) {
+                function(Data $state) {
                     $state['one'] = 'first';
                 })
             ->addCode(
-                function (Data $state) {
+                function(Data $state) {
                     $state['two'] = 'second';
                 })
             ->addCode(
-                function (Data $state) {
+                function(Data $state) {
                     $state['three'] = "{$state['one']} and {$state['two']}";
                 })
             ->run();
@@ -169,7 +169,7 @@ 
             ->add($first)
             ->add($second)
             ->addCode(
-                function (Data $state) {
+                function(Data $state) {
                     $state['three'] = "{$state['one']} and {$state['two']}";
                 })
             ->run();
@@ -198,7 +198,7 @@ 
             ->add($third)
                 ->defer(
                     $third,
-                    function ($task, $state) {
+                    function($task, $state) {
                         $task->provideData('three', "{$state['one']} and {$state['two']}");
                     }
                 )
@@ -230,7 +230,7 @@ 
             ->add($third)
                 ->defer(
                     $third,
-                    function ($task, $state) {
+                    function($task, $state) {
                         $task->provideData('three', "{$state['one']} and {$state['two']}");
                     }
                 )

tests/unit/Task/ComposerTest.php

Unused Use Statements -1 removed lines

@@ -1,6 +1,5 @@
 <?php
 use AspectMock\Test as test;
-
 use Robo\Traits\Common\AdjustQuotes;
 
 class ComposerTest extends \Codeception\TestCase\Test

tests/unit/Task/GulpTest.php

Unused Use Statements -1 removed lines

@@ -1,6 +1,5 @@
 <?php
 use AspectMock\Test as test;
-
 use Robo\Traits\Common\AdjustQuotes;
 
 class GulpTest extends \Codeception\TestCase\Test

Spacing +9 added lines, -10 removed lines

@@ -24,32 +24,32 @@ 
     {
         $this->assertEquals(
             $this->adjustQuotes("gulp 'default'"),
-            (new \Robo\Task\Gulp\Run('default','gulp'))->getCommand()
+            (new \Robo\Task\Gulp\Run('default', 'gulp'))->getCommand()
         );
 
         $this->assertEquals(
             $this->adjustQuotes("gulp 'another'"),
-            (new \Robo\Task\Gulp\Run('another','gulp'))->getCommand()
+            (new \Robo\Task\Gulp\Run('another', 'gulp'))->getCommand()
         );
 
         $this->assertEquals(
             $this->adjustQuotes("gulp 'default' --silent"),
-            (new \Robo\Task\Gulp\Run('default','gulp'))->silent()->getCommand()
+            (new \Robo\Task\Gulp\Run('default', 'gulp'))->silent()->getCommand()
         );
 
         $this->assertEquals(
             $this->adjustQuotes("gulp 'default' --no-color"),
-            (new \Robo\Task\Gulp\Run('default','gulp'))->noColor()->getCommand()
+            (new \Robo\Task\Gulp\Run('default', 'gulp'))->noColor()->getCommand()
         );
 
         $this->assertEquals(
             $this->adjustQuotes("gulp 'default' --color"),
-            (new \Robo\Task\Gulp\Run('default','gulp'))->color()->getCommand()
+            (new \Robo\Task\Gulp\Run('default', 'gulp'))->color()->getCommand()
         );
 
         $this->assertEquals(
             $this->adjustQuotes("gulp 'default' --tasks-simple"),
-            (new \Robo\Task\Gulp\Run('default','gulp'))->simple()->getCommand()
+            (new \Robo\Task\Gulp\Run('default', 'gulp'))->simple()->getCommand()
         );
     }
 
@@ -57,7 +57,7 @@ 
     {
         $gulp = test::double('Robo\Task\Gulp\Run', ['executeCommand' => null, 'getConfig' => new \Robo\Config(), 'logger' => new \Psr\Log\NullLogger()]);
 
-        $task = (new \Robo\Task\Gulp\Run('default','gulp'))->simple();
+        $task = (new \Robo\Task\Gulp\Run('default', 'gulp'))->simple();
         $this->assertEquals(
             $this->adjustQuotes("gulp 'default' --tasks-simple"),
             $task->getCommand());
@@ -69,12 +69,11 @@ 
     {
         $isWindows = defined('PHP_WINDOWS_VERSION_MAJOR');
         $expected = $isWindows ?
-            'gulp "anotherWith weired!(\"\') Chars"' :
-            "gulp 'anotherWith weired!(\"'\\'') Chars'";
+            'gulp "anotherWith weired!(\"\') Chars"' : "gulp 'anotherWith weired!(\"'\\'') Chars'";
 
         $this->assertEquals(
             $expected,
-            (new \Robo\Task\Gulp\Run('anotherWith weired!("\') Chars','gulp'))->getCommand()
+            (new \Robo\Task\Gulp\Run('anotherWith weired!("\') Chars', 'gulp'))->getCommand()
         );
     }
 }

src/Task/Composer/RequireDependency.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 
         if (isset($version)) {
             $project = array_map(
-                function ($item) use ($version) {
+                function($item) use ($version) {
                     return "$item:$version";
                 },
                 $project

tests/cli/CopyDirRecursiveExcludeCept.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 $I = new CliGuy($scenario);
 
 $I->wantTo('copy dir recursively with CopyDir task, but exclude a file');
-$I->amInPath(codecept_data_dir().'sandbox');
+$I->amInPath(codecept_data_dir() . 'sandbox');
 $I->seeDirFound('some/deeply/nested');
 $I->seeDirFound('some/deeply/nested2');
 $I->seeDirFound('some/deeply/nested3');

src/Task/Development/SemVer.php

Spacing +1 added lines, -1 removed lines

@@ -235,7 +235,7 @@
         }
 
         $this->version = array_intersect_key($matches, $this->version);
-        $this->version = array_map(function ($item) {
+        $this->version = array_map(function($item) {
             return $item[0];
         }, $this->version);
     }