MCLOUD-7457: update exit code when error occurs in build:custom:compose (#31)

Author: oshmyheliuk

src/Command/Image/GenerateEs.php

@@ -9,6 +9,7 @@
 
 use Magento\CloudDocker\Cli;
 use Magento\CloudDocker\Filesystem\DirectoryList;
+use Magento\CloudDocker\Filesystem\FileNotFoundException;
 use Magento\CloudDocker\Filesystem\Filesystem;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
@@ -93,6 +94,8 @@ protected function configure(): void
      * Generates data for elasticsearch images.
      *
      * {@inheritDoc}
+     * @throws FileNotFoundException
+     * @throws \Magento\CloudDocker\Filesystem\FileSystemException
      */
     public function execute(InputInterface $input, OutputInterface $output): int
     {

src/Command/Image/GeneratePhp.php

@@ -10,10 +10,10 @@
 use Composer\Semver\Semver;
 use Magento\CloudDocker\App\ConfigurationMismatchException;
 use Magento\CloudDocker\Cli;
-use Magento\CloudDocker\Filesystem\FileNotFoundException;
-use Magento\CloudDocker\Filesystem\Filesystem;
 use Magento\CloudDocker\Compose\Php\ExtensionResolver;
 use Magento\CloudDocker\Filesystem\DirectoryList;
+use Magento\CloudDocker\Filesystem\FileNotFoundException;
+use Magento\CloudDocker\Filesystem\Filesystem;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
@@ -132,6 +132,7 @@ protected function configure(): void
      *
      * @throws ConfigurationMismatchException
      * @throws FileNotFoundException
+     * @throws \Magento\CloudDocker\Filesystem\FileSystemException
      */
     public function execute(InputInterface $input, OutputInterface $output): int
     {
@@ -160,6 +161,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
      * @param string $edition
      * @throws ConfigurationMismatchException
      * @throws FileNotFoundException
+     * @throws \Magento\CloudDocker\Filesystem\FileSystemException
      */
     private function build(string $version, string $edition): void
     {

src/Config/Dist/Generator.php

@@ -120,6 +120,7 @@ private function generateByServices(Config $config): array
      *
      * @param string $filePath
      * @param array $config
+     * @throws FileSystemException
      */
     private function saveConfigDist(string $filePath, array $config): void
     {
@@ -140,6 +141,7 @@ private function saveConfigDist(string $filePath, array $config): void
      * @param array $config
      *
      * @throws ConfigurationMismatchException
+     * @throws FileSystemException
      */
     private function saveConfigEnv(string $filePath, array $config): void
     {

src/Filesystem/Filesystem.php

@@ -15,15 +15,25 @@
 class Filesystem
 {
     /**
-     * Write the contents of a file.
+     * Write contents to file in given path
      *
      * @param string $path
-     * @param string $contents
-     * @return int|bool
+     * @param string $content
+     * @return int The number of bytes that were written.
+     * @throws FileSystemException In case if can write content into file
      */
-    public function put($path, $contents)
+    public function put($path, $content)
     {
-        return file_put_contents($path, $contents);
+        $result = @file_put_contents($path, $content);
+        if (!$result) {
+            throw new FilesystemException(sprintf(
+                'The specified "%s" file could not be written %s',
+                $path,
+                $this->getWarningMessage()
+            ));
+        }
+
+        return $result;
     }
 
     /**
@@ -241,4 +251,19 @@ public function chmod(string $path, int $mode): bool
     {
         return chmod($path, $mode);
     }
+
+    /**
+     * Returns last warning message string
+     *
+     * @return string|null
+     */
+    private function getWarningMessage()
+    {
+        $warning = error_get_last();
+        if ($warning && $warning['type'] == E_WARNING) {
+            return 'Warning!' . $warning['message'];
+        }
+
+        return null;
+    }
 }