In simpletest_uninstall() the removed variables are out of date and some of the newer ones are not removed at all.

CommentFileSizeAuthor
#7 500270-simpletest-uninstall.patch834 bytesboombatower
#5 500270-simpletest-uninstall.patch839 bytesboombatower
#4 500270-simpletest-uninstall.patch881 bytesboombatower
#1 500270-simpletest-uninstall.patch811 bytesboombatower

Comments

boombatower’s picture

boombatower commented
Assigned: Unassigned » boombatower
Status: Active » Needs review
StatusFileSize
new 500270-simpletest-uninstall.patch811 bytes

Could not find a reference to simpletest_devel anywhere in Drupal 7.

simpletest_httpauth, simpletest_httpauth_username, simpletest_httpauth_pass are from the old httpauth setup.

simpletest_clear_results was not removed

Status: Needs review » Needs work

The last submitted patch failed testing.

boombatower’s picture

boombatower commented
Status: Needs work » Needs review

Test slave crapped.

boombatower’s picture

boombatower commented
StatusFileSize
new 500270-simpletest-uninstall.patch881 bytes

Updated per: #500292: Provide a settings page for hidden SimpleTest variables.

boombatower’s picture

boombatower commented
StatusFileSize
new 500270-simpletest-uninstall.patch839 bytes

Other issue renamed variables.

Status: Needs review » Needs work

The last submitted patch failed testing.

boombatower’s picture

boombatower commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new 500270-simpletest-uninstall.patch834 bytes

Extremely simple so I'll mark it for RTBC.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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