Skip to content

Initial Bats scaffold (DO NOT MERGE)#1170

Closed
hyperupcall wants to merge 1 commit intotj:mainfrom
hyperupcall:hyperupcall-bats
Closed

Initial Bats scaffold (DO NOT MERGE)#1170
hyperupcall wants to merge 1 commit intotj:mainfrom
hyperupcall:hyperupcall-bats

Conversation

@hyperupcall
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

@hyperupcall
Copy link
Copy Markdown
Collaborator Author

Closed in favor of #1187

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