🐛 fix curl lib not registering webhooks correctly

Author: sebastianMindee

.github/workflows/_test-integrations.yml

@@ -13,7 +13,8 @@ env:
   MINDEE_API_KEY: ${{ secrets.MINDEE_API_KEY_SE_TESTS }}
   WORKFLOW_ID: ${{ secrets.WORKFLOW_ID_SE_TESTS }}
   MINDEE_V2_API_KEY: ${{ secrets.MINDEE_V2_SE_TESTS_API_KEY }}
-  MINDEE_V2_FAILURE_WEBHOOK_ID: ${{ secrets.MINDEE_V2_SE_TESTS_FAILURE_WEBHOOK_ID }}
+  MINDEE_V2_FAILURE_WEBHOOK_ID: ${{ secrets.MINDEE_V2_SE_TESTS_WEBHOOK_ID }}
+  MINDEE_V2_SE_TESTS_FAILURE_WEBHOOK_ID: ${{ secrets.MINDEE_V2_SE_TESTS_FAILURE_WEBHOOK_ID }}
   MINDEE_V2_FINDOC_MODEL_ID: ${{ secrets.MINDEE_V2_SE_TESTS_FINDOC_MODEL_ID }}
   MINDEE_V2_SE_TESTS_BLANK_PDF_URL: ${{ secrets.MINDEE_V2_SE_TESTS_BLANK_PDF_URL }}
   MINDEE_V2_CLASSIFICATION_MODEL_ID: ${{ secrets.MINDEE_V2_SE_TESTS_CLASSIFICATION_MODEL_ID }}

src/V2/ClientOptions/BaseParameters.php

@@ -63,17 +63,9 @@ public function asHash(): array
             $outHash['alias'] = $this->alias;
         }
 
+
         if (isset($this->webhooksIds) && count($this->webhooksIds) > 0) {
-            if (PHP_VERSION_ID < 80200 && count($this->webhooksIds) > 1) {
-                // NOTE: see https://bugs.php.net/bug.php?id=51634
-                error_log("PHP version is too low to support webbook array destructuring.
-                \nOnly the first webhook ID will be sent to the server.");
-                $outHash['webhook_ids'] = $this->webhooksIds[0];
-            } else {
-                foreach ($this->webhooksIds as $webhookId) {
-                    $outHash['webhook_ids[]'] = $webhookId;
-                }
-            }
+            $outHash['webhook_ids'] = implode(',', $this->webhooksIds);
         }
         return $outHash;
     }

tests/V2/ClientOptions/BaseParametersTest.php

@@ -0,0 +1,25 @@
+<?php
+
+namespace V2\ClientOptions;
+
+use Mindee\V2\ClientOptions\BaseParameters;
+use PHPUnit\Framework\TestCase;
+
+class BaseParametersTest extends TestCase
+{
+    public function testAsHashShouldSerializeMultipleWebhookIdsAsIndexedFields(): void
+    {
+        $params = new class ('model-id', null, ['first-id', 'second-id'], null) extends BaseParameters {
+            public static string $slug = 'test';
+        };
+
+        $hash = $params->asHash();
+
+        $this->assertArrayHasKey('model_id', $hash);
+        $this->assertArrayHasKey('webhook_ids[0]', $hash);
+        $this->assertArrayHasKey('webhook_ids[1]', $hash);
+        $this->assertSame('model-id', $hash['model_id']);
+        $this->assertSame('first-id', $hash['webhook_ids[0]']);
+        $this->assertSame('second-id', $hash['webhook_ids[1]']);
+    }
+}

tests/V2/ClientV2TestFunctional.php

@@ -177,7 +177,8 @@ public function testUrlInputSourceMustNotRaiseErrors(): void
         $this->assertNotNull($result);
     }
 
-    public function testDataSchemaMustSucceed(): void {
+    public function testDataSchemaMustSucceed(): void
+    {
 
         $source = new PathInput(
             TestingUtilities::getFileTypesDir() . '/pdf/blank_1.pdf'
@@ -214,4 +215,18 @@ public function testDataSchemaMustSucceed(): void {
             $result->fields['test_replace']->value
         );
     }
+
+    public function testMultipleWebhooksMustSucceed(): void
+    {
+        $source = new PathInput(
+            TestingUtilities::getFileTypesDir() . '/pdf/blank_1.pdf'
+        );
+
+        $inferenceParams = new InferenceParameters($this->modelId, webhooksIds: [
+            getenv('MINDEE_V2_FAILURE_WEBHOOK_ID'),
+            getenv('MINDEE_V2_SE_TESTS_FAILURE_WEBHOOK_ID')]
+        );
+        $response = $this->mindeeClient->enqueue($source, $inferenceParams);
+        $this->assertEquals(2, count($response->job->webhooks));
+    }
 }

tests/V2/Http/MindeeApiV2MultipartTest.php

@@ -0,0 +1,37 @@
+<?php
+
+namespace V2\Http;
+
+use Mindee\Http\MindeeApiV2;
+use PHPUnit\Framework\TestCase;
+use ReflectionClass;
+
+class MindeeApiV2MultipartTest extends TestCase
+{
+    public function testMultipartPayloadShouldRepeatWebhookIdsKeyForEachWebhook(): void
+    {
+        $api = new MindeeApiV2('dummy-api-key');
+        $reflectionClass = new ReflectionClass($api);
+        $method = $reflectionClass->getMethod('buildMultipartPayload');
+        $method->setAccessible(true);
+
+        $payload = $method->invoke(
+            $api,
+            [
+                'model_id' => 'model-id',
+                'file' => new \CURLFile(
+                    __DIR__ . '/../../resources/file_types/pdf/blank_1.pdf',
+                    'application/pdf',
+                    'blank_1.pdf'
+                ),
+            ],
+            ['webhook-1', 'webhook-2']
+        );
+
+        $this->assertIsArray($payload);
+        $this->assertNotEmpty($payload['boundary']);
+        $this->assertStringContainsString('webhook-1', $payload['body']);
+        $this->assertStringContainsString('webhook-2', $payload['body']);
+        $this->assertSame(1, substr_count($payload['body'], 'name="webhook_ids"'));
+    }
+}

tests/V2/Parsing/JobResponseTest.php

@@ -39,6 +39,8 @@ public function testShouldLoadWhenStatusIsProcessing(): void
         $this->assertSame('Processing', $response->job->status);
         $this->assertNull($response->job->completedAt);
         $this->assertNull($response->job->error);
+        $this->assertIsArray($response->job->webhooks);
+        $this->assertCount(0, $response->job->webhooks);
     }
 
     /**