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.

Sign up for GitHub

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

Jump to bottom

Unit tests for benchcab.py #250

Open
abhaasgoyal opened this issue Jan 31, 2024 · 0 comments
Open

Unit tests for benchcab.py #250

abhaasgoyal opened this issue Jan 31, 2024 · 0 comments
Labels
priority:low Low priority Issues that do not need to be addressed in the near future. testing Add unit/integration tests

Comments

@abhaasgoyal
Copy link

abhaasgoyal commented Jan 31, 2024 •
edited
Loading

Currently, unit tests do not exist for benchcab.py, which run many sub-modules. Since some of these functions heavily depend on Gadi environment, it'd be great if we could mock the environment within the testing module.

Related as an extension to #246 , where tests for $PROJECT would be converted as unit

Notes:
test_benchcab.py used to exist for running PBS scripts - removed in 50dd9b5
@abhaasgoyal abhaasgoyal added the testing Add unit/integration tests label Jan 31, 2024
@ccarouge ccarouge added the priority:low Low priority Issues that do not need to be addressed in the near future. label May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority:low Low priority Issues that do not need to be addressed in the near future. testing Add unit/integration tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ccarouge @abhaasgoyal