Allow Flags without seperators in cubic beizer curves

Added tests for #6 and #7
Fixes #6

Author: masterashu

CHANGELOG.md

@@ -1,8 +1,13 @@
+## [1.1.1] - 19-July-2021
+
+* Allow Flags without separators in cubic Bézier curves
+  see details here: [issue#6](https://github.com/masterashu/svg_path_parser/issues/6)
+
 ## [1.1.0] - 19-July-2021
 
 * Fixed an issue where using an absolute command after 
   a relative command will lead to wrong parsing behavior.  
-  See details here: [issue#6](https://github.com/masterashu/svg_path_parser/issues/6)
+  See details here: [issue#7](https://github.com/masterashu/svg_path_parser/issues/7)
 
 ## [1.0.0] - 7-March-2021
 

README.md

@@ -10,7 +10,7 @@ Add this to your package's **pubspec.yaml** file:
 
 ```yaml
 dependencies:
-  svg_path_parser: ^1.1.0
+  svg_path_parser: ^1.1.1
 ```
 
 Now in your Dart code, you can use:

example/pubspec.lock

@@ -120,7 +120,7 @@ packages:
       path: ".."
       relative: true
     source: path
-    version: "1.0.0"
+    version: "1.1.1"
   term_glyph:
     dependency: transitive
     description:

lib/src/scanner.dart

@@ -50,7 +50,7 @@ class Scanner {
       RegExp(r'[+]?[0-9]*\.?[0-9]+([eE][-+]?[0-9]+)?');
 
   /// The [RegExp] pattern to match a boolean flag (`1` or `0`).
-  static final flagPattern = RegExp(r'(0|1)(?=[,\s])');
+  static final flagPattern = RegExp(r'(0|1)');
 
   /// Queue of tokens generated to be returned.
   final _tokens = Queue<Token>();

pubspec.yaml

@@ -2,7 +2,7 @@ name: svg_path_parser
 description: >
   A Flutter/Dart utility to parse an SVG path into a
   equivalent Path object from dart:ui library.
-version: 1.1.0
+version: 1.1.1
 homepage: https://github.com/masterashu/svg_path_parser
 repository: https://github.com/masterashu/svg_path_parser
 issue_tracker: https://github.com/masterashu/svg_path_parser/issues

test/parser_test.dart

@@ -7,7 +7,7 @@ import 'package:flutter_test/flutter_test.dart';
 // Tests the scanner for correctly parses the input.
 main() {
   final testingArea =
-      Rect.fromCenter(center: Offset.zero, width: 10, height: 10);
+      Rect.fromCenter(center: Offset.zero, width: 15, height: 15);
   group('Testing paths from commands', () {
     group('test moveTo commands', () {
       test('test consecutive arguments act as lineTo commands.', () {
@@ -216,4 +216,77 @@ main() {
       });
     });
   });
+
+  group('Testing combination of relative and absolute commands', () {
+    final testingArea =
+        Rect.fromCenter(center: Offset.zero, width: 20, height: 20);
+
+    group('Test Flag pattern on cubic curves', () {
+      final testingArea =
+          Rect.fromCenter(center: Offset.zero, width: 20, height: 20);
+      group('test ellipticalArcTo commands', () {
+        test('test ellipticalArtTo absolute', () {
+          var expectedPath = Path()
+            ..moveTo(0, 4)
+            ..arcToPoint(Offset(7, 1),
+                radius: Radius.elliptical(5, 8),
+                rotation: 0.12,
+                largeArc: true,
+                clockwise: true);
+          expect(
+              parseSvgPath('M0,4A5,8,.12 11 7 1'),
+              coversSameAreaAs(expectedPath,
+                  areaToCompare: testingArea, sampleSize: 400));
+        });
+
+        test('test ellipticalArtTo relative', () {
+          var expectedPath = Path()
+            ..moveTo(0, 0)
+            ..arcToPoint(Offset(5, 5),
+                radius: Radius.elliptical(11, 7),
+                largeArc: true,
+                clockwise: true)
+            ..close();
+          // Note: Make a check for out of range parameters:
+          // https://www.w3.org/TR/SVG11/implnote.html#ArcOutOfRangeParameters
+          expect(
+              parseSvgPath('a11,7, 0 115,5z'),
+              coversSameAreaAs(expectedPath,
+                  areaToCompare: testingArea, sampleSize: 400));
+        });
+      });
+    });
+  });
+
+  group('Test square path on different combinations', () {
+    test('Testing flags without any spaces', () {
+      var expectedPath = Path()
+        ..moveTo(0, 0)
+        ..lineTo(0, 5)
+        ..lineTo(5, 5)
+        ..lineTo(5, 0)
+        ..close();
+
+      expect(
+          parseSvgPath('M 0 0 V5 H5 V0z'),
+          coversSameAreaAs(expectedPath,
+              areaToCompare: testingArea, sampleSize: 400));
+      expect(
+          parseSvgPath('M0 0 v5 h5 V0z'),
+          coversSameAreaAs(expectedPath,
+              areaToCompare: testingArea, sampleSize: 400));
+      expect(
+          parseSvgPath('M0 0 V5 h5 v-5z'),
+          coversSameAreaAs(expectedPath,
+              areaToCompare: testingArea, sampleSize: 400));
+      expect(
+          parseSvgPath('M0 0 h5 V5 H0 z'),
+          coversSameAreaAs(expectedPath,
+              areaToCompare: testingArea, sampleSize: 400));
+      expect(
+          parseSvgPath('M0 0 v5 H5 v-5z'),
+          coversSameAreaAs(expectedPath,
+              areaToCompare: testingArea, sampleSize: 400));
+    });
+  });
 }