Refactor SimpleWithMod11 validator and update IdCardRuleTest

src/Algorithms/SimpleWithMod11.php

@@ -14,11 +14,6 @@ class SimpleWithMod11 implements Validator
      */
     public function validate(string $id): void
     {
-        // Check if the id length is exactly 13
-        if (strlen($id) !== 13) {
-            throw new InvalidThaiCitizenIdException('The id must be 13 digits.');
-        }
-
         // Calculate the sum of each digit multiplied by the weight
         $sum = 0;
         for ($i = 0; $i < 12; $i++) {

tests/Laravel/IdCardRuleTest.php

@@ -21,28 +21,21 @@
     $validator->validate();
 })->throws('The id must be 13 digits.');
 
-it('should match implementation when using Laravel 8', function () {
-    if (version_compare(app()->version(), '8.0.0', '<')) {
-        $this->markTestSkipped('This test only run on Laravel 8 or above.');
-    }
-
+it('should match implementation when using Laravel', function () {
     $validator = Validator::make(
         ['id_card' => '1410127443236'],
-        ['id_card' => new IdCard]
+        ['id_card' => $rule = new IdCard]
     );
 
-    expect($validator->passes())->toBeTrue();
-});
-
-it('should match implementation when using Laravel 10', function () {
-    if (version_compare(app()->version(), '8.0.0', '>=')) {
-        $this->markTestSkipped('This test only run on Laravel 10 or below.');
+    if (version_compare(app()->version(), '10.0.0', '<')) {
+        expect($rule)->toBeInstanceOf(Illuminate\Contracts\Validation\Rule::class);
+        expect($rule->message())->toBeString();
+    } else {
+        expect($rule)->toBeInstanceOf(Illuminate\Contracts\Validation\ValidationRule::class);
+        $rule->validate('id_card', '1410127443236', function ($message) {
+            expect($message)->toBeString();
+        });
     }
 
-    $validator = Validator::make(
-        ['id_card' => '1410127443236'],
-        ['id_card' => new IdCard]
-    );
-
     expect($validator->passes())->toBeTrue();
 });