Merge pull request #46 from airtimerewards/fix-exchangeratesapi-config

Fix configuration for exchange_rates_api provider

Author: florianv

DependencyInjection/Configuration.php

@@ -72,7 +72,18 @@ public function getConfigTreeBuilder()
                             ->end()
                         ->end()
                         ->append($this->createSimpleProviderNode('cryptonator'))
-                        ->append($this->createSimpleProviderNode('exchange_rates_api'))
+                        ->arrayNode('exchange_rates_api')
+                            ->children()
+                                ->integerNode('priority')->defaultValue(0)->end()
+                                ->scalarNode('access_key')
+                                    ->treatFalseLike(null)
+                                    ->treatTrueLike(null)
+                                    ->isRequired()
+                                    ->cannotBeEmpty()
+                                ->end()
+                                ->booleanNode('enterprise')->defaultFalse()->end()
+                            ->end()
+                        ->end()
                         ->append($this->createSimpleProviderNode('webservicex'))
                         ->append($this->createSimpleProviderNode('central_bank_of_czech_republic'))
                         ->append($this->createSimpleProviderNode('central_bank_of_republic_turkey'))

Tests/DependencyInjection/ConfigurationTest.php

@@ -125,7 +125,7 @@ public function provideValidProvidersConfigs(): array
             [['cryptonator' => null]],
             [['currency_data_feed' => ['api_key' => 'any']]],
             [['currency_layer' => ['access_key' => 'any', 'enterprise' => true]]],
-            [['exchange_rates_api' => null]],
+            [['exchange_rates_api' => ['access_key' => 'any', 'enterprise' => false]]],
             [['forge' => ['api_key' => 'any']]],
             [['open_exchange_rates' => ['app_id' => 'any']]],
             [['xignite' => ['token' => 'any']]],