consolidation / robo

src/ClassDiscovery/RelativeNamespaceDiscovery.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
         $relativePath = $this->convertNamespaceToPath($this->relativeNamespace);
 
         foreach ($this->classLoader->getPrefixesPsr4() as $baseNamespace => $directories) {
-            $directories = array_filter(array_map(function ($directory) use ($relativePath) {
+            $directories = array_filter(array_map(function($directory) use ($relativePath) {
                 return $directory . $relativePath;
             }, $directories), 'is_dir');
 

src/Collection/CollectionBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -309,7 +309,7 @@
 
     /**
      * @param string $functionName
-     * @param int|string $stateKey
+     * @param string $stateKey
      *
      * @return $this
      */

tests/integration/GenerateMarkdownDocTest.php

Spacing +7 added lines, -7 removed lines

@@ -33,13 +33,13 @@ 
 
         $collection = $this->collectionBuilder();
         $taskGenerator = $collection->taskGenDoc("TestedRoboTask.md");
-        $taskGenerator->filterClasses(function (\ReflectionClass $r) {
+        $taskGenerator->filterClasses(function(\ReflectionClass $r) {
             return !($r->isAbstract() || $r->isTrait()) && $r->implementsInterface('Robo\Contract\TaskInterface');
         })->prepend("# TestedRoboTask Tasks");
         $taskGenerator->docClass('TestedRoboTask');
 
         $taskGenerator->filterMethods(
-            function (\ReflectionMethod $m) {
+            function(\ReflectionMethod $m) {
                 if ($m->isConstructor() || $m->isDestructor() || $m->isStatic()) {
                     return false;
                 }
@@ -65,21 +65,21 @@ 
                 return !in_array($m->name, $undocumentedMethods) && $m->isPublic(); // methods are not documented
             }
         )->processClassSignature(
-            function ($c) {
+            function($c) {
                 return "## " . preg_replace('~Task$~', '', $c->getShortName()) . "\n";
             }
         )->processClassDocBlock(
-            function (\ReflectionClass $c, $doc) {
+            function(\ReflectionClass $c, $doc) {
                 $doc = preg_replace('~@method .*?(.*?)\)~', '* `$1)` ', $doc);
-                $doc = str_replace('\\'.$c->name, '', $doc);
+                $doc = str_replace('\\' . $c->name, '', $doc);
                 return $doc;
             }
         )->processMethodSignature(
-            function (\ReflectionMethod $m, $text) {
+            function(\ReflectionMethod $m, $text) {
                 return str_replace('#### *public* ', '* `', $text) . '`';
             }
         )->processMethodDocBlock(
-            function (\ReflectionMethod $m, $text) {
+            function(\ReflectionMethod $m, $text) {
 
                 return $text ? ' ' . trim(strtok($text, "\n"), "\n") : '';
             }

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

tests/integration/ForEachTest.php

Spacing +7 added lines, -7 removed lines

@@ -59,8 +59,8 @@ 
         // set iterable in the __constructor
         $result = $this
             ->taskForEach($items)
-            ->withBuilder(function (CollectionBuilder $builder, $key, $value) use (&$actual) {
-                $builder->addCode(function () use ($key, $value, &$actual) {
+            ->withBuilder(function(CollectionBuilder $builder, $key, $value) use (&$actual) {
+                $builder->addCode(function() use ($key, $value, &$actual) {
                     $actual[] = "$key = $value";
 
                     return 0;
@@ -82,7 +82,7 @@ 
         // set iterable with deferTaskConfiguration()
         $result = $this
             ->collectionBuilder()
-            ->addCode(function (Data $data) use ($items) {
+            ->addCode(function(Data $data) use ($items) {
                 $data['items'] = $items;
 
                 return 0;
@@ -91,8 +91,8 @@ 
                 $this
                     ->taskForEach()
                     ->deferTaskConfiguration('setIterable', 'items')
-                    ->withBuilder(function (CollectionBuilder $builder, $key, $value) use (&$actual) {
-                        $builder->addCode(function () use ($key, $value, &$actual) {
+                    ->withBuilder(function(CollectionBuilder $builder, $key, $value) use (&$actual) {
+                        $builder->addCode(function() use ($key, $value, &$actual) {
                             $actual[] = "$key = $value";
 
                             return 0;
@@ -111,8 +111,8 @@ 
         // call the __constructor() without argument
         $result = $this
             ->taskForEach()
-            ->withBuilder(function (CollectionBuilder $builder, $key, $value) use (&$actual) {
-                $builder->addCode(function () use ($key, $value, &$actual) {
+            ->withBuilder(function(CollectionBuilder $builder, $key, $value) use (&$actual) {
+                $builder->addCode(function() use ($key, $value, &$actual) {
                     $actual++;
 
                     return 0;

tests/integration/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

tests/integration/WriteFileTest.php

Indentation +17 added lines, -17 removed lines

@@ -27,10 +27,10 @@ 
     {
         // write lines with WriteToFile task
         $result = $this->taskWriteToFile('blogpost.md')
-           ->line('****')
-           ->line('hello world')
-           ->line('****')
-           ->run();
+            ->line('****')
+            ->line('hello world')
+            ->line('****')
+            ->run();
         $this->assertTrue($result->wasSuccessful(), $result->getMessage());
         $this->assertFileExists('blogpost.md');
         $contents = file_get_contents('blogpost.md');
@@ -46,9 +46,9 @@ 
     public function appendToFile()
     {
         $result = $this->taskWriteToFile('a.txt')
-           ->append()
-           ->line('hello world')
-           ->run();
+            ->append()
+            ->line('hello world')
+            ->run();
         $this->assertTrue($result->wasSuccessful(), $result->getMessage());
         $this->assertFileExists('a.txt');
         $contents = file_get_contents('a.txt');
@@ -62,7 +62,7 @@ 
     public function testWouldChange()
     {
         $writeTask = $this->taskWriteToFile('a.txt')
-           ->append();
+            ->append();
         $this->assertEquals(false, $writeTask->wouldChange(), "No changes to test file.");
         $writeTask->line('hello world');
         $this->assertEquals(true, $writeTask->wouldChange(), "Test file would change.");
@@ -90,15 +90,15 @@ 
     {
         // append lines with WriteToFile task, but only if pattern does not match
         $result = $this->taskWriteToFile('blogpost.md')
-           ->line('****')
-           ->line('hello world')
-           ->line('****')
-           ->appendUnlessMatches('/hello/', 'Should not add this')
-           ->appendUnlessMatches('/goodbye/', 'Should add this')
-           ->appendIfMatches('/hello/', ' and should also add this')
-           ->appendIfMatches('/goodbye/', ' but should not add this')
-           ->appendIfMatches('/should/', '!')
-           ->run();
+            ->line('****')
+            ->line('hello world')
+            ->line('****')
+            ->appendUnlessMatches('/hello/', 'Should not add this')
+            ->appendUnlessMatches('/goodbye/', 'Should add this')
+            ->appendIfMatches('/hello/', ' and should also add this')
+            ->appendIfMatches('/goodbye/', ' but should not add this')
+            ->appendIfMatches('/should/', '!')
+            ->run();
         $this->assertTrue($result->wasSuccessful(), $result->getMessage());
         $this->assertFileExists('blogpost.md');
         $contents = file_get_contents('blogpost.md');

tests/integration/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/integration/GenerateTaskTest.php

Indentation +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
         $result = $this->taskGenTask('Symfony\Component\Filesystem\Filesystem', 'FilesystemStack')->run();
         $this->assertTrue($result->wasSuccessful(), $result->getMessage());
         $this->assertContains(
-          'protected function _chgrp($files, $group, $recursive = false)',
-          $result->getMessage());
+            'protected function _chgrp($files, $group, $recursive = false)',
+            $result->getMessage());
     }
 
 }