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

Deploy Script: Add Unit Tests #8211

Closed
1 task
LilithCole opened this issue Feb 1, 2024 · 1 comment · Fixed by ISISComputingGroup/ibex_utils#190
Closed
1 task

Deploy Script: Add Unit Tests #8211

LilithCole opened this issue Feb 1, 2024 · 1 comment · Fixed by ISISComputingGroup/ibex_utils#190
Assignees
Labels
8 no_release_notes Tickets that do not need release notes, use sparingly! rework

Comments

@LilithCole
Copy link
Contributor

LilithCole commented Feb 1, 2024

As a developer, I would like the deploy script to be tested, so that we can catch potential issues before we start actual deployments. See #8201 and #8203. This ticket is just to setup the framework and possibly some small proof of concept tests

Acceptance Criteria

  • Unit testing framework is available for the deploy script

How to Test

verbose instructions for reviewer to test changes
(Add before making a PR)

@LowriJenkins LowriJenkins added the 8 label Feb 1, 2024
@github-actions github-actions bot added ready and removed proposal labels Feb 1, 2024
@KathrynBaker KathrynBaker added this to the SPRINT_2024_02_01 milestone Feb 1, 2024
@esmith1729 esmith1729 self-assigned this Feb 6, 2024
@esmith1729
Copy link
Contributor

Going to look into different possibilities of unit testing frameworks on batch files: something like this: https://stackoverflow.com/questions/6612415/is-there-a-unit-test-framework-for-windows-batch-files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8 no_release_notes Tickets that do not need release notes, use sparingly! rework
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

5 participants