No need to validate anymore with typed properties

Author: barryvdh

src/CorsService.php

@@ -88,26 +88,9 @@ public function setOptions(array $options): void
         $exposedHeaders = $options['exposedHeaders'] ?? $options['exposed_headers'] ?? $this->exposedHeaders;
         $this->exposedHeaders = $exposedHeaders === false ? [] : $exposedHeaders;
 
-        $this->validateOptions();
         $this->normalizeOptions();
     }
 
-    private function validateOptions(): void
-    {
-        $arrayHeaders = [
-            'allowedOrigins',
-            'allowedOriginsPatterns',
-            'allowedHeaders',
-            'allowedMethods',
-            'exposedHeaders',
-        ];
-        foreach ($arrayHeaders as $key) {
-            if (!is_array($this->{$key})) {
-                throw new InvalidOptionException("CORS option `{$key}` should be an array");
-            }
-        }
-    }
-
     private function normalizeOptions(): void
     {
         // Normalize case

tests/CorsServiceTest.php

@@ -201,7 +201,7 @@ public function itAllowsZeroMaxAge(): void
      */
     public function itThrowsExceptionOnInvalidExposedHeaders(): void
     {
-        $this->expectException(InvalidOptionException::class);
+        $this->expectException(\TypeError::class);
 
         /** @phpstan-ignore-next-line */
         $service = new CorsService(['exposedHeaders' => true]);
@@ -212,7 +212,7 @@ public function itThrowsExceptionOnInvalidExposedHeaders(): void
      */
     public function itThrowsExceptionOnInvalidOriginsArray(): void
     {
-        $this->expectException(InvalidOptionException::class);
+        $this->expectException(\TypeError::class);
 
         /** @phpstan-ignore-next-line */
         $service = new CorsService(['allowedOrigins' => 'string']);