-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add TAP formatter to spec suite #6286
Conversation
3719fea
to
40f0e40
Compare
|
||
def report(result) | ||
case result.kind | ||
when :success |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be when :success, :pending
. And the when :pending
should be removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, that would reduce visibility. Separate entries makes it obvious what we're looking at.
Might want to think about printing the version line, but it's not required for this version of the spec. |
@RX14 I had the idea to attach some diagnostics (location, failure message) to the test reports similar to this: https://github.com/MakeNowJust/tap/blob/master/spec/fizzbuzz.txt |
c43793b
to
0659bab
Compare
Rebased to squash changes in logical commits and trigger CI-rebuild. |
This PR adds a formatter for test anything protocol (TAP) to
Spec
.The first two commits refactor some aspects of spec formatter implementation, the third finally implements
TAPFormatter
.STDOUT
ti make it easier to test them.DotFormatter
andVerboseFormatter