From 8c0ae57065ae6da5c7d00217f948b269615fa75b Mon Sep 17 00:00:00 2001 From: jeremykendall Date: Tue, 23 Dec 2014 14:37:03 -0600 Subject: [PATCH] Adds methods that fetch a Github user's email addresses. --- src/Provider/Github.php | 24 ++++++++++++++++++++++++ test/src/Provider/GithubTest.php | 22 ++++++++++++++++++++++ 2 files changed, 46 insertions(+) diff --git a/src/Provider/Github.php b/src/Provider/Github.php index 009c5d8..5631ab8 100644 --- a/src/Provider/Github.php +++ b/src/Provider/Github.php @@ -28,6 +28,11 @@ class Github extends AbstractProvider return $this->domain.'/api/v3/user?access_token='.$token; } + public function urlUserEmails(\League\OAuth2\Client\Token\AccessToken $token) + { + return 'https://api.github.com/user/emails?access_token='.$token; + } + public function userDetails($response, \League\OAuth2\Client\Token\AccessToken $token) { $user = new User(); @@ -53,13 +58,32 @@ class Github extends AbstractProvider return $response->id; } + public function getUserEmails(\League\OAuth2\Client\Token\AccessToken $token) + { + $response = $this->fetchUserEmails($token); + + return $this->userEmails(json_decode($response), $token); + } + public function userEmail($response, \League\OAuth2\Client\Token\AccessToken $token) { return isset($response->email) && $response->email ? $response->email : null; } + public function userEmails($response, \League\OAuth2\Client\Token\AccessToken $token) + { + return $response; + } + public function userScreenName($response, \League\OAuth2\Client\Token\AccessToken $token) { return $response->name; } + + protected function fetchUserEmails(\League\OAuth2\Client\Token\AccessToken $token) + { + $url = $this->urlUserEmails($token); + + return $this->fetchProviderData($url); + } } diff --git a/test/src/Provider/GithubTest.php b/test/src/Provider/GithubTest.php index 4ca85f9..1e6c0e2 100644 --- a/test/src/Provider/GithubTest.php +++ b/test/src/Provider/GithubTest.php @@ -132,4 +132,26 @@ class GithubTest extends \PHPUnit_Framework_TestCase $this->assertEquals($this->provider->domain.'/login/oauth/access_token', $this->provider->urlAccessToken()); $this->assertEquals($this->provider->domain.'/api/v3/user?access_token=mock_access_token', $this->provider->urlUserDetails($token)); } + + public function testUserEmails() + { + $postResponse = m::mock('Guzzle\Http\Message\Response'); + $postResponse->shouldReceive('getBody')->times(1)->andReturn('access_token=mock_access_token&expires=3600&refresh_token=mock_refresh_token&uid=1'); + + $getResponse = m::mock('Guzzle\Http\Message\Response'); + $getResponse->shouldReceive('getBody')->times(1)->andReturn('[{"email":"mock_email_1","primary":false,"verified":true},{"email":"mock_email_2","primary":false,"verified":true},{"email":"mock_email_3","primary":true,"verified":true}]'); + + $client = m::mock('Guzzle\Service\Client'); + $client->shouldReceive('setBaseUrl')->times(2); + $client->shouldReceive('post->send')->times(1)->andReturn($postResponse); + $client->shouldReceive('get->send')->times(1)->andReturn($getResponse); + $this->provider->setHttpClient($client); + + $token = $this->provider->getAccessToken('authorization_code', ['code' => 'mock_authorization_code']); + $emails = $this->provider->getUserEmails($token); + $this->assertInternalType('array', $emails); + $this->assertCount(3, $emails); + $this->assertEquals('mock_email_3', $emails[2]->email); + $this->assertTrue($emails[2]->primary); + } }