PAYOSWXP-156: set transaction-state to „authorized“ on „appointed“

src/DependencyInjection/controllers.xml

@@ -61,19 +61,6 @@
             </call>
         </service>
 
-        <service id="PayonePayment\Storefront\Controller\Account\AccountOrderControllerDecorator"
-                 decorates="Shopware\Storefront\Controller\AccountOrderController">
-            <argument type="service" id="PayonePayment\Storefront\Controller\Account\AccountOrderControllerDecorator.inner" />
-            <argument type="service" id="order.repository" />
-
-            <call method="setContainer">
-                <argument type="service" id="service_container"/>
-            </call>
-            <call method="setTwig">
-                <argument type="service" id="twig"/>
-            </call>
-        </service>
-
         <service id="PayonePayment\Storefront\Controller\Debit\ManageMandateController">
             <argument type="service" id="PayonePayment\Payone\RequestParameter\RequestParameterFactory" />
             <argument type="service" id="PayonePayment\Payone\Client\PayoneClient" />

src/DependencyInjection/webhooks.xml

@@ -35,6 +35,10 @@
             <tag name="payone.webhook.handler" />
         </service>
 
+        <service id="PayonePayment\Payone\Webhook\Handler\PaymentStatusHandler" autowire="true">
+            <tag name="payone.webhook.handler" />
+        </service>
+
         <service id="PayonePayment\Payone\Webhook\MessageBus\MessageHandler\NotificationForwardHandler" autowire="true">
             <argument key="$forwardTargetRepository" type="service" id="payone_payment_notification_target.repository" />
             <argument key="$notificationForwardRepository" type="service" id="payone_payment_notification_forward.repository" />

src/Payone/Webhook/Handler/PaymentStatusHandler.php

@@ -0,0 +1,51 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PayonePayment\Payone\Webhook\Handler;
+
+use PayonePayment\Components\DataHandler\Transaction\TransactionDataHandlerInterface;
+use PayonePayment\Components\TransactionStatus\TransactionStatusService;
+use PayonePayment\Struct\PaymentTransaction;
+use Shopware\Core\Checkout\Order\Aggregate\OrderTransaction\OrderTransactionDefinition;
+use Shopware\Core\System\SalesChannel\SalesChannelContext;
+use Shopware\Core\System\StateMachine\Aggregation\StateMachineTransition\StateMachineTransitionActions;
+use Shopware\Core\System\StateMachine\StateMachineRegistry;
+use Shopware\Core\System\StateMachine\Transition;
+use Symfony\Component\HttpFoundation\Request;
+
+class PaymentStatusHandler implements WebhookHandlerInterface
+{
+    public function __construct(
+        private readonly TransactionDataHandlerInterface $transactionDataHandler,
+        private readonly StateMachineRegistry $stateMachineRegistry
+    ) {
+    }
+
+    public function process(SalesChannelContext $salesChannelContext, Request $request): void
+    {
+        $paymentTransaction = $this->transactionDataHandler->getPaymentTransactionByPayoneTransactionId(
+            $salesChannelContext->getContext(),
+            $request->request->getInt('txid')
+        );
+
+        if (!$paymentTransaction instanceof PaymentTransaction) {
+            return;
+        }
+
+        $this->stateMachineRegistry->transition(
+            new Transition(
+                OrderTransactionDefinition::ENTITY_NAME,
+                $paymentTransaction->getOrderTransaction()->getId(),
+                StateMachineTransitionActions::ACTION_AUTHORIZE,
+                'stateId'
+            ),
+            $salesChannelContext->getContext()
+        );
+    }
+
+    public function supports(SalesChannelContext $salesChannelContext, array $data): bool
+    {
+        return isset($data['txid']) && ($data['txaction'] ?? null) === TransactionStatusService::ACTION_APPOINTED;
+    }
+}

tests/Payone/Webhook/Handler/PaymentStatusHandlerTest.php

@@ -0,0 +1,68 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PayonePayment\Payone\Webhook\Handler;
+
+use PayonePayment\Components\DataHandler\Transaction\TransactionDataHandlerInterface;
+use PayonePayment\Components\TransactionStatus\TransactionStatusService;
+use PayonePayment\Struct\PaymentTransaction;
+use PHPUnit\Framework\TestCase;
+use Shopware\Core\System\SalesChannel\SalesChannelContext;
+use Shopware\Core\System\StateMachine\StateMachineRegistry;
+use Symfony\Component\HttpFoundation\Request;
+
+/**
+ * @covers \PayonePayment\Payone\Webhook\Handler\PaymentStatusHandler
+ */
+class PaymentStatusHandlerTest extends TestCase
+{
+    public function testSupports(): void
+    {
+        $handler = new PaymentStatusHandler(
+            $this->createMock(TransactionDataHandlerInterface::class),
+            $this->createMock(StateMachineRegistry::class),
+        );
+
+        $salesChannelMock = $this->createMock(SalesChannelContext::class);
+
+        static::assertTrue($handler->supports($salesChannelMock, ['txid' => 1, 'txaction' => 'appointed']), 'supports should return true, cause of valid txid nd correct status');
+
+        static::assertFalse($handler->supports($salesChannelMock, ['txaction' => 'appointed']), 'supports should return false, cause of missing txid');
+        static::assertFalse($handler->supports($salesChannelMock, ['txid' => 1]), 'supports should return false, cause of missing status');
+        static::assertFalse($handler->supports($salesChannelMock, []), 'supports should return false, cause of missing data');
+        static::assertFalse($handler->supports($salesChannelMock, []), 'supports should return false, cause of missing data');
+
+        static::assertFalse($handler->supports($salesChannelMock, ['txid' => 1, 'txaction' => TransactionStatusService::ACTION_CAPTURE]), 'supports should return false, cause of wrong status');
+        static::assertFalse($handler->supports($salesChannelMock, ['txid' => 1, 'txaction' => TransactionStatusService::ACTION_COMPLETED]), 'supports should return false, cause of wrong status');
+        static::assertFalse($handler->supports($salesChannelMock, ['txid' => 1, 'txaction' => TransactionStatusService::ACTION_FAILED]), 'supports should return false, cause of wrong status');
+    }
+
+    public function testIsCaptureGotExecuted(): void
+    {
+        $dataHandler = $this->createMock(TransactionDataHandlerInterface::class);
+        $dataHandler->method('getPaymentTransactionByPayoneTransactionId')->willReturn($this->createMock(PaymentTransaction::class));
+        $stateMachine = $this->createMock(StateMachineRegistry::class);
+        $stateMachine->expects(static::once())->method('transition');
+
+        $request = new Request();
+        $request->request->set('txid', 123);
+
+        (new PaymentStatusHandler($dataHandler, $stateMachine))
+            ->process($this->createMock(SalesChannelContext::class), $request);
+    }
+
+    public function testIsCaptureGotNotExecutedIfTransactionIsNotFound(): void
+    {
+        $dataHandler = $this->createMock(TransactionDataHandlerInterface::class);
+        $dataHandler->method('getPaymentTransactionByPayoneTransactionId')->willReturn(null);
+        $stateMachine = $this->createMock(StateMachineRegistry::class);
+        $stateMachine->expects(static::never())->method('transition');
+
+        $request = new Request();
+        $request->request->set('txid', 123);
+
+        (new PaymentStatusHandler($dataHandler, $stateMachine))
+            ->process($this->createMock(SalesChannelContext::class), $request);
+    }
+}