Add missing parameters for User/CurrentUser Repositories (#684)

* Add missing parameters for User/CurrentUser Repositories

* Make some StyleCI fixes to User and CurrentUser classes

* Some unrelated StyleCI fixes... ?

Author: Adam Campbell

lib/Github/Api/CurrentUser.php

@@ -111,18 +111,22 @@ public function teams()
     /**
      * @link http://developer.github.com/v3/repos/#list-your-repositories
      *
-     * @param string $type      role in the repository
-     * @param string $sort      sort by
-     * @param string $direction direction of sort, asc or desc
+     * @param string $type        role in the repository
+     * @param string $sort        sort by
+     * @param string $direction   direction of sort, asc or desc
+     * @param string $visibility  visibility of repository
+     * @param string $affiliation relationship to repository
      *
      * @return array
      */
-    public function repositories($type = 'owner', $sort = 'full_name', $direction = 'asc')
+    public function repositories($type = 'owner', $sort = 'full_name', $direction = 'asc', $visibility = 'all', $affiliation = 'owner,collaborator,organization_member')
     {
         return $this->get('/user/repos', [
             'type' => $type,
             'sort' => $sort,
             'direction' => $direction,
+            'visibility' => $visibility,
+            'affiliation' => $affiliation,
         ]);
     }
 

lib/Github/Api/Repository/Collaborators.php

@@ -70,9 +70,11 @@ public function remove($username, $repository, $collaborator)
 
     /**
      * @link https://developer.github.com/v3/repos/collaborators/#review-a-users-permission-level
+     *
      * @param $username
      * @param $repository
      * @param $collaborator
+     *
      * @return array|string
      */
     public function permission($username, $repository, $collaborator)

lib/Github/Api/User.php

@@ -173,19 +173,23 @@ public function subscriptions($username)
      *
      * @link https://developer.github.com/v3/repos/#list-user-repositories
      *
-     * @param string $username  the username
-     * @param string $type      role in the repository
-     * @param string $sort      sort by
-     * @param string $direction direction of sort, asc or desc
+     * @param string $username    the username
+     * @param string $type        role in the repository
+     * @param string $sort        sort by
+     * @param string $direction   direction of sort, asc or desc
+     * @param string $visibility  visibility of repository
+     * @param string $affiliation relationship to repository
      *
      * @return array list of the user repositories
      */
-    public function repositories($username, $type = 'owner', $sort = 'full_name', $direction = 'asc')
+    public function repositories($username, $type = 'owner', $sort = 'full_name', $direction = 'asc', $visibility = 'all', $affiliation = 'owner,collaborator,organization_member')
     {
         return $this->get('/users/'.rawurlencode($username).'/repos', [
             'type' => $type,
             'sort' => $sort,
             'direction' => $direction,
+            'visibility' => $visibility,
+            'affiliation' => $affiliation,
         ]);
     }
 

test/Github/Tests/Api/Repository/CollaboratorsTest.php

@@ -75,7 +75,7 @@ public function shouldRemoveRepositoryCollaborator()
      */
     public function shouldGetRepositoryCollaboratorPermission()
     {
-        $expectedValue = array(array('permission' => 'admin', 'user' => 'l3l0'));
+        $expectedValue = [['permission' => 'admin', 'user' => 'l3l0']];
 
         $api = $this->getApiMock();
         $api->expects($this->once())

test/Github/Tests/Api/UserTest.php

@@ -194,7 +194,7 @@ public function shouldGetUserRepositories()
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('get')
-            ->with('/users/l3l0/repos', ['type' => 'owner', 'sort' => 'full_name', 'direction' => 'asc'])
+            ->with('/users/l3l0/repos', ['type' => 'owner', 'sort' => 'full_name', 'direction' => 'asc', 'visibility' => 'all', 'affiliation' => 'owner,collaborator,organization_member'])
             ->will($this->returnValue($expectedArray));
 
         $this->assertEquals($expectedArray, $api->repositories('l3l0'));