Merge pull request #81 from Floating-Dartists/fix/#64_double-slashes-…

…in-url

fix + tests

lib/src/matomo_event.dart

@@ -4,6 +4,7 @@ import 'dart:ui';
 
 import 'package:clock/clock.dart';
 import 'package:matomo_tracker/matomo_tracker.dart';
+import 'package:matomo_tracker/utils/extensions.dart';
 
 class MatomoEvent {
   MatomoEvent({
@@ -139,8 +140,11 @@ class MatomoEvent {
     final actionName = action;
     final camp = campaign;
     final campKeyword = camp?.keyword;
+    final localPath = path;
     final uri = Uri.parse(
-      path != null ? '${tracker.contentBase}/$path' : tracker.contentBase,
+      localPath != null
+          ? '${tracker.contentBase}${localPath.prefixWithSlash()}'
+          : tracker.contentBase,
     );
     final url = uri.replace(
       queryParameters: {

lib/utils/extensions.dart

@@ -0,0 +1,7 @@
+extension StringExtensions on String {
+  /// If the string is not already prefixed with a slash, prefix it with one.
+  String prefixWithSlash() {
+    if (startsWith('/')) return this;
+    return '/$this';
+  }
+}

test/utils/extensions_test.dart

@@ -0,0 +1,18 @@
+import 'package:flutter_test/flutter_test.dart';
+import 'package:matomo_tracker/utils/extensions.dart';
+
+void main() {
+  group('StringExtensions', () {
+    group('prefixWithSlash', () {
+      test('should return the same string', () {
+        const String testString = '/test';
+        expect(testString.prefixWithSlash(), testString);
+      });
+
+      test('should return the string with a slash prefix', () {
+        const String testString = 'test';
+        expect(testString.prefixWithSlash(), '/$testString');
+      });
+    });
+  });
+}