If you run Simpletests locally, you have probably noticed that it leaves a lot of empty directories behind, e.g. sites/default/files/simpletest/12345 (with different numbers every time). These are not removed by pressing the “Clean environment” button.

This regression was introduced in #443200: User pictures does not work with private files (by me - oops) and in #517814: File API Stream Wrapper Conversion, because the patches for these issues were made a long time ago and then rerolled without noticing the change in #512104: Simpletest breaks high volume sites.

This patch also addresses the issue raised in #555364: Set a testing temp directory as well as public/private.

CommentFileSizeAuthor
simpletest-clean-filesystem-1.patch7.73 KBc960657

Comments

Status: Needs review » Needs work

The last submitted patch failed testing.

c960657’s picture

Status: Needs work » Needs review

Hmm, I cannot reproduce the test failure. Any ideas, anyone?

chx’s picture

#577398: Ensure test coverage of simpletest temporary file handling please check the patch there and if this one is the better then make that duplicate etc

boombatower’s picture

Status: Needs review » Closed (duplicate)