Add more shortcuts for API factory method to prevent some typos.

Fix functional tests.

Author: stloyd

lib/Github/Client.php

@@ -88,19 +88,23 @@ public function __construct(ClientInterface $httpClient = null)
     public function api($name)
     {
         switch ($name) {
+            case 'me':
             case 'current_user':
                 $api = new Api\CurrentUser($this);
                 break;
 
+            case 'git':
             case 'git_data':
                 $api = new Api\GitData($this);
                 break;
 
+            case 'gist':
             case 'gists':
                 $api = new Api\Gists($this);
                 break;
 
             case 'issue':
+            case 'issues':
                 $api = new Api\Issue($this);
                 break;
 
@@ -109,18 +113,25 @@ public function api($name)
                 break;
 
             case 'organization':
+            case 'organizations':
                 $api = new Api\Organization($this);
                 break;
 
+            case 'pr':
             case 'pull_request':
+            case 'pull_requests':
                 $api = new Api\PullRequest($this);
                 break;
 
             case 'repo':
+            case 'repos':
+            case 'repository':
+            case 'repositories':
                 $api = new Api\Repo($this);
                 break;
 
             case 'user':
+            case 'users':
                 $api = new Api\User($this);
                 break;
 

test/Github/Tests/ClientTest.php

@@ -58,7 +58,7 @@ public function shouldAuthenticateUsingUrlToken()
     /**
      * @test
      */
-    public function shouldClearHeadersLaizly()
+    public function shouldClearHeadersLazy()
     {
         $client = new Client();
         $client->clearHeaders();
@@ -102,14 +102,33 @@ public function getApiClassesProvider()
     {
         return array(
             array('user', 'Github\Api\User'),
+            array('users', 'Github\Api\User'),
+
+            array('me', 'Github\Api\CurrentUser'),
             array('current_user', 'Github\Api\CurrentUser'),
+
+            array('git', 'Github\Api\GitData'),
             array('git_data', 'Github\Api\GitData'),
+
+            array('gist', 'Github\Api\Gists'),
             array('gists', 'Github\Api\Gists'),
+
             array('issue', 'Github\Api\Issue'),
+            array('issues', 'Github\Api\Issue'),
+
             array('markdown', 'Github\Api\Markdown'),
+
             array('organization', 'Github\Api\Organization'),
+            array('organizations', 'Github\Api\Organization'),
+
             array('repo', 'Github\Api\Repo'),
+            array('repos', 'Github\Api\Repo'),
+            array('repository', 'Github\Api\Repo'),
+            array('repositories', 'Github\Api\Repo'),
+
+            array('pr', 'Github\Api\PullRequest'),
             array('pull_request', 'Github\Api\PullRequest'),
+            array('pull_requests', 'Github\Api\PullRequest'),
         );
     }
 }

test/Github/Tests/Functional/TestCase.php

@@ -4,8 +4,9 @@
 
 use Github\Client;
 use Github\Exception\ApiLimitExceedException;
+use Github\Exception\RuntimeException;
 
-class TestCase
+class TestCase extends \PHPUnit_Framework_TestCase
 {
     protected $client;
 
@@ -17,6 +18,10 @@ public function setUp()
             $client->api('current_user')->show();
         } catch (ApiLimitExceedException $e) {
             $this->markTestSkipped('API limit reached. Skipping to prevent unnecessary failure.');
+        } catch (RuntimeException $e) {
+            if ('Requires authentication' == $e->getMessage()) {
+                $this->markTestSkipped('Test requires authentication. Skipping to prevent unnecessary failure.');
+            }
         }
 
         $this->client = $client;