Adds test to cover Api::getMetaData() when given directory path.

Author: Potherca

src/Api.php

@@ -208,8 +208,20 @@ final public function getMetaData($path)
             $project = sprintf('%s/%s', $this->settings->getVendor(), $this->settings->getPackage());
             $reference = $this->settings->getReference();
 
-            $url = sprintf('%s/repos/%s/contents/%s?ref=%s', self::GITHUB_API_URL, $project, $path, $reference);
-            $htmlUrl = sprintf('%s/%s/blob/%s/%s', self::GITHUB_URL, $project, $reference, $path);
+            $url = sprintf(
+                '%s/repos/%s/contents/%s?ref=%s',
+                self::GITHUB_API_URL,
+                $project,
+                trim($path, '/'),
+                $reference
+            );
+            $htmlUrl = sprintf(
+                '%s/%s/blob/%s/%s',
+                self::GITHUB_URL,
+                $project,
+                $reference,
+                trim($path, '/')
+            );
 
             $metadata = [
                 self::KEY_TYPE => self::KEY_DIRECTORY,

tests/unit-tests/ApiTest.php

@@ -22,6 +22,7 @@ class ApiTest extends \PHPUnit_Framework_TestCase
     ////////////////////////////////// FIXTURES \\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\
     const MOCK_FILE_PATH = '/path/to/mock/file';
     const MOCK_FILE_CONTENTS = 'Mock file contents';
+    const MOCK_FOLDER_PATH = 'a-directory';
 
     /** @var Api */
     private $api;
@@ -221,6 +222,65 @@ final public function testApiShouldAccountForFileNotExistingWhenAskingInfoForFil
         self::assertEquals($expected, $actual);
     }
 
+    /**
+     * @covers ::getMetaData
+     */
+    final public function testApiShouldReturnMetadataForDirectoryWhenGivenPathIsDirectory()
+    {
+        $api = $this->api;
+
+        $mockPackage = 'mockPackage';
+        $mockPath = self::MOCK_FOLDER_PATH;
+        $mockReference = 'mockReference';
+        $mockVendor = 'mockVendor';
+
+        $expectedUrl = sprintf(
+            '%s/repos/%s/%s/contents/%s?ref=%s',
+            $api::GITHUB_API_URL,
+            $mockVendor,
+            $mockPackage,
+            $mockPath,
+            $mockReference
+        );
+
+        $expectedHtmlUrl = sprintf(
+            '%s/%s/%s/blob/%s/%s',
+            $api::GITHUB_URL,
+            $mockVendor,
+            $mockPackage,
+            $mockReference,
+            $mockPath
+        );
+
+        $expected = [
+            'type' => $api::KEY_DIRECTORY,
+            'url' => $expectedUrl,
+            'html_url' => $expectedHtmlUrl,
+            '_links' => Array (
+                'self' => $expectedUrl,
+                'html' => $expectedHtmlUrl,
+            ),
+        ];
+
+
+        $this->prepareMockSettings([
+            'getVendor' => $mockVendor,
+            'getPackage' => $mockPackage,
+            'getReference' => $mockReference,
+        ]);
+
+        $this->prepareMockApi(
+            'show',
+            $api::API_REPO,
+            [$mockVendor, $mockPackage, $mockPath, $mockReference],
+            [0 => null]
+        );
+
+        $actual = $api->getMetaData($mockPath);
+
+        self::assertEquals($expected, $actual);
+    }
+
     /**
      * @covers ::getMetaData
      */