consolidation / robo

src/Runner.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,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();
     }

src/Application.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     {
         $createRoboFile = new Command('init');
         $createRoboFile->setDescription("Intitalizes basic RoboFile in current dir");
-        $createRoboFile->setCode(function () use ($roboClass, $roboFile) {
+        $createRoboFile->setCode(function() use ($roboClass, $roboFile) {
             $output = Robo::output();
             $output->writeln("<comment>  ~~~ Welcome to Robo! ~~~~ </comment>");
             $output->writeln("<comment>  " . basename($roboFile) . " will be created in the current directory </comment>");

src/Task/Development/GenerateMarkdownDoc.php

Spacing +2 added lines, -2 removed lines

@@ -705,7 +705,7 @@ 
         return implode(
             "\n",
             array_map(
-                function ($line) use ($indent) {
+                function($line) use ($indent) {
                     return substr($line, $indent);
                 },
                 explode("\n", $doc)
@@ -727,7 +727,7 @@ 
         $params = implode(
             ', ',
             array_map(
-                function ($p) {
+                function($p) {
                     return $this->documentParam($p);
                 },
                 $reflectedMethod->getParameters()

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/functional/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/functional/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/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(), $result->getMessage());
 
-				$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/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');