SimpleTest: Batch API Screen Improvements [#314401]

This issue has been broken off of: #250047: Rework the SimpleTest results interface and clean-up of backend code.

Visual of changes, by webchick: http://img.skitch.com/20080925-n3cpgfgn6jxaw26qtbupncwa92.png.

I say we stick with the changes implemented and save the others for separate issues. Like the grouping can be solved through drill down stuff, but that is separate patch now. :)

Comment	File	Size	Author
#6	simpletest_batch_api_improvements.patch	6.6 KB	boombatower
#6
	simpletest_batch_api_improvements.patch	7.49 KB	boombatower

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

boombatower CreditAttribution: boombatower commented 28 September 2008 at 03:56

Status:

Needs work

» Needs review

Comment #2

boombatower CreditAttribution: boombatower commented 28 September 2008 at 03:58

This patch includes white-space fixes to files, as I would like to get those out of way early instead of in all my splits off patches. Only in files it changes though.

Comment #3

yched CreditAttribution: yched commented 28 September 2008 at 17:25

A generic 'cancelable' feature for batches would definitely be cool - of course, not all batches should be cancelable.

Comment #4

Dries CreditAttribution: Dries commented 28 September 2008 at 23:34

It is weird to have it scroll like that. There is no visual clue for it and not what I expected -- logs _always_ grow at the bottom.

Comment #5

drewish CreditAttribution: drewish commented 29 September 2008 at 08:16

i like the new results being added to the top. the old style was annoying because unless you kept paging down you'd never see the latest results.

Comment #6

boombatower CreditAttribution: boombatower commented 1 November 2008 at 04:57

File	Size
simpletest_batch_api_improvements.patch	6.6 KB

Re-rolled. This is holding up further development.

Comment #7

boombatower CreditAttribution: boombatower commented 1 November 2008 at 05:00

Status:

Needs review

» Reviewed & tested by the community

Comment #8

Dries CreditAttribution: Dries commented 1 November 2008 at 18:33

Status:

Reviewed & tested by the community

» Fixed

Committed to CVS HEAD. The red bar in #6 is confusing me big time. The patch actually applies fine. What is wrong with the test infrastructure? How can we get that fixed?