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

Updated rmg_runner.py and test_rmg_runner.py #86

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

calvinp0
Copy link
Member

rmg_runner.py: File has been updated with assertion checks to ensure the user is correctly choosing memory if they specifiy
test_rmg_runner.py: Test has descriptions and also checks for assertion errors when the memory is incorrectly specified.

rmg_runner.py: File has been updated with assertion checks to ensure the user is correctly choosing memory if they specifiy
test_rmg_runner.py: Test has descriptions and also checks for assertion errors when the memory is incorrectly specified.
tests/test_rmg_runner.py Fixed Show fixed Hide fixed
tests/test_rmg_runner.py Fixed Show fixed Hide fixed
@calvinp0 calvinp0 self-assigned this Dec 18, 2022
test_rmg_runner.py: Updated the class TestWriteSubmitScript with a function that runs executes t3 as a 'local' job, and then after 'converging' the function will ensure that the job.sh and submit.sub files are written as expected.
os.remove error that there was no job.sh to remove. Therefore, removed the function.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant