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

Initial Bats scaffold (DO NOT MERGE) #1170

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hyperupcall
Copy link
Collaborator

@hyperupcall hyperupcall commented Oct 3, 2024

The purpose of this PR is to better show the additions that might be made to incorporate Bats, a testing framework for Bash. More information about this PR can be found in this comment.

To run this code, be sure to:

  1. Install Bats (see instructions on Bats website)
  2. Switch to this branch (this branch adds a git submodule, hyperupcall/bats-all, be sure to update submodules (i think that git submodule update --init should do the trick))
  3. Now you can run bats ./tests

When executing Bats, Bats will look for files inside of ./tests that end with .bats. If it sees any, it "executes" it. In this PR, there are two .bats files used to demonstrate a few features of Bats.

@hyperupcall hyperupcall marked this pull request as draft October 3, 2024 08:19
@hyperupcall
Copy link
Collaborator Author

@spacewander Hey question separate from this pull request - what do you think about closing pull requests that haven't had activity in 1 year? I wanted to close old pull requests to make it easier to keep track of what might actually be merged, but I wanted to get your OK first

@spacewander
Copy link
Collaborator

@spacewander Hey question separate from this pull request - what do you think about closing pull requests that haven't had activity in 1 year? I wanted to close old pull requests to make it easier to keep track of what might actually be merged, but I wanted to get your OK first

LGTM. Let's use the stale action to do it. Robot is more suitable to do this job than human.

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.

2 participants