Added validation for the new config field.

ash-jc-allen · Jul 2, 2024 · d95b881 · d95b881
1 parent 7c3c676
commit d95b881
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/Classes/Validation.php b/src/Classes/Validation.php
@@ -32,6 +32,7 @@ public function validateConfig(): bool
                     Rule::make('enforce_https')->rules(['required', 'boolean']),
                     Rule::make('forward_query_params')->rules(['required', 'boolean']),
                     Rule::make('default_url')->rules(['nullable', 'string']),
+                    Rule::make('allowed_url_schemes')->rules(['required', 'array']),
                 ],
             ]);
 

diff --git a/tests/Unit/Classes/ValidationTest.php b/tests/Unit/Classes/ValidationTest.php
@@ -129,4 +129,16 @@ public function exception_is_thrown_if_the_default_url_is_not_a_string(): void
         $validation = new Validation();
         $validation->validateConfig();
     }
+
+    #[Test]
+    public function exception_is_thrown_if_the_allowed_url_schemes_is_not_an_array(): void
+    {
+        $this->expectException(ValidationException::class);
+        $this->expectExceptionMessage('The short-url.allowed_url_schemes field must be an array.');
+
+        Config::set('short-url.allowed_url_schemes', 'INVALID');
+
+        $validation = new Validation();
+        $validation->validateConfig();
+    }
 }