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/functional/ExecTest.php

Spacing +3 added lines, -3 removed lines

@@ -64,7 +64,7 @@ 
         // variables are present.
         $task = $this->taskExec('env | wc -l')->interactive(false);
         $result = $task->run();
-        $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
@@ -73,13 +73,13 @@ 
         $result = $task->run();
         $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.
         $task = $this->taskExec('env | wc -l')->interactive(false);
         $task->env('FOO', 'BAR');
         $result = $task->run();
-        $this->assertEquals($start_count + 1, (int) $result->getMessage());
+        $this->assertEquals($start_count + 1, (int)$result->getMessage());
     }
 }

tests/src/Fixtures.php

Doc Comments +1 added lines, -1 removed lines

@@ -42,7 +42,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)
     {

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.
             }
         }