Tests: Add instance restart to lifecycle test.

Fixes #103.
aiidalab · Feb 17, 2022 · 955f8ef · 955f8ef
1 parent 7138164
commit 955f8ef
Showing 1 changed file with 8 additions and 9 deletions.
diff --git a/tests/test_cli.py b/tests/test_cli.py
@@ -204,15 +204,14 @@ 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()
+        # This test is flaky, because for a currently unknown reason, the
+        # docker client will undeterministically 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()
 
         # Stop (and remove) instance.
         result: Result = runner.invoke(cli.cli, ["stop", "--remove"])