simpletest doesn't create a temporary files directory, so temp files during tests get written to the real sites temporary files directory. this is sure to cause hilarity when a simpletest operation eats someone's precious files...

CommentFileSizeAuthor
#5 simpletest_temp_dir.patch2.97 KBAnonymous (not verified)
#2 simpletest_temp_dir.patch2.97 KBAnonymous (not verified)
simpletest_temp_dir.patch1.56 KBAnonymous (not verified)

Comments

Status: Needs review » Needs work

The last submitted patch, , failed testing.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Needs review
StatusFileSize
new simpletest_temp_dir.patch2.97 KB

updated to fix broken test.

Status: Needs review » Needs work

The last submitted patch, , failed testing.

Status: Needs work » Needs review

Re-test of from comment #2406108 was requested by @user.

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new simpletest_temp_dir.patch2.97 KB

ahem, that last patch is clearly wrong, fixed one attached.

chx’s picture

chx commented
Status: Needs review » Reviewed & tested by the community
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD, thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.