-
-
Notifications
You must be signed in to change notification settings - Fork 597
Add Traffic #499
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Traffic #499
Changes from 9 commits
aad9546
f05e626
a229d8a
350b828
d6767ea
ded9d04
8264b32
bf17eab
73f9828
51d5ea1
079e7f9
b2ec158
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,61 @@ | ||
<?php | ||
namespace Github\Api\Repository; | ||
|
||
use Github\Api\AbstractApi; | ||
use Github\Exception\MissingArgumentException; | ||
|
||
/** | ||
* @link https://developer.github.com/v3/repos/traffic/ | ||
* @author Miguel Piedrafita <[email protected]> | ||
*/ | ||
class Traffic extends AbstractApi | ||
{ | ||
/** | ||
* @link https://developer.github.com/v3/repos/traffic/#list-referrers | ||
* | ||
* @param string $owner | ||
* @param string $repository | ||
* | ||
* @return array | ||
*/ | ||
public function referers($owner, $repository) | ||
{ | ||
return $this->get('/repos/'.rawurlencode($owner).'/'.rawurlencode($repository).'/traffic/popular/referrers'); | ||
} | ||
/** | ||
* @link https://developer.github.com/v3/repos/traffic/#list-paths | ||
* | ||
* @param string $owner | ||
* @param string $repository | ||
* | ||
* @return array | ||
*/ | ||
public function paths($owner, $repository) | ||
{ | ||
return $this->get('/repos/'.rawurlencode($owner).'/'.rawurlencode($repository).'/traffic/popular/paths'); | ||
} | ||
/** | ||
* @link https://developer.github.com/v3/repos/traffic/#views | ||
* | ||
* @param string $owner | ||
* @param string $repository | ||
* | ||
* @return array | ||
*/ | ||
public function views($owner, $repository) | ||
{ | ||
return $this->get('/repos/'.rawurlencode($owner).'/'.rawurlencode($repository).'/traffic/views'); | ||
} | ||
/** | ||
* @link https://developer.github.com/v3/repos/traffic/#clones | ||
* | ||
* @param string $owner | ||
* @param string $repository | ||
* | ||
* @return array | ||
*/ | ||
public function clones($owner, $repository) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Missing parameter There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Still not added. |
||
{ | ||
return $this->get('/repos/'.rawurlencode($owner).'/'.rawurlencode($repository).'/traffic/clones'); | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,37 @@ | ||
<?php | ||
|
||
use Github\Tests\Api\TestCase; | ||
|
||
class TrafficTest extends TestCase | ||
{ | ||
// ... | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should be removed |
||
|
||
/** | ||
* @test | ||
*/ | ||
public function shoulddoSomething() | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please update the name of this function There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You should also update the test content |
||
{ | ||
// Create a variable with the "Server response". | ||
$expectedValue = array('comment1'); | ||
|
||
// Get the API mock (see "getApiClass" below). | ||
$api = $this->getApiMock(); | ||
|
||
$api->expects($this->once()) // Expect one call | ||
->method('get') // A GET request | ||
->with('/gists/123/comments/456') // URI should be "/gists/123/comments/456" | ||
->will($this->returnValue($expectedValue)); // Should return the "Server response" | ||
|
||
// Call Comments::show | ||
$result = $api->show(123, 456); | ||
|
||
// Verify that the result is the "Server response" as we expect. | ||
$this->assertEquals($expectedValue, $result); | ||
} | ||
|
||
protected function getApiClass() | ||
{ | ||
// Tell the "getAPIMock" what class to mock. | ||
return \Github\Api\Gist\Comments::class; | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing parameter
per
. See https://developer.github.com/v3/repos/traffic/#views