Skip to content

Commit

Permalink
minor #34225 [HttpFoundation] MockFileSessionStorageTest::sessionDir …
Browse files Browse the repository at this point in the history
…being used after it's unset (HypeMC)

This PR was merged into the 3.4 branch.

Discussion
----------

[HttpFoundation] MockFileSessionStorageTest::sessionDir being used after it's unset

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

The `$sessionDir` property was used after being set to `null`.

Commits
-------

51b20dd Fix MockFileSessionStorageTest::sessionDir being used after it's unset
  • Loading branch information
fabpot committed Nov 3, 2019
2 parents 36c4c66 + 51b20dd commit 85bcd9d
Showing 1 changed file with 3 additions and 3 deletions.
Expand Up @@ -41,12 +41,12 @@ protected function setUp()

protected function tearDown()
{
$this->sessionDir = null;
$this->storage = null;
array_map('unlink', glob($this->sessionDir.'/*.session'));
array_map('unlink', glob($this->sessionDir.'/*'));
if (is_dir($this->sessionDir)) {
rmdir($this->sessionDir);
}
$this->sessionDir = null;
$this->storage = null;
}

public function testStart()
Expand Down

0 comments on commit 85bcd9d

Please sign in to comment.