Fix fail of functional test when API limit was reached

Author: stloyd

test/Github/Tests/Functional/CommitTest.php

@@ -2,9 +2,7 @@
 
 namespace Github\Tests\Functional;
 
-use Github\Client;
-
-class CommitTest extends \PHPUnit_Framework_TestCase
+class CommitTest extends TestCase
 {
     /**
      * @test
@@ -15,9 +13,8 @@ public function shouldRetrieveCommitsForRepositoryBranch()
         $repo     = 'php-github-api';
         $branch   = 'master';
 
-        $github = new Client();
-        $commits = $github->api('repo')->commits()->all($username, $repo, array('sha' => $branch));
-        $commit = array_pop($commits);
+        $commits = $this->client->api('repo')->commits()->all($username, $repo, array('sha' => $branch));
+        $commit  = array_pop($commits);
 
         $this->assertArrayHasKey('url', $commit);
         $this->assertArrayHasKey('committer', $commit);
@@ -34,8 +31,7 @@ public function shouldRetrieveCommitBySha()
         $username = 'KnpLabs';
         $repo     = 'php-github-api';
 
-        $github = new Client();
-        $commit = $github->api('repo')->commits()->show($username, $repo, '6df3adf5bd16745299c6429e163265daed430fa1');
+        $commit = $this->client->api('repo')->commits()->show($username, $repo, '6df3adf5bd16745299c6429e163265daed430fa1');
 
         $this->assertArrayHasKey('url', $commit);
         $this->assertArrayHasKey('committer', $commit);
@@ -54,8 +50,7 @@ public function shouldRetrieveCommitsForFile()
         $repo     = 'php-github-api';
         $branch   = 'master';
 
-        $github = new Client();
-        $commits = $github->api('repo')->commits()->all($username, $repo, array('sha' => $branch, 'path' => 'composer.json'));
+        $commits = $this->client->api('repo')->commits()->all($username, $repo, array('sha' => $branch, 'path' => 'composer.json'));
         $commit = array_pop($commits);
 
         $this->assertArrayHasKey('url', $commit);

test/Github/Tests/Functional/GistTest.php

@@ -2,17 +2,14 @@
 
 namespace Github\Tests\Functional;
 
-use Github\Client;
-
-class GistTest extends \PHPUnit_Framework_TestCase
+class GistTest extends TestCase
 {
     /**
      * @test
      */
     public function shouldRetrievePublicGistsListWhenCalledAnonymously()
     {
-        $github = new Client();
-        $gists = $github->api('gists')->all();
+        $gists = $this->client->api('gists')->all();
         $gist = array_pop($gists);
 
         $this->assertArrayHasKey('url', $gist);
@@ -29,17 +26,15 @@ public function shouldRetrievePublicGistsListWhenCalledAnonymously()
      */
     public function shouldNotGetStarredListWithoutAuthorization()
     {
-        $github = new Client();
-        $github->api('gists')->all('starred');
+        $this->client->api('gists')->all('starred');
     }
 
     /**
      * @test
      */
     public function shouldRetrievePublicGistsList()
     {
-        $github = new Client();
-        $gists = $github->api('gists')->all('public');
+        $gists = $this->client->api('gists')->all('public');
         $gist = array_pop($gists);
 
         $this->assertArrayHasKey('url', $gist);
@@ -57,8 +52,7 @@ public function shouldRetrieveGistById()
     {
         $id = 1;
 
-        $github = new Client();
-        $gist = $github->api('gists')->show($id);
+        $gist = $this->client->api('gists')->show($id);
 
         $this->assertArrayHasKey('url', $gist);
         $this->assertArrayHasKey('files', $gist);

test/Github/Tests/Functional/MarkdownTest.php

@@ -2,17 +2,14 @@
 
 namespace Github\Tests\Functional;
 
-use Github\Client;
-
-class MarkdownTest extends \PHPUnit_Framework_TestCase
+class MarkdownTest extends TestCase
 {
     /**
      * @test
      */
     public function shouldRetrieveParsedMarkdownContent()
     {
-        $api = new Client();
-        $api = $api->api('markdown');
+        $api = $this->client->api('markdown');
 
         $input  = 'Hello world github/linguist#1 **cool**, and #1!';
         $output = '<p>Hello world github/linguist#1 <strong>cool</strong>, and #1!</p>';

test/Github/Tests/Functional/RepoTest.php

@@ -2,9 +2,7 @@
 
 namespace Github\Tests\Functional;
 
-use Github\Client;
-
-class RepoTest extends \PHPUnit_Framework_TestCase
+class RepoTest extends TestCase
 {
     /**
      * @test
@@ -14,8 +12,7 @@ public function shouldRetrieveContributorsList()
         $username = 'KnpLabs';
         $repo     = 'php-github-api';
 
-        $github = new Client();
-        $contributors = $github->api('repo')->contributors($username, $repo);
+        $contributors = $this->client->api('repo')->contributors($username, $repo);
         $contributor = array_pop($contributors);
 
         $this->assertArrayHasKey('url', $contributor);
@@ -34,8 +31,7 @@ public function shouldShowRepo()
         $username = 'KnpLabs';
         $repo     = 'php-github-api';
 
-        $github = new Client();
-        $repo = $github->api('repo')->show($username, $repo);
+        $repo = $this->client->api('repo')->show($username, $repo);
 
         $this->assertArrayHasKey('id', $repo);
         $this->assertArrayHasKey('name', $repo);

test/Github/Tests/Functional/TestCase.php

@@ -0,0 +1,24 @@
+<?php
+
+namespace Github\Tests\Functional;
+
+use Github\Client;
+use Github\Exception\ApiLimitExceedException;
+
+class TestCase
+{
+    protected $client;
+
+    public function setUp()
+    {
+        $client = new Client();
+
+        try {
+            $client->api('current_user')->show();
+        } catch (ApiLimitExceedException $e) {
+            $this->markTestSkipped('API limit reached. Skipping to prevent unnecessary failure.');
+        }
+
+        $this->client = $client;
+    }
+}

test/Github/Tests/Functional/UserTest.php

@@ -2,9 +2,7 @@
 
 namespace Github\Tests\Functional;
 
-use Github\Client;
-
-class UsetTest extends \PHPUnit_Framework_TestCase
+class UsetTest extends TestCase
 {
     /**
      * @test
@@ -13,8 +11,7 @@ public function shouldShowUserData()
     {
         $username = 'KnpLabs';
 
-        $github = new Client();
-        $user = $github->api('user')->show($username);
+        $user = $this->client->api('user')->show($username);
 
         $this->assertArrayHasKey('id', $user);
         $this->assertEquals('KnpLabs', $user['login']);
@@ -36,8 +33,7 @@ public function shouldShowUserData()
      */
     public function shouldNotUpdateUserWithoutAuthorization()
     {
-        $github = new Client();
-        $github->api('current_user')->update(array('email' => '[email protected]'));
+        $this->client->api('current_user')->update(array('email' => '[email protected]'));
     }
 
     /**
@@ -47,8 +43,7 @@ public function shouldGetUsersWhoUserIsFollowing()
     {
         $username = 'l3l0';
 
-        $github = new Client();
-        $users = $github->api('user')->following($username);
+        $users = $this->client->api('user')->following($username);
         $user = array_pop($users);
 
         $this->assertArrayHasKey('id', $user);
@@ -62,8 +57,12 @@ public function shouldGetFollowersUsers()
     {
         $username = 'KnpLabs';
 
-        $github = new Client();
-        $users = $github->api('user')->followers($username);
+        $this->client = new Client();
+        try {
+            $users = $this->client->api('user')->followers($username);
+        } catch (ApiLimitExceedException $e) {
+            $this->markTestSkipped('API limit reached. Skipping to prevent unnecessary failure.');
+        }
         $user = array_pop($users);
 
         $this->assertArrayHasKey('id', $user);
@@ -76,8 +75,7 @@ public function shouldGetFollowersUsers()
      */
     public function shouldNotFollowUserWithoutAuthorization()
     {
-        $github = new Client();
-        $github->api('current_user')->follow()->follow('KnpLabs');
+        $this->client->api('current_user')->follow()->follow('KnpLabs');
     }
 
     /**
@@ -86,8 +84,7 @@ public function shouldNotFollowUserWithoutAuthorization()
      */
     public function shouldNotUnfollowUserWithoutAuthorization()
     {
-        $github = new Client();
-        $github->api('current_user')->follow()->unfollow('KnpLabs');
+        $this->client->api('current_user')->follow()->unfollow('KnpLabs');
     }
 
     /**
@@ -97,8 +94,7 @@ public function shouldGetReposBeingWatched()
     {
         $username = 'l3l0';
 
-        $github = new Client();
-        $repos = $github->api('user')->watched($username);
+        $repos = $this->client->api('user')->watched($username);
         $repo = array_pop($repos);
 
         $this->assertArrayHasKey('id', $repo);