Add more tests for artifact API

Author: wissem

src/Api/Packages.php

@@ -93,7 +93,7 @@ public function editVcsPackage($packageName, $url, $credentialId = null, $type =
 
     public function editArtifactPackage($packageName, array $artifactPackageFileIds)
     {
-        return $this->put(sprintf('/packages/%s/', $packageName), ['artifactIds' => $artifactPackageFileIds, 'repoType' => 'artifact']);
+        return $this->put(sprintf('/packages/%s/', $packageName), ['repoType' => 'artifact', 'artifactIds' => $artifactPackageFileIds]);
     }
 
     /**

tests/Api/PackagesTest.php

@@ -142,6 +142,23 @@ public function testCreateCustomPackage($customJson)
         $this->assertSame($expected, $api->createCustomPackage($customJson));
     }
 
+    public function testCreateArtifactPackage()
+    {
+        $expected = [
+            'id' => 'job-id',
+            'status' => 'queued',
+        ];
+
+        /** @var Packages&\PHPUnit_Framework_MockObject_MockObject $api */
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('post')
+            ->with($this->equalTo('/packages/'), $this->equalTo(['repoType' => 'artifact', 'artifactIds' => [42]]))
+            ->willReturn($expected);
+
+        $this->assertSame($expected, $api->createArtifactPackage([42]));
+    }
+
     public function customProvider()
     {
         return [
@@ -184,6 +201,23 @@ public function testEditCustomPackage()
         $this->assertSame($expected, $api->editCustomPackage('acme-website/package', '{}'));
     }
 
+    public function testEditArtifactPackage()
+    {
+        $expected = [
+            'id' => 'job-id',
+            'status' => 'queued',
+        ];
+
+        /** @var Packages&\PHPUnit_Framework_MockObject_MockObject $api */
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('put')
+            ->with($this->equalTo('/packages/acme-website/package/'), $this->equalTo(['repoType' => 'artifact', 'artifactIds' => [1, 3]]))
+            ->willReturn($expected);
+
+        $this->assertSame($expected, $api->editArtifactPackage('acme-website/package', [1, 3]));
+    }
+
     public function testRemove()
     {
         $expected = [];