This repository has been archived by the owner. It is now read-only.

Uncaught exceptions should be highlighted #105

Closed
jodal opened this Issue Feb 23, 2012 · 3 comments

Comments

Projects
None yet
2 participants
@jodal
Member

jodal commented Feb 23, 2012

I feel that the "Uncaught exception: ..." lines in the output of buster test should be more visible, as I've missed them several times and been looking for errors elsewhere when the error has been listed right in front of me.

In the case where you run buster test by hand, it's OK enough, as it's the first thing printed:

$ buster test
Uncaught exception: invalid assignment left-hand side
Firefox 11.0 Linux: ...EEEE
# ... more output ...

When I'm running buster test from a script with a loop and some other output as well, it becomes invisible:

# ... more output ...
Setting up watches.  Beware: since -r was given, this may take a while!
Watches established.
testdata/ CREATE 4913
Uncaught exception: invalid assignment left-hand side
Firefox 11.0 Linux: ...EEEE
# ... more output ...

My suggestion is to make the "Uncaught exception:" text yellow.

@cjohansen

This comment has been minimized.

Member

cjohansen commented Feb 23, 2012

Uncaught exception: invalid assignment left-hand side

What was the cause of that error? I would've imagined the syntax check would stop the tests from running when something like that happened.

@jodal

This comment has been minimized.

Member

jodal commented Feb 24, 2012

To be honest, I can't remember. I scrolled back through my terminal history to find an example, which was some hours old.

@cjohansen

This comment has been minimized.

Member

cjohansen commented Mar 4, 2012

@cjohansen cjohansen closed this Mar 4, 2012

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.