I was convinced on IRC that we need the old way of running setUp and tearDown for each method instead of each test class. That's why they exist in the first place. And this is how old testing lib worked.

CommentFileSizeAuthor
setuppermethod.patch838 byteschx

Comments

R.Muilwijk’s picture

R.Muilwijk commented

+1 for this patch. For all the methods for example in cache tests I had to add a resetter function.

Crell’s picture

Crell commented
Status: Needs review » Reviewed & tested by the community

I can confirm the bug (I'm the one who growled at chx about it in IRC after the database tests broke for the 47th time :-) ) and that this patch fixes it.

catch’s picture

catch
he/him
Primary language English
commented

running all tests is fine too.

dries’s picture

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

Ran all tests before and after and the tests continued to work. Committed to CVS HEAD.

yched’s picture

yched commented

Not sure where simpletest D6 lies on this regard ? Does this need backporting ?

boombatower’s picture

boombatower commented

It was never changed so it still works this way, although there are a number of things that would be helpful back-porting. Already open issue #274542: Back-port the batch API patch & related imporvements.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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