Tokens: throw InvalidArgumentException if both teamId and accessToAllPackages are set

Author: glaubinix

src/Api/Tokens.php

@@ -20,6 +20,10 @@ public function all()
 
     public function create(array $tokenData)
     {
+        if (isset($tokenData['teamId'], $tokenData['accessToAllPackages'])) {
+            throw new InvalidArgumentException('Only set either "accessToAllPackages" or "teamId"');
+        }
+
         return $this->post('/tokens/', $tokenData);
     }
 

tests/Api/TokensTest.php

@@ -67,6 +67,22 @@ public function testCreate()
         ]));
     }
 
+    public function testCreateTeamIdAndAllAccess()
+    {
+        $this->expectException(\InvalidArgumentException::class);
+        /** @var Tokens&MockObject $api */
+        $api = $this->getApiMock();
+        $api->expects($this->never())
+            ->method('post');
+
+        $api->create([
+            'description' => 'Team Token',
+            'access' => 'read',
+            'teamId' => 1,
+            'accessToAllPackages' => true,
+        ]);
+    }
+
     public function testRemove()
     {
         $expected = [];