consolidation / robo

tests/unit/Task/SemVerTest.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
     public function testSemverParseFileWithWindowsLineEndings()
     {
         $fixturePath = tempnam(sys_get_temp_dir(), 'semver');
-        $semverFile = str_replace("\n", "\r\n", file_get_contents(codecept_data_dir().'.semver'));
+        $semverFile = str_replace("\n", "\r\n", file_get_contents(codecept_data_dir() . '.semver'));
         file_put_contents($fixturePath, $semverFile);
 
         $res = (new \Robo\Task\Development\SemVer($fixturePath))

tests/cli/CollectionCest.php

Spacing +12 added lines, -12 removed lines

@@ -10,7 +10,7 @@ 
 {
     public function _before(CliGuy $I)
     {
-        $I->amInPath(codecept_data_dir().'sandbox');
+        $I->amInPath(codecept_data_dir() . 'sandbox');
     }
 
     public function toRunMultipleTasksViaACollectionBuilder(CliGuy $I)
@@ -122,7 +122,7 @@ 
             ->rollback(
                 $I->taskDeleteDir('j')
             )
-            ->rollbackCode(function () {
+            ->rollbackCode(function() {
                 throw new AbortTasksException('Aborting rollback.');
             })
             ->taskFilesystemStack()
@@ -181,7 +181,7 @@ 
                 ->mkdir('stuff')
             ->taskForEach($processList)
                 ->withBuilder(
-                    function ($builder, $key, $value) {
+                    function($builder, $key, $value) {
                         return $builder
                             ->taskFilesystemStack()
                                 ->touch("stuff/{$value}.txt");
@@ -238,32 +238,32 @@ 
 
     public function toRollbackInCorrectOrder(CliGuy $I)
     {
-        $expected_order = [6,5,4,3,2,1];
+        $expected_order = [6, 5, 4, 3, 2, 1];
         $actual_order = [];
         $collection = $I->collectionBuilder();
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 1;
         });
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 2;
         });
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 3;
         });
         // Add a nested collection with rollbacks.
         $nested_collection = $I->collectionBuilder();
-        $nested_collection->rollbackCode(function () use (&$actual_order) {
+        $nested_collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 4;
         });
-        $nested_collection->rollbackCode(function () use (&$actual_order) {
+        $nested_collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 5;
         });
         $collection->addTask($nested_collection);
 
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 6;
         });
-        $collection->addCode(function () {
+        $collection->addCode(function() {
             return Result::EXITCODE_ERROR;
         });
         $result = $collection->run();
@@ -463,7 +463,7 @@ 
         $collection = $I->getContainer()->get('collection');
 
         $collection->addCode(
-            function () {
+            function() {
                 throw new \RuntimeException('Error');
             }
         );

src/Runner.php

Spacing +5 added lines, -5 removed lines

@@ -80,7 +80,7 @@ 
     public function __construct($roboClass = null, $roboFile = null)
     {
         // set the const as class properties to allow overwriting in child classes
-        $this->roboClass = $roboClass ? $roboClass : self::ROBOCLASS ;
+        $this->roboClass = $roboClass ? $roboClass : self::ROBOCLASS;
         $this->roboFile  = $roboFile ? $roboFile : self::ROBOFILE;
         $this->dir = getcwd();
     }
@@ -279,7 +279,7 @@ 
     {
         /** @var \Robo\ClassDiscovery\RelativeNamespaceDiscovery $discovery */
         $discovery = Robo::service('relativeNamespaceDiscovery');
-        $discovery->setRelativeNamespace($relativeNamespace.'\Commands')
+        $discovery->setRelativeNamespace($relativeNamespace . '\Commands')
             ->setSearchPattern('/.*Commands?\.php$/');
         return $discovery->getClasses();
     }
@@ -460,9 +460,9 @@ 
 
         if (substr($argv[$pos], 0, 12) == '--load-from=') {
             $this->dir = substr($argv[$pos], 12);
-        } elseif (isset($argv[$pos +1])) {
-            $this->dir = $argv[$pos +1];
-            unset($argv[$pos +1]);
+        } elseif (isset($argv[$pos + 1])) {
+            $this->dir = $argv[$pos + 1];
+            unset($argv[$pos + 1]);
         }
         unset($argv[$pos]);
         // Make adjustments if '--load-from' points at a file.

tests/unit/ClassDiscovery/RelativeNamespaceDiscoveryTest.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
     public function testGetClasses()
     {
         $classLoader = new ClassLoader();
-        $classLoader->addPsr4('\\Robo\\PluginTest\\', [realpath(__DIR__.'/../../plugins')]);
+        $classLoader->addPsr4('\\Robo\\PluginTest\\', [realpath(__DIR__ . '/../../plugins')]);
         $service = new RelativeNamespaceDiscovery($classLoader);
         $service->setRelativeNamespace('Robo\Plugin');
         $service->setSearchPattern('/.*Commands?\.php$/');
@@ -26,7 +26,7 @@ 
     public function testGetFile()
     {
         $classLoader = new ClassLoader();
-        $classLoader->addPsr4('\\Robo\\PluginTest\\', [realpath(__DIR__.'/../../plugins')]);
+        $classLoader->addPsr4('\\Robo\\PluginTest\\', [realpath(__DIR__ . '/../../plugins')]);
         $service = new RelativeNamespaceDiscovery($classLoader);
         $service->setRelativeNamespace('Robo\Plugin');
 

tests/_bootstrap.php

Spacing +4 added lines, -4 removed lines

@@ -6,9 +6,9 @@
     'debug' => true,
     'cacheDir' => '/tmp',
     'includePaths' => [
-        __DIR__.'/../src',
-        __DIR__.'/../vendor/symfony/process',
-        __DIR__.'/../vendor/symfony/console',
-        __DIR__.'/../vendor/henrikbjorn/lurker/src',
+        __DIR__ . '/../src',
+        __DIR__ . '/../vendor/symfony/process',
+        __DIR__ . '/../vendor/symfony/console',
+        __DIR__ . '/../vendor/henrikbjorn/lurker/src',
     ]
 ]);

tests/unit/Task/GulpTest.php

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()
         );
     }
 }

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;

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/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());
     }
 }