payflexv4pos - fix order status request with transaction_id

mewebstudio · Aug 17, 2024 · ff4a3c6 · ff4a3c6
1 parent b333850
commit ff4a3c6
Show file tree

Hide file tree

Showing 2 changed files with 51 additions and 1 deletion.
diff --git a/src/DataMapper/RequestDataMapper/PayFlexV4PosRequestDataMapper.php b/src/DataMapper/RequestDataMapper/PayFlexV4PosRequestDataMapper.php
@@ -341,7 +341,8 @@ protected function preparePostPaymentOrder(array $order): array
     protected function prepareStatusOrder(array $order): array
     {
         return [
-            'id' => $order['id'],
+            'id'             => $order['id'],
+            'transaction_id' => $order['transaction_id'] ?? null,
         ];
     }
 

diff --git a/tests/Unit/DataMapper/RequestDataMapper/PayFlexV4PosRequestDataMapperTest.php b/tests/Unit/DataMapper/RequestDataMapper/PayFlexV4PosRequestDataMapperTest.php
@@ -248,6 +248,16 @@ public function testCreate3DFormData(): void
         $this->assertEquals($expectedValue, $actualData);
     }
 
+    /**
+     * @dataProvider createStatusRequestDataDataProvider
+     */
+    public function testCreateStatusRequestData(array $order, array $expectedData): void
+    {
+        $actual = $this->requestDataMapper->createStatusRequestData($this->account, $order);
+
+        $this->assertSame($expectedData, $actual);
+    }
+
     public function testCreateOrderHistoryRequestData(): void
     {
         $this->expectException(\Mews\Pos\Exceptions\NotImplementedException::class);
@@ -260,6 +270,45 @@ public function testCreateHistoryRequestData(): void
         $this->requestDataMapper->createHistoryRequestData($this->account, []);
     }
 
+    public static function createStatusRequestDataDataProvider(): array
+    {
+        return [
+            'only_with_order_id'      => [
+                'order'    => [
+                    'id' => 'order222',
+                ],
+                'expected' => [
+                    'MerchantCriteria'    => [
+                        'HostMerchantId'   => '000000000111111',
+                        'MerchantPassword' => '3XTgER89as',
+                    ],
+                    'TransactionCriteria' => [
+                        'TransactionId' => '',
+                        'OrderId'       => 'order222',
+                        'AuthCode'      => '',
+                    ],
+                ],
+            ],
+            'with_order_id_and_tx_id' => [
+                'order'    => [
+                    'id'             => 'order222',
+                    'transaction_id' => 'tx222',
+                ],
+                'expected' => [
+                    'MerchantCriteria'    => [
+                        'HostMerchantId'   => '000000000111111',
+                        'MerchantPassword' => '3XTgER89as',
+                    ],
+                    'TransactionCriteria' => [
+                        'TransactionId' => 'tx222',
+                        'OrderId'       => 'order222',
+                        'AuthCode'      => '',
+                    ],
+                ],
+            ],
+        ];
+    }
+
     /**
      * @return array
      */