Skip to content

Commit

Permalink
Fix test_code for mock slurm cluster tests
Browse files Browse the repository at this point in the history
Signed-off-by: Fabrice Normandin <[email protected]>
  • Loading branch information
lebrice committed Apr 18, 2024
1 parent 51bc370 commit 78bb594
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions tests/integration/test_code_command.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,8 @@ def test_check_disk_quota(
): # noqa: F811
with caplog.at_level(logging.DEBUG):
check_disk_quota(remote=login_node)
# TODO: Maybe figure out a way to actually test this, (not just by running it and
# expecting no errors).
# Check that it doesn't raise any errors.
# IF the quota is nearly met, then a warning is logged.
# IF the quota is met, then a `MilatoolsUserError` is logged.
Expand All @@ -69,6 +71,10 @@ def test_code(
capsys: pytest.CaptureFixture,
allocation_flags: list[str],
):
if login_node.hostname == "localhost":
pytest.skip(
"TODO: This test doesn't yet work with the slurm cluster spun up in the GitHub CI."
)
home = login_node.run("echo $HOME", display=False, hide=True).stdout.strip()
scratch = login_node.get_output("echo $SCRATCH")
relative_path = "bob"
Expand Down

0 comments on commit 78bb594

Please sign in to comment.