test: consolidate svg parser scenarios

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>

Author: vitormattos

tests/Unit/Pdf/Svg/SvgPathCommandParserTest.php

@@ -363,6 +363,18 @@ public static function provideBasicPathConversionScenarios(): iterable
                 '5.333333 11.333333 6.666667 11.333333 8.000000 10.000000 c',
             ],
         ];
+
+        yield 'multiple line commands keep progress strictly increasing' => [
+            'pathData' => 'M 0 0 L 10 10 L 5 5',
+            'minX' => 0.0,
+            'maxY' => 10.0,
+            'source' => '/tmp/progress.svg',
+            'expectedSnippets' => [
+                '0.000000 10.000000 m',
+                '10.000000 0.000000 l',
+                '5.000000 5.000000 l',
+            ],
+        ];
     }
 
     /**
@@ -409,6 +421,18 @@ public static function provideTransformAndCoordinateConversionScenarios(): itera
             ],
         ];
 
+        yield 'minx offset for cubic command' => [
+            'pathData' => 'M 2 0 C 4 2 6 2 8 0',
+            'minX' => 2.0,
+            'maxY' => 10.0,
+            'transformMatrix' => [1.0, 0.0, 0.0, 1.0, 0.0, 0.0],
+            'source' => '/tmp/minx-cubic.svg (via transform)',
+            'expectedSnippets' => [
+                '0.000000 10.000000 m',
+                '2.000000 8.000000 4.000000 8.000000 6.000000 10.000000 c',
+            ],
+        ];
+
         yield 'minx offset for arc commands' => [
             'pathData' => 'M 0 10 A 6 4 0 0 1 12 10',
             'minX' => 5.0,
@@ -717,60 +741,4 @@ public static function provideRelativeAfterClosePathScenarios(): iterable
             'description' => 'S after Z should treat previous curve as non-existent',
         ];
     }
-
-    public function testConvertPathDataEnforcesIndexProgressionStrictly(): void
-    {
-        $parser = new SvgPathCommandParser();
-
-        // Index must increase (strict > not <=); valid path must complete without error
-        $result = $parser->convertPathData('M 0 0 L 10 10 L 5 5', 0.0, 10.0, 'test.svg', [1, 0, 0, 1, 0, 0]);
-        self::assertNotEmpty($result);
-    }
-
-    public function testConvertPathDataDetectsZeroProgressNotEqualProgress(): void
-    {
-        $parser = new SvgPathCommandParser();
-
-        // Boundary: <= versus < check is crucial for progress detection
-        // Valid paths with standard commands must succeed
-
-        try {
-            // Valid path should not throw
-            $result = $parser->convertPathData('M 0 0 L 10 10 L 5 5', 0.0, 10.0, 'test.svg', [1, 0, 0, 1, 0, 0]);
-            self::assertNotEmpty($result);
-        } catch (InvalidArgumentException $e) {
-            self::fail('Valid path should not throw: ' . $e->getMessage());
-        }
-    }
-
-    public function testCoordinateTransformSubtractsMinXNotAdds(): void
-    {
-        $parser = new SvgPathCommandParser();
-
-        // Transform must subtract minX from x-coordinates
-        $result = $parser->convertPathData('M 0 0 L 10 10', 5.0, 10.0, 'test.svg', [1, 0, 0, 1, 0, 0]);
-
-        // With minX=5, starting x=0 should become x=0-5=-5, NOT x=0+5=5
-        self::assertStringContainsString('-5.000000 10.000000 m', $result);
-        self::assertStringContainsString('5.000000 0.000000 l', $result);
-    }
-
-    public function testCoordinateTransformMinusOperator(): void
-    {
-        $parser = new SvgPathCommandParser();
-
-        // Verify the cubic bezier transform subtracts minX correctly
-        $result = $parser->convertPathData(
-            'M 2 0 C 4 2 6 2 8 0',
-            2.0,
-            10.0,
-            'test.svg',
-            [1, 0, 0, 1, 0, 0],
-        );
-
-        // Expected: (2-2) (10-0) = 0 10, then cubic
-        self::assertStringContainsString('0.000000 10.000000 m', $result);
-        // Cubic control points: (4-2, 10-2) (6-2, 10-2) (8-2, 10-0)
-        self::assertStringContainsString('2.000000 8.000000 4.000000 8.000000 6.000000 10.000000 c', $result);
-    }
 }