Skip to content
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

Update TAP reporter to TAP 13 #2149

Closed
lencioni opened this issue Mar 10, 2016 · 1 comment · Fixed by #3552
Closed

Update TAP reporter to TAP 13 #2149

lencioni opened this issue Mar 10, 2016 · 1 comment · Fixed by #3552
Labels
status: accepting prs Mocha can use your help with this one! type: feature enhancement proposal

Comments

@lencioni
Copy link

I noticed that mocha is using the older TAP specification and wondering if you would be open to updating it to TAP 13? It is pretty similar. I think the main differences are that version 13 starts with a line that says TAP version 13, and encodes test output in YAML.

General format:

TAP version 13
1..N
ok 1 Description # Directive
# Diagnostic
  ---
  message: 'Failure message'
  severity: fail
  data:
    got:
      - 1
      - 3
      - 2
    expect:
      - 1
      - 2
      - 3
  ...
ok 47 Description
ok 48 Description
more tests....
@cades
Copy link

cades commented Jun 18, 2016

+1 for TAP 13
with YAML, multiline error message can be handled correctly by testem test runner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: accepting prs Mocha can use your help with this one! type: feature enhancement proposal
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants