Mute OK messages by default

ezyang - January 19, 2008 - 20:11
Project:Drupal
Version:7.x-dev
Component:simpletest.module
Category:feature request
Priority:normal
Assigned:Unassigned
Status:active (needs more info)
Description

While having a full screen of "OK" gives one a nice fuzzy feeling, it's not so great when you've got a single test that has started failing and you now need to find exactly which one it is. I propose, just as the default SimpleTest reporter does, that Drupal's SimpleTest reporter not display passes. Comments?

#1

webchick - January 20, 2008 - 18:23

Hmmm. The only thing is that it's nice to have the context around the fails to know exactly what you were trying to do when whatever it was didn't work.

Maybe a set of "jump-down" links that skip directly to the failures themselves..?

#2

Rok Žlender - January 21, 2008 - 06:21

Ok msgs can you give you a trail where exaclty did your test fail if you get a general msg like
[browser] Setting title="!SimpleTest test node! simpletest_y1ouaeAn_E" at [/Applications/MAMP/htdocs/drupalHEAD/sites/all/modules/simpletest/drupal_test_case.php line 204]

You have no idea where this is. I am in favor of partitioning test results even more. ATM we display test results for one test class together. It would help a lot if we would display it separately for every test function in a test class.

#3

yched - January 21, 2008 - 09:47

+1 for partitioning.
Having an option to hide passes would be nice too.

I sort of miss the ability to have 'neutral' information messages (neither pass nor fail), to ease tracking progress and spot in which part of a test an error happens. Not sure the simpletest framework itself allows that, though...

#4

Rok Žlender - January 21, 2008 - 22:13

Simpletest does support partitioning down to function level with paintMethodStart and paintMethodEnd. It shouldnt be too hard to implement.

#5

boombatower - March 18, 2008 - 00:34
Category:bug report» feature request
Status:active» active (needs more info)

This has been stagnant for quiet some time. A consensus needs to be reached on this issue.

This isn't a bug report, it is a feature request.

#6

boombatower - May 15, 2008 - 01:50
Project:SimpleTest» Drupal
Version:6.x-1.x-dev» 7.x-dev
Component:User interface» simpletest.module

A consensus needs to be reached on this issue.

I don't think it is very useful. I prefer to see context.

 
 

Drupal is a registered trademark of Dries Buytaert.