ENGCOM-2552: [Backport] Improve "Invalid country code" error message on tax import #17160

Stanislav Idolov · web-flow · commit 302712a0ff72 · 2018-07-26T18:48:04.000+03:00
diff --git a/app/code/Magento/TaxImportExport/Model/Rate/CsvImportHandler.php b/app/code/Magento/TaxImportExport/Model/Rate/CsvImportHandler.php
@@ -234,7 +234,7 @@ protected function _importRate(array $rateData, array $regionsCache, array $stor
         $countryCode = $rateData[1];
         $country = $this->_countryFactory->create()->loadByCode($countryCode, 'iso2_code');
         if (!$country->getId()) {
-            throw new \Magento\Framework\Exception\LocalizedException(__('One of the countries has invalid code.'));
+            throw new \Magento\Framework\Exception\LocalizedException(__('Country code is invalid: %1', $countryCode));
         }
         $regionsCache = $this->_addCountryRegionsToCache($countryCode, $regionsCache);
 
diff --git a/app/code/Magento/TaxImportExport/i18n/en_US.csv b/app/code/Magento/TaxImportExport/i18n/en_US.csv
@@ -12,7 +12,7 @@ Rate,Rate
 "Invalid file upload attempt","Invalid file upload attempt"
 "Invalid file upload attempt.","Invalid file upload attempt."
 "Invalid file format.","Invalid file format."
-"One of the countries has invalid code.","One of the countries has invalid code."
+"Country code is invalid: %1","Country code is invalid: %1"
 "Import Tax Rates","Import Tax Rates"
 "Export Tax Rates","Export Tax Rates"
 CSV,CSV
diff --git a/dev/tests/integration/testsuite/Magento/TaxImportExport/Model/Rate/CsvImportHandlerTest.php b/dev/tests/integration/testsuite/Magento/TaxImportExport/Model/Rate/CsvImportHandlerTest.php
@@ -57,7 +57,7 @@ public function testImportFromCsvFileWithCorrectData()
     /**
      * @magentoDbIsolation enabled
      * @expectedException \Magento\Framework\Exception\LocalizedException
-     * @expectedExceptionMessage One of the countries has invalid code.
+     * @expectedExceptionMessage Country code is invalid: ZZ
      */
     public function testImportFromCsvFileThrowsExceptionWhenCountryCodeIsInvalid()
     {

Original file line number	Diff line number	Diff line change
`@@ -234,7 +234,7 @@ protected function _importRate(array $rateData, array $regionsCache, array $stor`
`234`	`234`	`$countryCode = $rateData[1];`
`235`	`235`	`$country = $this->_countryFactory->create()->loadByCode($countryCode, 'iso2_code');`
`236`	`236`	`if (!$country->getId()) {`
`237`		`- throw new \Magento\Framework\Exception\LocalizedException(__('One of the countries has invalid code.'));`
	`237`	`+ throw new \Magento\Framework\Exception\LocalizedException(__('Country code is invalid: %1', $countryCode));`
`238`	`238`	`}`
`239`	`239`	`$regionsCache = $this->_addCountryRegionsToCache($countryCode, $regionsCache);`
`240`	`240`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ public function testImportFromCsvFileWithCorrectData()`
`57`	`57`	`/**`
`58`	`58`	`* @magentoDbIsolation enabled`
`59`	`59`	`* @expectedException \Magento\Framework\Exception\LocalizedException`
`60`		`- * @expectedExceptionMessage One of the countries has invalid code.`
	`60`	`+ * @expectedExceptionMessage Country code is invalid: ZZ`
`61`	`61`	`*/`
`62`	`62`	`public function testImportFromCsvFileThrowsExceptionWhenCountryCodeIsInvalid()`
`63`	`63`	`{`