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

TAP-compatible test function #4

Closed
wants to merge 4 commits into from
Closed

TAP-compatible test function #4

wants to merge 4 commits into from

Conversation

pocomane
Copy link

@pocomane pocomane commented Mar 6, 2018

Some note on the code:

  • I changed the one-argument mode to implement the "Tap diagnostic" since Tap have not the concept of "Suite". However, in my previous implementation, Suite was just logging so, you can reproduce it with Tap diagnostic.

  • Using taptest to test itself make the example a bit confusing. I tryed as much ad possible to clearify the example. Maybe it is better to write a simplier one?

  • The example write another 'ok X' line after ending plan summary. This is not allowed in standard TAP, but for now I can not guess how get rid of it. IIt is just an issue of the example, I hope your CI scripts will accept it.

  • For now, no argument type check are performed

  • I put it in the sys directory, maybe you have some better solution

@pocomane
Copy link
Author

pocomane commented Mar 6, 2018

Indeed the CI complains about the line after the ending plan :(

@pocomane
Copy link
Author

pocomane commented Mar 6, 2018

Sorry for hammering. At the end, I disabled the final faliures count, which I found to be very useful when looking at the test report without an harnesser. However it is still failing due to the test masking trick. :(

@pocomane pocomane closed this Mar 6, 2018
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

1 participant