Merge pull request #47 from dragosprotung/master

Allow configuring AbstractApi
florianv · Sep 24, 2021 · 2cfd260 · 2cfd260
2 parents 3c788ce + 6cca708
commit 2cfd260
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
@@ -59,6 +59,17 @@ public function getConfigTreeBuilder()
                 ->end()
                 ->arrayNode('providers')
                     ->children()
+                        ->arrayNode('abstract_api')
+                            ->children()
+                                ->integerNode('priority')->defaultValue(0)->end()
+                                ->scalarNode('api_key')
+                                    ->treatFalseLike(null)
+                                    ->treatTrueLike(null)
+                                    ->isRequired()
+                                    ->cannotBeEmpty()
+                                ->end()
+                            ->end()
+                        ->end()
                         ->arrayNode('fixer')
                             ->children()
                                 ->integerNode('priority')->defaultValue(0)->end()

diff --git a/Tests/DependencyInjection/ConfigurationTest.php b/Tests/DependencyInjection/ConfigurationTest.php
@@ -115,6 +115,7 @@ public function testInvalidCacheConfig(array $cache): void
     public function provideValidProvidersConfigs(): array
     {
         return [
+            [['abstract_api' => ['api_key' => 'any']]],
             [['central_bank_of_czech_republic' => null]],
             [['central_bank_of_republic_turkey' => []]],
             [['european_central_bank' => null]],