feat(internal-api-php-client): add ability to add artifacts in JobArtifacts

Author: janvanicek

src/Result/JobArtifacts.php

@@ -35,6 +35,18 @@ public function setDownloaded(array $downloadedFiles): self
         return $this;
     }
 
+    /** @param Result[] $downloadedArtifacts */
+    public function addDownloaded(array $downloadedArtifacts): void
+    {
+        $this->downloaded = array_merge($this->downloaded, $downloadedArtifacts);
+    }
+
+    /** @param Result[] $uploadedArtifacts */
+    public function addUploaded(array $uploadedArtifacts): void
+    {
+        $this->uploaded = array_merge($this->uploaded, $uploadedArtifacts);
+    }
+
     public function jsonSerialize(): array
     {
         return [

tests/Result/ArtifactsTest.php

@@ -51,4 +51,78 @@ public function testSetDownloaded(): void
             ],
         ], $artifacts->jsonSerialize());
     }
+
+    public function testAddUploaded(): void
+    {
+        $artifacts = new JobArtifacts();
+        $artifacts->setUploaded([
+            new Result(100),
+        ]);
+
+        $artifacts->addUploaded([
+            new Result(200),
+            new Result(300),
+        ]);
+
+        self::assertSame([
+            'uploaded' => [
+                ['storageFileId' => '100'],
+                ['storageFileId' => '200'],
+                ['storageFileId' => '300'],
+            ],
+            'downloaded' => [],
+        ], $artifacts->jsonSerialize());
+    }
+
+    public function testAddUploadedToEmpty(): void
+    {
+        $artifacts = new JobArtifacts();
+        $artifacts->addUploaded([
+            new Result(400),
+        ]);
+
+        self::assertSame([
+            'uploaded' => [
+                ['storageFileId' => '400'],
+            ],
+            'downloaded' => [],
+        ], $artifacts->jsonSerialize());
+    }
+
+    public function testAddDownloaded(): void
+    {
+        $artifacts = new JobArtifacts();
+        $artifacts->setDownloaded([
+            new Result(500),
+        ]);
+
+        $artifacts->addDownloaded([
+            new Result(600),
+            new Result(700),
+        ]);
+
+        self::assertSame([
+            'uploaded' => [],
+            'downloaded' => [
+                ['storageFileId' => '500'],
+                ['storageFileId' => '600'],
+                ['storageFileId' => '700'],
+            ],
+        ], $artifacts->jsonSerialize());
+    }
+
+    public function testAddDownloadedToEmpty(): void
+    {
+        $artifacts = new JobArtifacts();
+        $artifacts->addDownloaded([
+            new Result(800),
+        ]);
+
+        self::assertSame([
+            'uploaded' => [],
+            'downloaded' => [
+                ['storageFileId' => '800'],
+            ],
+        ], $artifacts->jsonSerialize());
+    }
 }