feature #810 Use the extra.runtime.test_envs composer variabels in the symfony:dump-env command (cmodijk)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Use the extra.runtime.test_envs composer variabels in the symfony:dump-env command

Added the option to read the `extra.runtime.test_envs` to load the test envs during the environment dump. This is my attempt to fix #747.

Commits
-------

6c3655b Added the option to read the `extra.runtime.test_envs` to load the test envs during the environment dump

Author: nicolas-grekas

src/Command/DumpEnvCommand.php

@@ -13,6 +13,7 @@
 
 use Composer\Command\BaseCommand;
 use Composer\Config;
+use Composer\Factory;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
@@ -111,13 +112,16 @@ private function loadEnv(string $path, ?string $env): array
                 throw new \RuntimeException('Please provide the name of the environment either by passing it as command line argument or by defining the "APP_ENV" variable in the ".env.local" file.');
             }
 
+            $testEnvs = $config['extra']['runtime']['test_envs'] ?? ['test'];
+
             if (method_exists($dotenv, 'loadEnv')) {
-                $dotenv->loadEnv($path);
+                $config = @json_decode(file_get_contents(Factory::getComposerFile()), true);
+                $dotenv->loadEnv($path, 'APP_ENV', 'dev', $testEnvs);
             } else {
                 // fallback code in case your Dotenv component is not 4.2 or higher (when loadEnv() was added)
                 $dotenv->load(file_exists($path) || !file_exists($p = "$path.dist") ? $path : $p);
 
-                if ('test' !== $env && file_exists($p = "$path.local")) {
+                if (!\in_array($env, $testEnvs, true) && file_exists($p = "$path.local")) {
                     $dotenv->load($p);
                 }
 

tests/Command/DumpEnvCommandTest.php

@@ -164,6 +164,46 @@ public function testDoesNotRequireToSpecifyEnvArgumentWhenLocalFileIsPresent()
         unlink($envLocalPhp);
     }
 
+    public function testLoadLocalEnvWhenTestEnvIsNotEqual()
+    {
+        @mkdir(FLEX_TEST_DIR);
+        $env = FLEX_TEST_DIR.'/.env';
+        $envLocal = FLEX_TEST_DIR.'/.env.local';
+        $envLocalPhp = FLEX_TEST_DIR.'/.env.local.php';
+        $composer = __DIR__.'/../../composer.json';
+        @unlink($envLocalPhp);
+
+        file_put_contents($env, 'APP_ENV=dev');
+        $envContent = <<<EOF
+APP_ENV=test
+APP_SECRET=abcdefgh123456789
+EOF;
+        file_put_contents($envLocal, $envContent);
+
+        copy($composer, FLEX_TEST_DIR.'/composer-backup.json');
+        $composerContent = @json_decode(file_get_contents($composer), true);
+        $composerContent['extra']['runtime']['test_envs'] = [];
+        file_put_contents($composer, json_encode($composerContent));
+
+        $command = $this->createCommandDumpEnv();
+        $command->execute([
+            'env' => 'test',
+        ]);
+
+        $this->assertFileExists($envLocalPhp);
+
+        $vars = require $envLocalPhp;
+        $this->assertSame([
+            'APP_ENV' => 'test',
+            'APP_SECRET' => 'abcdefgh123456789',
+        ], $vars);
+
+        unlink($env);
+        unlink($envLocal);
+        unlink($envLocalPhp);
+        copy(FLEX_TEST_DIR.'/composer-backup.json', $composer);
+    }
+
     private function createCommandDumpEnv()
     {
         $command = new DumpEnvCommand(