MAGETWO-65208: Store checksum for every section of configuration file & change behavior on read-only FS & add sorting of importers

Author: Bohdan Korablov

app/code/Magento/Deploy/Console/Command/App/ConfigImport/Importer.php

@@ -7,7 +7,6 @@
 
 use Magento\Framework\App\DeploymentConfig\ImporterInterface;
 use Magento\Framework\App\DeploymentConfig;
-use Magento\Framework\Exception\LocalizedException;
 use Magento\Framework\Exception\RuntimeException;
 use Psr\Log\LoggerInterface as Logger;
 use Magento\Deploy\Model\DeploymentConfig\Validator;
@@ -118,12 +117,9 @@ public function import(OutputInterface $output)
                     $this->configHash->regenerate($section);
                 }
             }
-        } catch (LocalizedException $exception) {
-            $this->logger->error($exception);
-            $output->writeln(sprintf('<error>%s</error>', $exception->getMessage()));
         } catch (\Exception $exception) {
             $this->logger->error($exception);
-            throw new RuntimeException(__('Import is failed. Please see the log report.'), $exception);
+            throw new RuntimeException(__('Import is failed: %1', $exception->getMessage()), $exception);
         }
     }
 }

app/code/Magento/Deploy/Test/Unit/Console/Command/App/ConfigImport/ImporterTest.php

@@ -7,7 +7,6 @@
 
 use Magento\Framework\App\DeploymentConfig\ImporterInterface;
 use Magento\Framework\App\DeploymentConfig;
-use Magento\Framework\Exception\LocalizedException;
 use Psr\Log\LoggerInterface as Logger;
 use Magento\Deploy\Console\Command\App\ConfigImport\Importer;
 use Magento\Deploy\Model\DeploymentConfig\Validator;
@@ -135,8 +134,8 @@ public function testImport()
     }
 
     /**
-     * @expectedException \Magento\Framework\Exception\LocalizedException
-     * @expectedExceptionMessage Import is failed. Please see the log report.
+     * @expectedException \Magento\Framework\Exception\RuntimeException
+     * @expectedExceptionMessage Import is failed: Some error
      */
     public function testImportWithException()
     {
@@ -159,28 +158,6 @@ public function testImportWithException()
         $this->importer->import($this->outputMock);
     }
 
-    public function testImportWithLocalizedException()
-    {
-        $exception = new LocalizedException(__('Some error'));
-        $this->outputMock->expects($this->once())
-            ->method('writeln')
-            ->with('<error>Some error</error>');
-        $this->configHashMock->expects($this->never())
-            ->method('regenerate');
-        $this->configValidatorMock->expects($this->never())
-            ->method('isValid');
-        $this->deploymentConfigMock->expects($this->never())
-            ->method('getConfigData');
-        $this->configImporterPoolMock->expects($this->once())
-            ->method('getImporters')
-            ->willThrowException($exception);
-        $this->loggerMock->expects($this->once())
-            ->method('error')
-            ->with($exception);
-
-        $this->importer->import($this->outputMock);
-    }
-
     /**
      * @param array $importers
      * @param bool $isValid