Transfer bats#142 - This commit corrects the problem of incorrect TAP format for skipped tests #19
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
format for skipped tests.
Separate "skip" from the test name (description) to be distinct from "skip" as part of the TAP Directive.
mbland
suggested changes
Oct 3, 2017
test/bats.bats
Outdated
| } | ||
|
|
||
| @test "tap passing and skipping tests" { | ||
| run filter_control_sequences bats --tap $FIXTURE_ROOT/passing_and_skipping.bats |
There was a problem hiding this comment.
@agent-0028 You good to make this change?
There was a problem hiding this comment.
Indeed. I'll take care of it.
mbland
approved these changes
Oct 3, 2017
mbland
added a commit
to mbland/go-script-bash
that referenced
this pull request
Jun 10, 2018
https://github.com/bats-core/bats-core/releases/tag/v1.0.0, while honoring the same interface as v0.4.0, introduced a few changes that required some cleanup of our Bats utilities: - bin/bats is no longer a symlink to libexec/bats, and the latter can no longer be invoked directly; hence _GO_BATS_PATH is now set to bin/bats. - TAP output for skipped tests changed in bats-core/bats-core#19. - run_bats_test_suite_in_isolation depended upon _GO_ROOTDIR and _GO_BATS_PATH, which led to bats not being able to find its libexec scripts. The bats path is now computed using `command -v bats`, which makes the library itself more portable. - Some assertion-test-helpers output didn't end with a newline. This prevented the `while` loop used to replace `sed` in bats-core/bats-core#88 from reading the final line of output from the test. While this was fixed in bats-core/bats-core#99, updating the code to always emit newlines was still the right thing to do. There was a bug in Bats v1.0.0 in setting `BATS_CWD` that broke the "assertion-test-helpers: failing assertion must disable shell options" test case. Specifically, stack trace paths that should've looked like: tests/assertion-test-helpers.bash instead looked like: go-script-bash/tests/assertion-test-helpers.bash This was fixed by bats-core/bats-core#98.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
todo
This transfers sstephenson/bats#142 to bats-core and makes "skipped" test output compliant to TAP spec