Replace deprecated organization team repository add/remove urls

KnpLabs · Mar 28, 2021 · dfe1bc1 · dfe1bc1
1 parent d343143
commit dfe1bc1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/Github/Api/Organization/Teams.php b/lib/Github/Api/Organization/Teams.php
@@ -149,15 +149,15 @@ public function repository($team, $organization, $repository)
 
     public function addRepository($team, $organization, $repository, $params = [])
     {
-        if (isset($params['permission']) && !in_array($params['permission'], ['pull', 'push', 'admin'])) {
+        if (isset($params['permission']) && !in_array($params['permission'], ['pull', 'push', 'admin', 'maintain', 'triage'])) {
             $params['permission'] = 'pull';
         }
 
-        return $this->put('/teams/'.rawurlencode($team).'/repos/'.rawurlencode($organization).'/'.rawurlencode($repository), $params);
+        return $this->put('/orgs/'.rawurlencode($organization).'/teams/'.rawurlencode($team).'/repos/'.rawurlencode($organization).'/'.rawurlencode($repository), $params);
     }
 
     public function removeRepository($team, $organization, $repository)
     {
-        return $this->delete('/teams/'.rawurlencode($team).'/repos/'.rawurlencode($organization).'/'.rawurlencode($repository));
+        return $this->delete('/orgs/'.rawurlencode($organization).'/teams/'.rawurlencode($team).'/repos/'.rawurlencode($organization).'/'.rawurlencode($repository));
     }
 }
diff --git a/test/Github/Tests/Api/Organization/TeamsTest.php b/test/Github/Tests/Api/Organization/TeamsTest.php
@@ -161,7 +161,7 @@ public function shouldAddTeamRepository()
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('put')
-            ->with('/teams/KnpWorld/repos/l3l0/l3l0Repo')
+            ->with('/orgs/l3l0/teams/KnpWorld/repos/KnpWorld/l3l0Repo')
             ->will($this->returnValue($expectedValue));
 
         $this->assertEquals($expectedValue, $api->addRepository('KnpWorld', 'l3l0', 'l3l0Repo'));
@@ -177,7 +177,7 @@ public function shouldRemoveTeamRepository()
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('delete')
-            ->with('/teams/KnpWorld/repos/l3l0/l3l0Repo')
+            ->with('/orgs/l3l0/teams/KnpWorld/repos/KnpWorld/l3l0Repo')
             ->will($this->returnValue($expectedValue));
 
         $this->assertEquals($expectedValue, $api->removeRepository('KnpWorld', 'l3l0', 'l3l0Repo'));