Team Access: Update after API Endpoint Change

Author: zanbaldwin

README.md

@@ -15,6 +15,8 @@
             * [Create a New Team](#create-a-new-team)
             * [Show a Team](#show-a-team)
             * [Edit a Team](#edit-a-team)
+            * [Grant All Package Access](#grant-all-package-access)
+            * [Revoke All Package Access](#revoke-all-package-access)
             * [Delete a Team](#delete-a-team)
             * [Add Member to Team (by User ID)](#add-member-to-team-by-user-id)
             * [Remove Member from Team](#remove-member-from-team)
@@ -126,7 +128,7 @@
          * [Validate incoming webhook payloads](#validate-incoming-webhook-payloads)
       * [License](#license)
 
-<!-- Added by: glaubinix, at: Thu  9 Feb 2023 15:40:34 GMT -->
+<!-- Added by: zanbaldwin, at: Wed May 17 20:53:35 CEST 2023 -->
 
 <!--te-->
 
@@ -252,6 +254,20 @@ Edits a team's name and permissions to be applied to team members. Returns the u
 
 The `$canAccessAllPackages` argument defaults to `null`, meaning that the setting is left unchanged.
 
+#### Grant All Package Access
+```php
+$team = $client->teams()->grantAccessToAllPackages($teamId);
+```
+
+Granting a team access to all packages will give this team access to all current and future organization packages which do not have their permissions synchronized.
+
+#### Revoke All Package Access
+```php
+$team = $client->teams()->revokeAccessToAllPackages($teamId);
+```
+
+Revoking a team's access to all packages will not remove access to packages the team can currently access, but will prevent access to new packages and allow revoking individual package access.
+
 #### Delete a Team
 ```php
 $client->teams()->remove($teamId);

src/Api/Teams.php

@@ -18,7 +18,7 @@ public function all()
         return $this->get('/teams/');
     }
 
-    public function create(string $name, TeamPermissions $permissions, bool $canAccessAllPackages = false): array
+    public function create(string $name, TeamPermissions $permissions): array
     {
         $parameters = [
             'name' => $name,
@@ -29,7 +29,6 @@ public function create(string $name, TeamPermissions $permissions, bool $canAcce
                 'canViewVendorCustomers' => (bool) $permissions->canViewVendorCustomers,
                 'canManageVendorCustomers' => (bool) $permissions->canManageVendorCustomers,
             ],
-            'canAccessAllPackages' => $canAccessAllPackages,
         ];
 
         return $this->post('/teams/', $parameters);
@@ -40,7 +39,7 @@ public function show($teamId)
         return $this->get(sprintf('/teams/%s/', $teamId));
     }
 
-    public function edit($teamId, string $name, TeamPermissions $permissions, ?bool $canAccessAllPackages = null): array
+    public function edit($teamId, string $name, TeamPermissions $permissions): array
     {
         $parameters = [
             'name' => $name,
@@ -52,13 +51,20 @@ public function edit($teamId, string $name, TeamPermissions $permissions, ?bool
                 'canManageVendorCustomers' => (bool) $permissions->canManageVendorCustomers,
             ],
         ];
-        if ($canAccessAllPackages !== null) {
-            $parameters['canAccessAllPackages'] = $canAccessAllPackages;
-        }
 
         return $this->put(sprintf('/teams/%s/', $teamId), $parameters);
     }
 
+    public function grantAccessToAllPackages($teamId): array
+    {
+        return $this->put(sprintf('/teams/%s/all-package-access/grant', $teamId));
+    }
+
+    public function revokeAccessToAllPackages($teamId): array
+    {
+        return $this->put(sprintf('/teams/%s/all-package-access/revoke', $teamId));
+    }
+
     public function remove($teamId): array
     {
         return $this->delete(sprintf('/teams/%s/', $teamId));

tests/Api/TeamsTest.php

@@ -106,7 +106,6 @@ public function testCreateTeam(): void
                 'canViewVendorCustomers' => true,
                 'canManageVendorCustomers' => false,
             ],
-            'canAccessAllPackages' => false,
         ];
 
         /** @var Teams&MockObject $api */
@@ -122,7 +121,6 @@ public function testCreateTeam(): void
                     'canViewVendorCustomers' => true,
                     'canManageVendorCustomers' => false,
                 ],
-                'canAccessAllPackages' => false,
             ]))
             ->willReturn($expected);
 
@@ -207,7 +205,6 @@ public function testEditTeam(): void
                 'canViewVendorCustomers' => true,
                 'canManageVendorCustomers' => false,
             ],
-            'canAccessAllPackages' => true,
         ];
 
         /** @var Teams&MockObject $api */
@@ -223,14 +220,61 @@ public function testEditTeam(): void
                     'canViewVendorCustomers' => true,
                     'canManageVendorCustomers' => false,
                 ],
-                'canAccessAllPackages' => true,
             ]))
             ->willReturn($expected);
 
         $permissions = new TeamPermissions;
         $permissions->canEditTeamPackages = true;
         $permissions->canViewVendorCustomers = true;
-        $this->assertSame($expected, $api->edit(123, 'New Team', $permissions, true));
+        $this->assertSame($expected, $api->edit(123, 'New Team', $permissions));
+    }
+
+    public function testTeamGrant(): void
+    {
+        $expected = [
+            'id' => 123,
+            'name' => 'New Team',
+            'permissions' => [
+                'canEditTeamPackages' => true,
+                'canAddPackages' => false,
+                'canCreateSubrepositories' => false,
+                'canViewVendorCustomers' => true,
+                'canManageVendorCustomers' => false,
+            ],
+        ];
+
+        /** @var Teams&MockObject $api */
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('put')
+            ->with($this->equalTo('/teams/123/all-package-access/grant'), $this->equalTo([]))
+            ->willReturn($expected);
+
+        $this->assertSame($expected, $api->grantAccessToAllPackages(123));
+    }
+
+    public function testTeamRevoke(): void
+    {
+        $expected = [
+            'id' => 123,
+            'name' => 'New Team',
+            'permissions' => [
+                'canEditTeamPackages' => true,
+                'canAddPackages' => false,
+                'canCreateSubrepositories' => false,
+                'canViewVendorCustomers' => true,
+                'canManageVendorCustomers' => false,
+            ],
+        ];
+
+        /** @var Teams&MockObject $api */
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('put')
+            ->with($this->equalTo('/teams/123/all-package-access/revoke'), $this->equalTo([]))
+            ->willReturn($expected);
+
+        $this->assertSame($expected, $api->revokeAccessToAllPackages(123));
     }
 
     public function testDeleteTeam(): void