Bug #32723
Files unit tests create but don't remove a lot of temporary files
| Status: | Resolved | Start date: | 2011-12-20 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | - | % Done: | 100% |
|
| Category: | - Testing - | |||
| Target version: | TYPO3 Flow Base Distribution - 1.1 beta 1 | |||
| PHP Version: | Complexity: | easy | ||
| Has patch: | No | Affected Flow version: | Git master |
Description
The tests in TYPO3\FLOW3\Tests\Unit\Utility\Files generate various temporary files through PHP's tempnam() function.
However, these files are never removed again, which led to our CI server having > 400,000 files in /tmp
The test case should use a subdirectory in $TMP and have a teardown method which removes that subdirectory recursively.
Associated revisions
[BUGFIX] Unit tests remove their temporary files
Some unit tests create many temporary files outside vfs://
that are now removed after running the test.
Change-Id: I758ff86bb969fe11c734fc7dd9bba45f3b4b21c5
Fixes: #32723
Releases: 1.0, 1.1
[BUGFIX] Unit tests remove their temporary files
Some unit tests create many temporary files outside vfs://
that are now removed after running the test.
Change-Id: I889d72649b4a55c54c96799cbe0768e9362e6110
Fixes: #32723
Releases: 1.0, 1.1
History
#1 Updated by Bastian Waidelich over 3 years ago
- Status changed from New to Accepted
- Assigned To set to Bastian Waidelich
- Target version set to 1.1 beta 1
#2 Updated by Bastian Waidelich over 3 years ago
- Assigned To deleted (
Bastian Waidelich)
#3 Updated by Gerrit Code Review over 3 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7433
#4 Updated by Gerrit Code Review over 3 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7433
#5 Updated by Gerrit Code Review over 3 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7433
#6 Updated by Adrian Föder over 3 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 78a7482ac2927fc3898e25bf6edf011da4255240.
#7 Updated by Gerrit Code Review over 3 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch FLOW3-1.0 has been pushed to the review server.
It is available at http://review.typo3.org/7439
#8 Updated by Karsten Dambekalns over 3 years ago
- Status changed from Under Review to Resolved
#9 Updated by Gerrit Code Review over 3 years ago
- Status changed from Resolved to Under Review
Patch set 2 for branch FLOW3-1.0 has been pushed to the review server.
It is available at http://review.typo3.org/7439
#10 Updated by Adrian Föder over 3 years ago
- Status changed from Under Review to Resolved
Applied in changeset f95aa8385b91967eb342513d08dbdbc4f87cb38e.