Actually throw if it is required regardless

Author: dantleech

lib/Resolver.php

@@ -126,11 +126,10 @@ public function resolve(array $config): array
         $config = array_merge($this->defaults, $config);
 
         if ($diff = array_diff($this->required, array_keys($config))) {
-            $this->throwOrLogError(new InvalidMap(sprintf(
+            throw new InvalidMap(sprintf(
                 'Key(s) "%s" are required',
                 implode('", "', $diff)
-            )));
-            $config = $this->removeKeys($config, $diff);
+            ));
         }
 
         foreach ($config as $key => $value) {
@@ -149,12 +148,12 @@ public function resolve(array $config): array
                 }
 
                 if (false === $valid) {
-                    $this->throwOrLogError(new InvalidMap(sprintf(
+                    throw new InvalidMap(sprintf(
                         'Type for "%s" expected to be "%s", got "%s"',
                         $key,
                         $this->types[$key],
                         $type
-                    )));
+                    ));
                 }
             }
         }
@@ -234,4 +233,12 @@ private function removeKeys(array $config, array $keys): array
 
         return $config;
     }
+
+    /**
+     * @return InvalidMap[]
+     */
+    public function errors(): array
+    {
+        return $this->errors;
+    }
 }

tests/Unit/ResolverTest.php

@@ -45,6 +45,7 @@ public function testIgnoresUnknownKey(): void
             'one' => 1,
             'two' => 2
         ]);
+        self::assertCount(1, $resolver->errors());
     }
 
     public function testMergesDefaults(): void