http://drupal.org/node/243773

It's too awesome to not be in 6.x too. :)

CommentFileSizeAuthor
#10 simpletest_7-6_backport.patch255.72 KBboombatower

Comments

boombatower’s picture

Title: Back-port the batch API patch » Back-port the batch API patch & related imporvements

We should make sure we also port much of the added setUp stuff that was fixed before batch API.

moshe weitzman’s picture

Anyone working on this? Sure would be swell for contrib and custom modules.

boombatower’s picture

Not at current. There were a few more issues I'd like to see committed first, but anyone is free to work on this. Should be fairly straight forward since the base DrupalWebTestCase has been back-ported.

boombatower’s picture

The batch API patch rewrites much of the simpletest.module and removes the dependency of the simpletest library. I will look into it, but it may require that we drop support for old format of tests. Which would also alleviate some confusions, but would fry a number of tests.

boombatower’s picture

Assigned: Unassigned » boombatower

I'm looking into this.

moshe weitzman’s picture

Personally, I think it is OK to break some existing tests. Interested parties can update them as needed. If you make a new branch, you won't disturb people relying on current behavior.

boombatower’s picture

I'll make a d6 2.x branch?

webchick’s picture

Yeah. D6 2.x for the backwards-compatibility breaking stuff.

boombatower’s picture

I have it working. Waiting for CVS to come back in order to make patch.

boombatower’s picture

Status: Active » Needs review
StatusFileSize
new255.72 KB

Patch to checkout.

boombatower’s picture

Status: Needs review » Fixed

Committed.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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