Fix for issue #19 plus test case for it.

Author: Slavey Karadzhov

module/Client/src/Client/Service/PathInvokable.php

@@ -22,8 +22,12 @@ public function getAbsolute($path)
             $path = getenv('HOME').substr($path,1);
         }
 
+        /*
+         * A path is relative when it does not start with / and, in the case of a Windows system,
+         * it does not start with x:\ where x is letter from a to z.
+         */
         if (
-            strpos($path, '/')!==0 ||
+            (!$isWindows && strpos($path, '/')!==0) ||
             ($isWindows && !preg_match("/^[a-z]:\\\/i", $path))
         ) { // if we have relative path
             $cwd = $this->getCwd();
@@ -68,4 +72,17 @@ protected function isWindows()
 
         return self::$isWindows;
     }
+
+    /**
+     * Allows explicitly setting if this is a Windows system or not.
+     * @param boolean $isWindows
+     * @return boolean the old state
+     */
+    public function setWindows($isWindows)
+    {
+        $currentState = self::$isWindows;
+        self::$isWindows = $isWindows;
+        return $currentState;
+    }
+
 }

module/Client/tests/ClientTest/Service/PathInvokableTest.php

@@ -0,0 +1,31 @@
+<?php
+namespace ClientTest\Service;
+
+use PHPUnit_Framework_TestCase;
+use Client\Service\PathInvokable;
+
+class PathInvokableTest extends PHPUnit_Framework_TestCase
+{
+    /**
+     * @var PathInvokable
+     */
+    protected $pathService;
+
+    public function setUp()
+    {
+        $this->pathService = new PathInvokable();
+    }
+
+    /**
+     * Tests to see if the absolute path resolver functions correctly on most systems
+     */
+    public function testGetAbsolute()
+    {
+        $this->assertEquals($this->pathService->getAbsolute('/tmp'), '/tmp');
+        $this->assertEquals($this->pathService->getAbsolute('tmp/a/b'), getcwd().'/tmp/a/b');
+
+        $this->pathService->setWindows(true);
+        $this->assertEquals($this->pathService->getAbsolute('E:\\tmp'), 'E:\\tmp');
+        $this->assertEquals($this->pathService->getAbsolute('tmp\\'), getcwd().DIRECTORY_SEPARATOR.'tmp\\');
+    }
+}