some fix

Author: aynsix

api/config/packages/messenger.yaml

@@ -16,15 +16,6 @@ framework:
             'App\Consumer\Handler\DeleteArchive': async
             'App\Consumer\Handler\CleanOldArchives': async
 
-# when@test:
-#    framework:
-#        messenger:
-#            transports:
-#                # replace with your transport name here (e.g., my_transport: 'in-memory://')
-#                # For more Messenger testing tools, see https://github.com/zenstruck/messenger-test
-#                async: 'in-memory://'
-
-
 when@dev:
     framework:
         messenger:

api/src/Api/Processor/ArchiveProcessor.php

@@ -45,16 +45,27 @@ public function __construct(
 
     public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = []): Archive
     {
-        $isNew = !isset($context[AbstractItemNormalizer::OBJECT_TO_POPULATE]);
 
+        $isNew = false;
+        if ($operation instanceof \ApiPlatform\Metadata\Post) {
+            $isNew = true;
+        }
+       
         $this->validator->validate($data, [
             'groups' => [
                 'Default',
                 $isNew ? 'Post' : 'Patch',
             ],
         ]);
 
-        $object = $context[AbstractItemNormalizer::OBJECT_TO_POPULATE] ?? new Archive();
+        if ($isNew) {
+            $object = new Archive();
+        } else {
+            $object = $this->em->find(Archive::class, $uriVariables['id']);
+            if (null === $object) {
+                throw new BadRequestHttpException('Archive not found');
+            }
+        }
 
         if (null !== $data->getDownloadFilename()) {
             $object->setDownloadFilename($data->getDownloadFilename());

api/src/Consumer/Handler/CleanOldArchives.php

@@ -4,7 +4,7 @@
 
 namespace App\Consumer\Handler;
 
-class CleanOldArchives extends BuildArchive
+class CleanOldArchives
 {
 
 }

api/src/Consumer/Handler/DeleteArchive.php

@@ -4,7 +4,14 @@
 
 namespace App\Consumer\Handler;
 
-class DeleteArchive extends BuildArchive
+class DeleteArchive
 {
+    public function __construct(private string $id)
+    {
+    }
 
+    public function getId(): string
+    {
+        return $this->id;
+    }
 }

api/src/Entity/Archive.php

@@ -37,10 +37,12 @@
            processor: ArchiveProcessor::class
        ),
        new Get(),
-       new Patch(),
+       new Patch(
+            input: ArchiveInput::class,
+            processor: ArchiveProcessor::class
+       ),
        new Delete(),
    ],
-   input: ArchiveInput::class,
    output: ArchiveOutput::class
   )]
 #[ORM\Entity(repositoryClass: ArchiveRepository::class)]

api/tests/Archive/ArchiveApiTest.php

@@ -83,7 +83,7 @@ public function testPatchArchive(): void
         $this->assertEquals(200, $response->getStatusCode());
 
         $archive = $this->getArchiveFromDatabase($archive->getId());
-        // $this->expectedFiles($files, $archive);
+        $this->expectedFiles($files, $archive);
     }
 
     public function testDeleteArchive(): void

api/tests/Archive/PostArchiveApiTest.php

@@ -94,6 +94,9 @@ private function doTestPostArchiveOK(array $files, array $options = []): void
         $this->assertEquals('created', $json['status']);
         $this->assertArrayHasKey('downloadUrl', $json);
         $this->assertMatchesRegularExpression(sprintf('#^http://localhost/archives/%s/download\?jwt=.+$#', $id), $json['downloadUrl']);
+    
+        $archive = $this->getArchiveFromDatabase($id);
+        $this->expectedFiles($files, $archive);  
     }
 
     public function testPostArchiveWithEmptyIdentifierReturns422(): void