Skip to content

NO TICKET: make run_bdd.sh useable for Windows | ensure on main branch#229

Merged
jirhiker merged 3 commits into
stagingfrom
jab-run-bdd-fix-for-different-platforms
Nov 4, 2025
Merged

NO TICKET: make run_bdd.sh useable for Windows | ensure on main branch#229
jirhiker merged 3 commits into
stagingfrom
jab-run-bdd-fix-for-different-platforms

Conversation

@jacob-a-brown

Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem / context:

  • some of the functionality in run_bdd.sh is only available for Unix and not Windows
  • we need to ensure that we are on the main branch before git pulling

How

Implementation summary - the following was changed / added / removed:

  • Use if statements to determine which functions to use

Notes

Any special considerations, workarounds, or follow-up work to note?

  • Incase someone is working on OcotilloBDD, should we record the branch that is currently checked out, stash the changes, run the tests, then go back to that branch and apply the stash?
# Save current branch name
CURRENT_BRANCH=$(git rev-parse --abbrev-ref HEAD)

# Stash any uncommitted changes
git stash push -u -m "Auto-stash before BDD run"

# Checkout main and update
git checkout main
git pull

# ... (your BDD test steps here) ...

# Return to your original branch
git checkout "$CURRENT_BRANCH"

# Apply stashed changes if any
if git stash list | grep -q "Auto-stash before BDD run"; then
  git stash pop
fi

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.
see 7 files with indirect coverage changes

@jirhiker jirhiker merged commit 2093d3e into staging Nov 4, 2025
6 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the jab-run-bdd-fix-for-different-platforms branch February 5, 2026 18:09
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.

3 participants