cutest icon indicating copy to clipboard operation
cutest copied to clipboard

Bail on first error

Open gtramontina opened this issue 9 years ago • 7 comments

Currently cutest exits when the first error occur. Would you be interested in adding a flag -b to make that optional? The reason is that sometimes it is useful to get all errors at once.

gtramontina avatar May 22 '15 02:05 gtramontina

+1

I think cutest should keep the default behaviour of failing fast. It's very useful while developing.

Running all tests regardless of there being failures would be very useful for running automated tests on a CI server for example.

britishtea avatar May 22 '15 15:05 britishtea

Cool. Yeah, I agree with you. And keeping the default behaviour makes the change backwards compatible. I'll invert the flag (or how should it be called then?).

gtramontina avatar May 22 '15 15:05 gtramontina

@djanowski any comments here? I can just flip the flag if you're ok going ahead with #28.

gtramontina avatar May 28 '15 05:05 gtramontina

@gtramontina The flag can be -c (continue).

djanowski avatar Dec 04 '15 12:12 djanowski

Hard to come up with the right name. I was thinking maybe -f (force) or -a (all).

soveran avatar Dec 04 '15 13:12 soveran

-a for all seems sensible.

I think the usage string (in the executable) should be clearer, though. [-a] is not very clear on what it means.

britishtea avatar Dec 04 '15 15:12 britishtea

We already have -s and -o to filter what Cutest should run. -a for "all" seems to compete with that.

I'm still inclined for -c (continue).

djanowski avatar Dec 09 '15 15:12 djanowski