minor #31405 [Dotenv] Test do not use putenv (xuanquynh)

fabpot · fabpot · commit 555280878b86 · 2019-05-07T12:14:10.000+02:00
This PR was merged into the 4.3-dev branch. Discussion ---------- [Dotenv] Test do not use putenv | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | The related pull request is #31062. If the `$usePutenv` flag is set to `false`, `putenv` won't be executed. I just add a small test for this situation. Commits ------- 6d1a76e Test do not use putenv
diff --git a/src/Symfony/Component/Dotenv/Tests/DotenvTest.php b/src/Symfony/Component/Dotenv/Tests/DotenvTest.php
@@ -431,4 +431,14 @@ public function testNoDeprecationWarning()
         $dotenv = new Dotenv(false);
         $this->assertInstanceOf(Dotenv::class, $dotenv);
     }
+
+    public function testDoNotUsePutenv()
+    {
+        $dotenv = new Dotenv(false);
+        $dotenv->populate(['TEST_USE_PUTENV' => 'no']);
+
+        $this->assertSame('no', $_SERVER['TEST_USE_PUTENV']);
+        $this->assertSame('no', $_ENV['TEST_USE_PUTENV']);
+        $this->assertFalse(getenv('TEST_USE_PUTENV'));
+    }
 }
