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

Replace bats embed with bats-core submodule #241

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sparr
Copy link
Contributor

@sparr sparr commented Jun 20, 2023

bats was archived two years ago and bats-core is the active community-maintained fork. This PR is motivated by significantly increased functionality in the newer version, including -f to filter and run specific tests, options to control test output for debugging, additional warnings (which have identified one already-broken test which I'll fix in a PR and link shortly), and much more.

Description of changes:
Remove test/bats/* and add a new submodule that checks out https://github.com/bats-core/bats-core to test/bats-core.

Add Makefile target that checks out the submodule if necessary for make test

Relevant doc and comment updates.

Also update test Github workflow to allow manual dispatch.

Also minor formatting updates to CONTRIBUTING.md.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sparr
Copy link
Contributor Author

sparr commented Jun 20, 2023

Failures on tests 21 and 23, and all but one warning, are expected and will be fixed by #221
Remaining warning will be fixed by #242

Signed-off-by: Clarence "Sparr" Risher <[email protected]>
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.

1 participant