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. :)

Comments

boombatower’s picture

Status: Needs work » Needs review
boombatower’s picture

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.

yched’s picture

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

dries’s picture

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.

drewish’s picture

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.

boombatower’s picture

StatusFileSize
new6.6 KB

Re-rolled. This is holding up further development.

boombatower’s picture

Status: Needs review » Reviewed & tested by the community
dries’s picture

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?

dave reid’s picture

Marked #325824: Notice: Undefined index: test_id in _simpletest_batch_finished() fixed due thanks to this patch being committed!

boombatower’s picture

@Dries: appears to be issue on t.d.o which occurred recently. We are investigating, but for the time being server is down.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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