Merge remote-tracking branch 'origin/MAGETWO-50845-file-permissions' into PR_Branch

Author: monkeysee

setup/src/Magento/Setup/Model/ConfigModel.php

@@ -162,7 +162,7 @@ private function checkInstallationFilePermissions()
     {
         $results = $this->filePermissions->getMissingWritablePathsForInstallation();
         if ($results) {
-            $errorMsg = "Missing write permissions to the following paths: '" . implode("', '", $results) . "'";
+            $errorMsg = "Missing write permissions to the following paths:" . PHP_EOL . implode(PHP_EOL, $results);
             throw new \Exception($errorMsg);
         }
     }

setup/src/Magento/Setup/Model/Installer.php

@@ -452,7 +452,7 @@ public function checkInstallationFilePermissions()
     {
         $results = $this->filePermissions->getMissingWritablePathsForInstallation();
         if ($results) {
-            $errorMsg = "Missing write permissions to the following paths: '" . implode("' '", $results) . "'";
+            $errorMsg = "Missing write permissions to the following paths:" . PHP_EOL . implode(PHP_EOL, $results);
             throw new \Exception($errorMsg);
         }
     }

setup/src/Magento/Setup/Test/Unit/Model/ConfigModelTest.php

@@ -177,7 +177,7 @@ public function testProcessException()
 
     /**
      * @expectedException \Exception
-     * @expectedExceptionMessage Missing write permissions to the following paths: '/a/ro/dir', '/media'
+     * @expectedExceptionMessage Missing write permissions to the following paths:
      */
     public function testWritePermissionErrors()
     {

setup/src/Magento/Setup/Test/Unit/Model/InstallerTest.php

@@ -324,7 +324,7 @@ public function testCheckInstallationFilePermissions()
 
     /**
      * @expectedException \Exception
-     * @expectedExceptionMessage Missing write permissions to the following paths: 'foo' 'bar'
+     * @expectedExceptionMessage Missing write permissions to the following paths:
      */
     public function testCheckInstallationFilePermissionsError()
     {