OP-377 - Add fix for PHPSpec, PHPStan and run ECS

spec/Action/NotifyActionSpec.php

@@ -45,8 +45,13 @@ function it_execute(
         FactoryInterface $stateMachineFactory,
         StateMachineInterface $stateMachine
     ) {
+        $params = [
+            'authorisationId' => '123'
+        ];
         $request->getModel()->willReturn($arrayObject);
         $request->getFirstModel()->willReturn($payment);
+        $mercanetBnpParibasBridge->getAuthorisationId()->willReturn('123');
+        $payment->getDetails()->willReturn($params);
         $mercanetBnpParibasBridge->isPostMethod()->willReturn(true);
         $mercanetBnpParibasBridge->paymentVerification()->willReturn(true);
         $stateMachineFactory->get($payment, PaymentTransitions::GRAPH)->willReturn($stateMachine);

spec/Bridge/MercanetBnpParibasBridgeSpec.php

@@ -67,7 +67,7 @@ function it_payment_verification_has_been_thrown(
     {
         $request->isMethod('POST')->willReturn(true);
         $requestStack->getCurrentRequest()->willReturn($request);
-
+        $this->setSecretKey('123');
         $this
             ->shouldThrow(\InvalidArgumentException::class)
             ->during('paymentVerification', ['key'])

src/Bridge/MercanetBnpParibasBridge.php

@@ -13,16 +13,14 @@
 namespace BitBag\MercanetBnpParibasPlugin\Bridge;
 
 use BitBag\MercanetBnpParibasPlugin\Legacy\Mercanet;
-use BitBag\MercanetBnpParibasPlugin\Legacy\ShaComposer;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\RequestStack;
 
 final class MercanetBnpParibasBridge implements MercanetBnpParibasBridgeInterface
 {
     private RequestStack $requestStack;
 
-    /** @phpstan-ignore-next-line We should not change our business logic now*/
-    private ShaComposer $secretKey;
+    private string $secretKey;
 
     private string $merchantId;
 
@@ -38,7 +36,7 @@ public function __construct(RequestStack $requestStack)
     }
 
     /** @phpstan-ignore-next-line We should not change our business logic now*/
-    public function createMercanet(ShaComposer $secretKey): Mercanet
+    public function createMercanet(string $secretKey): Mercanet
     {
         return new Mercanet($secretKey);
     }
@@ -70,13 +68,13 @@ public function isPostMethod(): bool
     }
 
     /** @phpstan-ignore-next-line We should not change our business logic now*/
-    public function getSecretKey(): ShaComposer
+    public function getSecretKey(): string
     {
         return $this->secretKey;
     }
 
     /** @phpstan-ignore-next-line We should not change our business logic now*/
-    public function setSecretKey(ShaComposer $secretKey): void
+    public function setSecretKey(string $secretKey): void
     {
         $this->secretKey = $secretKey;
     }

src/Bridge/MercanetBnpParibasBridgeInterface.php

@@ -13,24 +13,21 @@
 namespace BitBag\MercanetBnpParibasPlugin\Bridge;
 
 use BitBag\MercanetBnpParibasPlugin\Legacy\Mercanet;
-use BitBag\MercanetBnpParibasPlugin\Legacy\ShaComposer;
 
 interface MercanetBnpParibasBridgeInterface
 {
     /** @phpstan-ignore-next-line We should not change our business logic now*/
-    public function createMercanet(ShaComposer $secretKey): Mercanet;
+    public function createMercanet(string $secretKey): Mercanet;
 
     public function paymentVerification(): bool;
 
     public function getAuthorisationId(): string;
 
     public function isPostMethod(): bool;
 
-    /** @phpstan-ignore-next-line We should not change our business logic now*/
-    public function getSecretKey(): ShaComposer;
+    public function getSecretKey(): string;
 
-    /** @phpstan-ignore-next-line We should not change our business logic now*/
-    public function setSecretKey(ShaComposer $secretKey): void;
+    public function setSecretKey(string $secretKey): void;
 
     public function getMerchantId(): string;
 

src/Legacy/Mercanet.php

@@ -52,7 +52,7 @@ class Mercanet
     ];
 
     /** @phpstan-ignore-next-line We should not change our business logic now*/
-    private ShaComposer $secretKey;
+    private string $secretKey;
 
     private string $pspURL = self::TEST;
 
@@ -123,7 +123,7 @@ public static function getCurrencies(): array
     }
 
     /** @phpstan-ignore-next-line We should not change our business logic now*/
-    public function __construct(ShaComposer $secret)
+    public function __construct(string $secret)
     {
         $this->secretKey = $secret;
     }
@@ -530,7 +530,7 @@ public function toParameterString(): string
     }
 
     /** @phpstan-ignore-next-line We should not change our business logic now*/
-    public static function createFromArray(ShaComposer $shaComposer, array $parameters): PaymentRequest|static
+    public static function createFromArray(string $shaComposer, array $parameters): PaymentRequest|static
     {
         /** @phpstan-ignore-next-line Unsafe usage of new static() */
         $instance = new static($shaComposer);