Allow zero amount items

.github/workflows/run-workflow.yaml

@@ -31,15 +31,15 @@ jobs:
         run: composer lint
 
       - name: Run phpstan
-        uses: php-actions/phpstan@v3.0.1
+        uses: php-actions/phpstan@v3
         with:
           php_version: ${{ matrix.php-version }}
           version: 1.9.14
           configuration: phpstan.neon
           memory_limit: 256M
 
       - name: Run tests
-        uses: php-actions/phpunit@v3.0.0
+        uses: php-actions/phpunit@v3
         with:
           php_version: ${{ matrix.php-version }}
           version: 9

src/Model/Item.php

@@ -376,13 +376,16 @@ public function validate()
     {
         $props = get_object_vars($this);
 
-        if (empty($props['unitPrice'])) {
+        if ($props['unitPrice'] === null) {
             throw new ValidationException('Item unitPrice is empty');
         }
         if ($props['unitPrice'] < 0) {
             throw new ValidationException('Items unitPrice can\'t be a negative number');
         }
-        if (empty($props['units'])) {
+        if ($props['unitPrice'] > 99999999) {
+            throw new ValidationException('Items unitPrice can\'t be over 99999999');
+        }
+        if ($props['units'] === null) {
             throw new ValidationException('Item units is empty');
         }
         if ($props['units'] < 0) {

tests/ClientTest.php

@@ -495,7 +495,8 @@ public function testRequestSettlementsWithInvalidDateThrowsException()
     public function testRequestSettlementsReturnsValidResponse()
     {
         $settlementRequest = (new SettlementRequest());
-        $this->client->requestSettlements($settlementRequest);
+        $settlementResponse = $this->client->requestSettlements($settlementRequest);
+        $this->assertIsArray($settlementResponse->getSettlements());
     }
 
     public function testGetSettlementsReturnsValidResponse()

tests/Model/ItemTest.php

@@ -20,7 +20,7 @@ public function testIsItemValid()
             ->setDeliveryDate('2023-01-01')
             ->setDescription('description');
 
-        $this->assertEquals(true, $item->validate());
+        $this->assertTrue($item->validate());
     }
 
     public function testItemWithoutUnitPriceThrowsError()
@@ -34,18 +34,6 @@ public function testItemWithoutUnitPriceThrowsError()
             ->validate();
     }
 
-    public function testItemWithNegativeUnitPriceThrowsError()
-    {
-        $this->expectException(ValidationException::class);
-        (new Item())->setUnitPrice(-1)
-            ->setUnits(2)
-            ->setStamp('thisIsStamp')
-            ->setVatPercentage(0)
-            ->setProductCode('productCode123')
-            ->setDescription('description')
-            ->validate();
-    }
-
     public function testItemWithoutUnitsTrowsError()
     {
         $this->expectException(ValidationException::class);
@@ -91,4 +79,36 @@ public function testItemWithoutProductCodeThrowsError()
             ->setDescription('description')
             ->validate();
     }
+
+    public static function providerForUnitPriceLimitValues()
+    {
+        return [
+            'Negative amount' => [-1, false],
+            'Zero amount' => [0, true],
+            'Maximum amount' => [99999999, true],
+            'Over maximum amount' => [100000000, false]
+        ];
+    }
+
+    /**
+     * @dataProvider providerForUnitPriceLimitValues
+     */
+    public function testUnitPriceLimitValues($unitPrice, $expectedResult)
+    {
+        $item = (new Item())->setUnitPrice($unitPrice)
+            ->setUnits(2)
+            ->setStamp('thisIsStamp')
+            ->setVatPercentage(0)
+            ->setProductCode('productCode123')
+            ->setDeliveryDate('2023-01-01')
+            ->setDescription('description');
+
+        try {
+            $validationResult = $item->validate();
+        } catch (ValidationException $exception) {
+            $validationResult = false;
+        }
+
+        $this->assertEquals($expectedResult, $validationResult);
+    }
 }