More test coverage

Author: zooba

tests/test_install_command.py

@@ -1,6 +1,7 @@
+import os
 import pytest
 import secrets
-from pathlib import PurePath
+from pathlib import Path, PurePath
 
 from manage import install_command as IC
 from manage import installs
@@ -101,16 +102,32 @@ def test_write_alias_fallback_platform(alias_checker):
     alias_checker.check_w64(alias_checker.Cmd("-spam"), "1.0", "testB")
 
 
-def test_print_cli_shortcuts(patched_installs, assert_log):
+def test_print_cli_shortcuts(patched_installs, assert_log, monkeypatch, tmp_path):
     class Cmd:
-        global_dir = None
+        global_dir = Path(tmp_path)
         def get_installs(self):
             return installs.get_installs(None)
 
+    (tmp_path / "fake.exe").write_bytes(b"")
+
+    monkeypatch.setitem(os.environ, "PATH", f"{os.environ['PATH']};{Cmd.global_dir}")
     IC.print_cli_shortcuts(Cmd())
-    print(assert_log)
     assert_log(
         assert_log.skip_until("Installed %s", ["Python 2.0-64", PurePath("C:\\2.0-64")]),
         assert_log.skip_until("%s will be launched by %s", ["Python 1.0-64", "py1.0[-64].exe"]),
         ("%s will be launched by %s", ["Python 1.0-32", "py1.0-32.exe"]),
     )
+
+
+def test_print_path_warning(patched_installs, assert_log, tmp_path):
+    class Cmd:
+        global_dir = Path(tmp_path)
+        def get_installs(self):
+            return installs.get_installs(None)
+
+    (tmp_path / "fake.exe").write_bytes(b"")
+
+    IC.print_cli_shortcuts(Cmd())
+    assert_log(
+        assert_log.skip_until(".*Global shortcuts directory is not on PATH")
+    )