add id validation and tests

Author: xr0master

lib/src/utils/is_limit_rate_hit.dart

@@ -19,9 +19,10 @@ Future<bool> isLimitRateHit(
     return false;
   }
 
-  validateLimitRateParams(options.throttle!);
-
   final id = options.id ?? 'default';
+  
+  validateLimitRateParams(options.throttle!, id);
+
   final leftTime = await getLeftTime(id, options.throttle!, storage);
 
   if (leftTime > 0) {

lib/src/utils/validate_limit_rate_params.dart

@@ -1,5 +1,9 @@
-void validateLimitRateParams(int throttle) {
+void validateLimitRateParams(int throttle, [String? id]) {
   if (throttle < 0) {
     throw 'The LimitRate throttle has to be a positive number';
   }
+
+  if (id != null && id.isEmpty) {
+    throw 'The LimitRate ID has to be a non-empty string';
+  }
 }

test/utils/validate_limit_rate_params_test.dart

@@ -0,0 +1,36 @@
+import 'package:emailjs/src/utils/validate_limit_rate_params.dart';
+import 'package:flutter_test/flutter_test.dart';
+
+void main() {
+  group('should fail', () {
+    test('throttle is a negative number', () {
+      expect(
+        () => validateLimitRateParams(-1000),
+        throwsA(startsWith('The LimitRate throttle has to be a positive number')),
+      );
+    });
+
+    test('ID is empty', () {
+      expect(
+        () => validateLimitRateParams(1000, ''),
+        throwsA(startsWith('The LimitRate ID has to be a non-empty string')),
+      );
+    });
+  });
+
+  group('should successfully pass the validation', () {
+    test('throttle is a positive number', () {
+      expect(
+        () => validateLimitRateParams(1000),
+        returnsNormally,
+      );
+    });
+
+    test('ID is valid string', () {
+      expect(
+        () => validateLimitRateParams(1000, 'app'),
+        returnsNormally,
+      );
+    });
+  });
+}