MAGETWO-69854: Can't set allowed or default currencies using config:set command

Author: oshmyheliuk

app/code/Magento/Config/Console/Command/ConfigSet/ProcessorFacade.php

@@ -6,6 +6,7 @@
 namespace Magento\Config\Console\Command\ConfigSet;
 
 use Magento\Config\Console\Command\ConfigSetCommand;
+use Magento\Framework\App\Config\ScopeConfigInterface;
 use Magento\Framework\App\Scope\ValidatorInterface;
 use Magento\Config\Model\Config\PathValidator;
 use Magento\Framework\Exception\LocalizedException;
@@ -14,6 +15,7 @@
 use Magento\Framework\Exception\ValidatorException;
 use Magento\Deploy\Model\DeploymentConfig\Hash;
 use Magento\Config\App\Config\Type\System;
+use Magento\Framework\App\Config;
 
 /**
  * Processor facade for config:set command.
@@ -55,22 +57,32 @@ class ProcessorFacade
      */
     private $hash;
 
+    /**
+     * The application config storage.
+     *
+     * @var ScopeConfigInterface
+     */
+    private $scopeConfig;
+
     /**
      * @param ValidatorInterface $scopeValidator The scope validator
      * @param PathValidator $pathValidator The path validator
      * @param ConfigSetProcessorFactory $configSetProcessorFactory The factory for config:set processors
      * @param Hash $hash The hash manager
+     * @param ScopeConfigInterface $scopeConfig The application config storage
      */
     public function __construct(
         ValidatorInterface $scopeValidator,
         PathValidator $pathValidator,
         ConfigSetProcessorFactory $configSetProcessorFactory,
-        Hash $hash
+        Hash $hash,
+        ScopeConfigInterface $scopeConfig
     ) {
         $this->scopeValidator = $scopeValidator;
         $this->pathValidator = $pathValidator;
         $this->configSetProcessorFactory = $configSetProcessorFactory;
         $this->hash = $hash;
+        $this->scopeConfig = $scopeConfig;
     }
 
     /**
@@ -107,6 +119,10 @@ public function process($path, $value, $scope, $scopeCode, $lock)
 
         $this->hash->regenerate(System::CONFIG_TYPE);
 
+        if ($this->scopeConfig instanceof Config) {
+            $this->scopeConfig->clean();
+        }
+
         return $message;
     }
 }

app/code/Magento/Config/Test/Unit/Console/Command/ConfigSet/ProcessorFacadeTest.php

@@ -17,6 +17,7 @@
 use Magento\Framework\Exception\ConfigurationMismatchException;
 use Magento\Deploy\Model\DeploymentConfig\Hash;
 use Magento\Config\App\Config\Type\System;
+use Magento\Framework\App\Config;
 use PHPUnit_Framework_MockObject_MockObject as Mock;
 
 /**
@@ -56,6 +57,11 @@ class ProcessorFacadeTest extends \PHPUnit_Framework_TestCase
      */
     private $hashMock;
 
+    /**
+     * @var Config|Mock
+     */
+    private $configMock;
+
     /**
      * @inheritdoc
      */
@@ -79,12 +85,16 @@ protected function setUp()
         $this->hashMock = $this->getMockBuilder(Hash::class)
             ->disableOriginalConstructor()
             ->getMock();
+        $this->configMock = $this->getMockBuilder(Config::class)
+            ->disableOriginalConstructor()
+            ->getMock();
 
         $this->model = new ProcessorFacade(
             $this->scopeValidatorMock,
             $this->pathValidatorMock,
             $this->configSetProcessorFactoryMock,
-            $this->hashMock
+            $this->hashMock,
+            $this->configMock
         );
     }
 
@@ -106,6 +116,8 @@ public function testProcess()
         $this->hashMock->expects($this->once())
             ->method('regenerate')
             ->with(System::CONFIG_TYPE);
+        $this->configMock->expects($this->once())
+            ->method('clean');
 
         $this->assertSame(
             'Value was saved.',
@@ -156,6 +168,8 @@ public function testProcessWithConfigurationMismatchException()
             ->willThrowException(new ConfigurationMismatchException(__('Some error')));
         $this->processorMock->expects($this->never())
             ->method('process');
+        $this->configMock->expects($this->never())
+            ->method('clean');
 
         $this->model->process('test/test/test', 'test', ScopeConfigInterface::SCOPE_TYPE_DEFAULT, null, false);
     }
@@ -180,6 +194,8 @@ public function testProcessWithCouldNotSaveException()
             ->method('process')
             ->with('test/test/test', 'test', ScopeConfigInterface::SCOPE_TYPE_DEFAULT, null)
             ->willThrowException(new CouldNotSaveException(__('Some error')));
+        $this->configMock->expects($this->never())
+            ->method('clean');
 
         $this->model->process('test/test/test', 'test', ScopeConfigInterface::SCOPE_TYPE_DEFAULT, null, false);
     }
@@ -196,6 +212,8 @@ public function testExecuteLock()
         $this->processorMock->expects($this->once())
             ->method('process')
             ->with('test/test/test', 'test', ScopeConfigInterface::SCOPE_TYPE_DEFAULT, null);
+        $this->configMock->expects($this->once())
+            ->method('clean');
 
         $this->assertSame(
             'Value was saved and locked.',