fix: type and lint

src/Endpoints/Merchants.php

@@ -101,7 +101,7 @@ public function sendCartInitiatedBusinessEvent(CartInitiatedBusinessEvent $cartE
     }
 
     /**
-     * Prepare and send a business event for a cart initiated
+     * Prepare and send a business event for Order confirmed
      *
      * @param OrderConfirmedBusinessEvent $orderConfirmedBusinessEvent
      * @return void

src/Entities/DTO/MerchantBusinessEvent/OrderConfirmedBusinessEvent.php

@@ -29,7 +29,7 @@ class OrderConfirmedBusinessEvent extends AbstractBusinessEvent
      */
     private $cartId;
     /**
-     * @var string
+     * @var string | null
      */
     private $almaPaymentId;
 

tests/Unit/Endpoints/MerchantsTest.php

@@ -6,7 +6,6 @@
 use Alma\API\Endpoints\Merchants;
 use Alma\API\Entities\DTO\MerchantBusinessEvent\CartInitiatedBusinessEvent;
 use Alma\API\Entities\DTO\MerchantBusinessEvent\OrderConfirmedBusinessEvent;
-use Alma\API\Exceptions\ParametersException;
 use Alma\API\Exceptions\RequestException;
 use Alma\API\Request;
 use Alma\API\RequestError;
@@ -45,6 +44,7 @@ public function setUp(): void
         $this->clientContext->logger = $loggerMock;
         $this->merchantEndpoint->setClientContext($this->clientContext);
     }
+
     public function tearDown(): void
     {
         $this->merchantEndpoint = null;
@@ -69,6 +69,7 @@ public function testSendBusinessEventPostThrowRequestErrorThrowRequestException(
         $this->expectException(RequestException::class);
         $this->merchantEndpoint->sendCartInitiatedBusinessEvent(new CartInitiatedBusinessEvent('42'));
     }
+
     public function testSendBusinessEventBadResponseRequestException()
     {
         $this->merchantEndpoint->shouldReceive('request')
@@ -88,10 +89,7 @@ public function testSendBusinessEventBadResponseRequestException()
         $this->expectException(RequestException::class);
         $this->merchantEndpoint->sendCartInitiatedBusinessEvent(new CartInitiatedBusinessEvent('42'));
     }
-    /**
-     *
-     * @return void
-     */
+
     public function testSendCartInitiatedEvent()
     {
         $cartInitiatedEvent = new CartInitiatedBusinessEvent('42');
@@ -107,10 +105,7 @@ public function testSendCartInitiatedEvent()
         $this->requestObject->shouldReceive('post')->once()->andReturn($this->responseMock);
         $this->assertNull($this->merchantEndpoint->sendCartInitiatedBusinessEvent($cartInitiatedEvent));
     }
-    /**
-     *
-     * @return void
-     */
+
     public function testSendOrderConfirmedBusinessEventForNonAlmaPayment()
     {
         $orderConfirmedBusinessEvent = new OrderConfirmedBusinessEvent(
@@ -142,6 +137,7 @@ public function testSendOrderConfirmedBusinessEventForNonAlmaPayment()
         $this->requestObject->shouldReceive('post')->once()->andReturn($this->responseMock);
         $this->assertNull($this->merchantEndpoint->sendOrderConfirmedBusinessEvent($orderConfirmedBusinessEvent));
     }
+
     public function testSendOrderConfirmedBusinessEventForAlmaPayment()
     {
         $orderConfirmedBusinessEvent = new OrderConfirmedBusinessEvent(
@@ -174,5 +170,4 @@ public function testSendOrderConfirmedBusinessEventForAlmaPayment()
         $this->requestObject->shouldReceive('post')->once()->andReturn($this->responseMock);
         $this->assertNull($this->merchantEndpoint->sendOrderConfirmedBusinessEvent($orderConfirmedBusinessEvent));
     }
-
 }

tests/Unit/Entities/DTO/MerchantBusinessEvent/OrderConfirmedBusinessEventTest.php

@@ -52,6 +52,7 @@ public function testAlmaPaymentIdIsMandatoryForBnplAlmaPayment()
         $this->expectException(ParametersException::class);
         new OrderConfirmedBusinessEvent(false, true, true, "42", "54");
     }
+
     public function testAlmaPaymentIdShouldBeAbsentForNonAlmaPayments()
     {
         $this->expectException(ParametersException::class);
@@ -92,7 +93,6 @@ public function testAlmaPaymentIdDataForAlmaPayment()
         }
     }
 
-
     public static function invalidDataForBusinessEventDataProvider()
     {
         return [