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

enhaced tap input parsed #15

Merged
merged 3 commits into from Jul 11, 2016
Merged

enhaced tap input parsed #15

merged 3 commits into from Jul 11, 2016

Conversation

bySabi
Copy link
Owner

@bySabi bySabi commented Jul 7, 2016

This commit fix many things but mainly parsed output.
Now we have suite name.

Output Example:

07 07 2016 17:46:24.794:WARN [karma]: No captured browser, open http://localhost:9876/
07 07 2016 17:46:24.807:INFO [karma]: Karma v1.1.0 server started at http://localhost:9876/
07 07 2016 17:46:24.808:INFO [launcher]: Launching browser jsdom with unlimited concurrency
07 07 2016 17:46:24.865:INFO [launcher]: Starting browser jsdom
07 07 2016 17:46:25.310:INFO [Node.js (darwin; U; rv:v6.2.2)]: Connected on socket /#fefarukDGWxaOeikAAAA with id 56437320
TAP version 13
# beep
ok 1 should be equal
ok 2 should be equivalent
# SKIP skipped1
# boop
ok 3 should be equal
ok 4 should be truthy
# fail
not ok 5 (unnamed assert)
  ---
    operator: fail
  ...
# SKIP skipped2

1..5
# tests 5
# pass  4
# fail  1

a caveat, not related to this project, is when used with babel. at of object fail test is not show tape-testing/tape#242

@@ -0,0 +1,8 @@
.DS_Store
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please don't add a .gitignore: these files should be in your global gitignore.

@tmcw
Copy link
Collaborator

tmcw commented Jul 7, 2016

What is this "enhanced" syntax? Is it a standard, is it used by any other tools?

@bySabi
Copy link
Owner Author

bySabi commented Jul 7, 2016

sorry typos and .gitignore.

This PR

  • implement suite and skipped test
  • change parse.js to module tap-parser, additional build step needed.
  • update zuul ripped part
  • update README

I karma we cannot pass protocol version from adapters to reporters thats why I put 'TAP Protocol version 13' on README.

I touch karma-tap-reporter too, dont´t PR yet. From there is the output.

@bySabi
Copy link
Owner Author

bySabi commented Jul 7, 2016

I create a boilerplate repo to see it in action

https://github.com/bySabi/karma--tap--boilerplate

@bySabi
Copy link
Owner Author

bySabi commented Jul 11, 2016

Created karma-tap-pretty-report for prettify TAP output from karma-tap

@tmcw Please review this PR.
Thanks

@tmcw
Copy link
Collaborator

tmcw commented Jul 11, 2016

👍 thanks

@tmcw tmcw merged commit b2b4031 into bySabi:master Jul 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants