Merge pull request consolidation#36 from boedah/bugfix/issue-35-command-is-stripped

Bugfix/issue 35 command is stripped

Author: DavertMik

src/Task/Shared/CommandStack.php

@@ -27,11 +27,21 @@ public function exec($command)
             $command = implode(' ', array_filter($command));
         }
 
-        $command = $this->executable . ' ' . trim(ltrim($command, $this->executable));
+        $command = $this->executable . ' ' . $this->stripExecutableFromCommand($command);
         array_push($this->exec, trim($command));
         return $this;
     }
 
+    protected function stripExecutableFromCommand($command)
+    {
+        $command = trim($command);
+        $executable = $this->executable . ' ';
+        if (strpos($command, $executable) === 0) {
+            $command = substr($command, strlen($executable));
+        }
+        return $command;
+    }
+
     public function run()
     {
         if (empty($this->exec)) {

tests/unit/CommandStackTest.php

@@ -0,0 +1,27 @@
+<?php
+use Codeception\Util\Stub;
+
+class CommandStackTest extends \Codeception\TestCase\Test
+{
+    public function testExecStackExecutableIsTrimmedFromCommand()
+    {
+        $commandStack = Stub::make('Robo\Task\Shared\CommandStack', array(
+            'executable' => 'some-executable'
+        ));
+        verify($commandStack
+                ->exec('some-executable status')
+                ->getCommand()
+        )->equals('some-executable status');
+    }
+
+    public function testExecStackCommandIsNotTrimmedIfHavingSameCharsAsExecutable()
+    {
+        $commandStack = Stub::make('Robo\Task\Shared\CommandStack', array(
+            'executable' => 'some-executable'
+        ));
+        verify($commandStack
+                ->exec('status')
+                ->getCommand()
+        )->equals('some-executable status');
+    }
+}