Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
simpletest.module
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
28 Sep 2008 at 03:56 UTC
Updated:
15 Nov 2008 at 18:51 UTC
Jump to comment: Most recent file
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 |
| simpletest_batch_api_improvements.patch | 7.49 KB | boombatower |
Comments
Comment #1
boombatower commentedComment #2
boombatower commentedThis 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 commentedA generic 'cancelable' feature for batches would definitely be cool - of course, not all batches should be cancelable.
Comment #4
dries commentedIt 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 commentedi 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 commentedRe-rolled. This is holding up further development.
Comment #7
boombatower commentedComment #8
dries commentedCommitted 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?
Comment #9
dave reidMarked #325824: Notice: Undefined index: test_id in _simpletest_batch_finished() fixed due thanks to this patch being committed!
Comment #10
boombatower commented@Dries: appears to be issue on t.d.o which occurred recently. We are investigating, but for the time being server is down.
Comment #11
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.