Skip to content

Commit

Permalink
Tests: Add instance restart to lifecycle test.
Browse files Browse the repository at this point in the history
Fixes #103.
  • Loading branch information
csadorf authored and danielhollas committed Feb 23, 2023
1 parent d77d438 commit 55205c1
Showing 1 changed file with 5 additions and 9 deletions.
14 changes: 5 additions & 9 deletions tests/test_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -236,15 +236,11 @@ def assert_status_down():
assert_status_up()

# Restart instance.
# TODO: This test is currently disabled, because it is too flaky. For
# a currently unknown reason, the docker client will not be able to
# reach the container anymore to check whether the notebook server is
# online.
# result: Result = runner.invoke(
# cli.cli, ["start", "--no-browser", "--wait=120", "--restart"]
# )
# assert result.exit_code == 0
# assert_status_up()
result: Result = runner.invoke(
cli.cli, ["start", "--no-browser", "--no-pull", "--wait=120", "--restart"]
)
assert result.exit_code == 0
assert_status_up()

# Stop (and remove) instance.
result: Result = runner.invoke(cli.cli, ["stop", "--remove"])
Expand Down

0 comments on commit 55205c1

Please sign in to comment.