Merge pull request #209 from skwashd/memberships-api

Switch from deprecated team member API to team memberships API

Author: pilot

lib/Github/Api/Organization/Teams.php

@@ -60,17 +60,17 @@ public function members($team)
 
     public function check($team, $username)
     {
-        return $this->get('teams/'.rawurlencode($team).'/members/'.rawurlencode($username));
+        return $this->get('teams/'.rawurlencode($team).'/memberships/'.rawurlencode($username));
     }
 
     public function addMember($team, $username)
     {
-        return $this->put('teams/'.rawurlencode($team).'/members/'.rawurlencode($username));
+        return $this->put('teams/'.rawurlencode($team).'/memberships/'.rawurlencode($username));
     }
 
     public function removeMember($team, $username)
     {
-        return $this->delete('teams/'.rawurlencode($team).'/members/'.rawurlencode($username));
+        return $this->delete('teams/'.rawurlencode($team).'/memberships/'.rawurlencode($username));
     }
 
     public function repositories($team)

test/Github/Tests/Api/Organization/TeamsTest.php

@@ -32,7 +32,7 @@ public function shouldCheckIfMemberIsInOrganizationTeam()
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('get')
-            ->with('teams/KnpWorld/members/l3l0')
+            ->with('teams/KnpWorld/memberships/l3l0')
             ->will($this->returnValue($expectedValue));
 
         $this->assertEquals($expectedValue, $api->check('KnpWorld', 'l3l0'));
@@ -96,7 +96,7 @@ public function shouldAddTeamMembers()
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('put')
-            ->with('teams/KnpWorld/members/l3l0')
+            ->with('teams/KnpWorld/memberships/l3l0')
             ->will($this->returnValue($expectedValue));
 
         $this->assertEquals($expectedValue, $api->addMember('KnpWorld', 'l3l0'));
@@ -112,7 +112,7 @@ public function shouldRemoveTeamMembers()
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('delete')
-            ->with('teams/KnpWorld/members/l3l0')
+            ->with('teams/KnpWorld/memberships/l3l0')
             ->will($this->returnValue($expectedValue));
 
         $this->assertEquals($expectedValue, $api->removeMember('KnpWorld', 'l3l0'));