Skip to content
This repository has been archived by the owner on Apr 29, 2021. It is now read-only.

adds ability to output TAP to file and release STDOUT from tests #149

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

harschware
Copy link

This pull request makes bats more amenable to running at command line and for running from inside build systems such as Jenkins.
You can now provide parameters "-t -r -o myresults.tap" and will see any output coming from bats tests or commands that bats tests call that are producing output go directly to STDOUT in real time. The results of the test will be directed to the file myresults.tap and will be in TAP format per the -t parameter.
The effect is that the Jenkins console will contain the output from the test execution and the tap results can be picked up by the TAP plugin by scanning the workspace for the myresults.tap file.

@ztombol ztombol mentioned this pull request May 10, 2016
@harschware
Copy link
Author

I'm hoping the call for maintainers in issue #150 will help work out PR's to include in a 1.0 release, and this PR would be included.

@ztombol ztombol mentioned this pull request Dec 13, 2016
18 tasks
@mstormi
Copy link

mstormi commented Aug 25, 2019

@sstephenson any reason this never made in into bats ?

@LordJashin32
Copy link

LordJashin32 commented Mar 26, 2020

I guess this doesn't fit the purpose of a testing framework in general. If you are going to check the output of the commands anyway. Why use a testing framework that is supposed to check it for you. If you have to use your eyes on some commands to see if the output is good. Then run those maybe outside of bats instead. I think maybe if you want both worlds available to you. Run your commands by themselves (all of them) and output their stdout to a file. And then run bats afterwards.
EDIT: Sorry to bump this from the dead.
Also perhaps making Bats compatible with TAP plugin is not a bad idea. But i think what i said above is the reason the owner doesn't want to merge it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants