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?

Comments

webchick’s picture

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..?

Rok Žlender’s picture

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.

yched’s picture

+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...

Rok Žlender’s picture

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

boombatower’s picture

Category: bug » feature
Status: Active » Postponed (maintainer 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.

boombatower’s picture

Project: SimpleTest » Drupal core
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.

sun.core’s picture

Status: Postponed (maintainer needs more info) » Closed (won't fix)

Sorry, without further information this issue can only be marked as won't fix.

Feel free to re-open this issue if you want to provide further information. Thanks.