minor #35056 Fixed test added in #35022 (GrahamCampbell)

nicolas-grekas · nicolas-grekas · commit 412fcf5463cb · 2019-12-26T08:58:50.000+01:00
This PR was merged into the 3.4 branch. Discussion ---------- Fixed test added in #35022 | Q | A | ------------- | --- | Branch? | master for features / 3.4, 4.3, 4.4 or 5.0 for bug fixes  | Bug fix? | yes/no | New feature? | yes/no  | Deprecations? | yes/no  | Tickets | Fix #...  | License | MIT | Doc PR | symfony/symfony-docs#...   Commits ------- 6eeec7c Fixed test added in #35022
diff --git a/src/Symfony/Component/Dotenv/Tests/DotenvTest.php b/src/Symfony/Component/Dotenv/Tests/DotenvTest.php
@@ -320,9 +320,12 @@ public function testGetVariablesValueFromGetenv()
         putenv('Foo=Bar');
 
         $dotenv = new Dotenv(true);
-        $values = $dotenv->parse('Foo=${Foo}');
-        $this->assertSame('Bar', $values['Foo']);
 
-        putenv('Foo');
+        try {
+            $values = $dotenv->parse('Foo=${Foo}');
+            $this->assertSame('Bar', $values['Foo']);
+        } finally {
+            putenv('Foo');
+        }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -320,9 +320,12 @@ public function testGetVariablesValueFromGetenv()`
`320`	`320`	`putenv('Foo=Bar');`
`321`	`321`
`322`	`322`	`$dotenv = new Dotenv(true);`
`323`		`- $values = $dotenv->parse('Foo=${Foo}');`
`324`		`- $this->assertSame('Bar', $values['Foo']);`
`325`	`323`
`326`		`- putenv('Foo');`
	`324`	`+ try {`
	`325`	`+ $values = $dotenv->parse('Foo=${Foo}');`
	`326`	`+ $this->assertSame('Bar', $values['Foo']);`
	`327`	`+ } finally {`
	`328`	`+ putenv('Foo');`
	`329`	`+ }`
`327`	`330`	`}`
`328`	`331`	`}`