Update package artifacts methods

Author: wissem

README.md

@@ -70,9 +70,9 @@
             * [Delete a package](#delete-a-package)
             * [List all dependents of a package](#list-all-dependents-of-a-package)
             * [List all customers with access to a package](#list-all-customers-with-access-to-a-package)
-            * [Create a package uploaded file](#create-a-package-uploaded-file)
+            * [Create an artifact package file](#create-an-artifact-package-file)
             * [Create an artifact package](#create-an-artifact-package)
-            * [Update an artifact package files](#update-an-artifact-package-files)
+            * [Update artifact files of a package](#update-artifact-files-of-a-package)
          * [Credential](#credential)
             * [List an organization's credentials](#list-an-organizations-credentials)
             * [Show a credential](#show-a-credential)
@@ -98,7 +98,7 @@
          * [Validate incoming webhook payloads](#validate-incoming-webhook-payloads)
       * [License](#license)
 
-<!-- Added by: wissem, at: Tue Jul 14 11:42:54 CEST 2020 -->
+<!-- Added by: wissem, at: Tue Jul 21 10:32:47 CEST 2020 -->
 
 <!--te-->
 
@@ -583,22 +583,22 @@ Returns a list of customers with access to the package.
 ```php
 $fileName = 'package1.zip'; // your package archive artifact containing a valid composer.json in root directory
 $file = file_get_contents($fileName);
-$client->packageArtifact()->create($file, 'application/zip', $fileName);
+$client->packages()->artifacts()->create($file, 'application/zip', $fileName);
 ```
 
 #### Create an artifact package
 
 ```php
 $fileName = 'package1.zip';
 $file = file_get_contents($fileName);
-$response = $client->packageArtifact()->create($file, 'application/zip', $fileName);
+$response = $client->packages()->artifacts()->create($file, 'application/zip', $fileName);
 $artifactId = $response['id'];
 $client->packages()->createArtifactPackage([$artifactId]);
 ```
-#### Update an artifact package files
+#### Update artifact files of a package
 
 ```php
-$result = $client->packages()->showArtifacts('acme-website/package'); // get artifact files details for a package
+$result = $client->packages()->packages()->artifacts()->showPackageArtifacts('acme-website/package'); // get artifact files details for a package
 $artifactFileIds = [42, 43];
 $client->packages()->editArtifactPackage('acme-website/package', $artifactFileIds);
 ```

src/Api/PackageArtifacts.php

@@ -23,4 +23,9 @@ public function show($artifactId)
     {
         return $this->get(sprintf('/packages/artifacts/%s/', $artifactId));
     }
+
+    public function showPackageArtifacts($packageName)
+    {
+        return $this->get(sprintf('/packages/%s/artifacts/', $packageName));
+    }
 }

src/Api/Packages.php

@@ -54,11 +54,6 @@ public function show($packageName)
         return $this->get(sprintf('/packages/%s/', $packageName));
     }
 
-    public function showArtifacts($packageName)
-    {
-        return $this->get(sprintf('/packages/%s/artifacts/', $packageName));
-    }
-
     public function createVcsPackage($url, $credentialId = null, $type = 'vcs')
     {
         return $this->post('/packages/', ['repoType' => $type, 'repoUrl' => $url, 'credentials' => $credentialId]);
@@ -123,4 +118,9 @@ public function listDependents($packageName)
     {
         return $this->get(sprintf('/packages/%s/dependents/', $packageName));
     }
+
+    public function artifacts()
+    {
+        return new PackageArtifacts($this->client, $this->client->getResponseMediator());
+    }
 }

src/Client.php

@@ -98,14 +98,14 @@ public function mirroredRepositories()
         return new Api\MirroredRepositories($this, $this->responseMediator);
     }
 
-    public function packageArtifact()
+    public function getHttpClient()
     {
-        return new Api\PackageArtifacts($this, $this->responseMediator);
+        return $this->getHttpClientBuilder()->getHttpClient();
     }
 
-    public function getHttpClient()
+    public function getResponseMediator()
     {
-        return $this->getHttpClientBuilder()->getHttpClient();
+        return $this->responseMediator;
     }
 
     protected function getHttpClientBuilder()

tests/Api/PackagArtifactsTest.php renamed to tests/Api/PackageArtifactsTest.php

@@ -9,7 +9,7 @@
 
 namespace PrivatePackagist\ApiClient\Api;
 
-class PackagArtifactsTest extends ApiTestCase
+class PackageArtifactsTest extends ApiTestCase
 {
     public function testCreate()
     {
@@ -50,6 +50,24 @@ public function testShow()
         $this->assertSame($expected, $api->show('1'));
     }
 
+    public function testShowPackageArtifacts()
+    {
+        $expected = [
+            'name' => 'acme-website/package',
+            'repoType' => 'artifact',
+            'artifactFiles' => 'artifact',
+        ];
+
+        /** @var PackageArtifacts&\PHPUnit_Framework_MockObject_MockObject $api */
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with($this->equalTo('/packages/acme-website/package/artifacts/'))
+            ->willReturn($expected);
+
+        $this->assertSame($expected, $api->showPackageArtifacts('acme-website/package'));
+    }
+
     /**
      * @return string
      */

tests/Api/PackagesTest.php

@@ -87,24 +87,6 @@ public function testShow()
         $this->assertSame($expected, $api->show('acme-website/package'));
     }
 
-    public function testShowArtifacts()
-    {
-        $expected = [
-            'name' => 'acme-website/package',
-            'repoType' => 'artifact',
-            'artifactIds' => 'artifact',
-        ];
-
-        /** @var Packages&\PHPUnit_Framework_MockObject_MockObject $api */
-        $api = $this->getApiMock();
-        $api->expects($this->once())
-            ->method('get')
-            ->with($this->equalTo('/packages/acme-website/package/artifacts/'))
-            ->willReturn($expected);
-
-        $this->assertSame($expected, $api->showArtifacts('acme-website/package'));
-    }
-
     public function testCreateVcsPackage()
     {
         $expected = [