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

Remove bats as a requirements for testing #320

Open
underwoo opened this issue Oct 25, 2024 · 0 comments · May be fixed by #322
Open

Remove bats as a requirements for testing #320

underwoo opened this issue Oct 25, 2024 · 0 comments · May be fixed by #322
Assignees

Comments

@underwoo
Copy link
Member

We have had issues with bats in the past (leaving long-running processes when a test fails), and while getting native TAP output is nice, debugging failing tests was challenging -- especially when the script has multiple tests.

Another issue users who build/instal FRE-NCtools may not know that test will only run if bats is installed, and in the user's PATH -- as configure.ac doesn't fail if bats is not found. We should rewrite the tests not to use bats.

@underwoo underwoo self-assigned this Oct 25, 2024
@underwoo underwoo linked a pull request Oct 25, 2024 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant