491e7dea1f
Introduce built-in launchd management to remove shell-wrapper indirection and make macOS autostart reliable. - Add `install-launchagent` and `uninstall-launchagent` CLI subcommands. - Create launchd helper module to: - write `~/Library/LaunchAgents/<label>.plist` - invoke app via `sys.executable -m mpd_now_playable.cli` (absolute interpreter path) - set `RunAtLoad`, `KeepAlive` - run `launchctl bootout/bootstrap/kickstart` for install and `bootout` for uninstall - support `--label` and `--force` - Add tests for subcommand dispatch, plist generation, and launchctl flow. - Document the new launchd workflow in README. - Run markdownlint This keeps the setup DRY/KISS, avoids PATH/shell expansion pitfalls in launchd, and gives users a supported and easy autostart path out of the box.
140 lines
4 KiB
Python
140 lines
4 KiB
Python
from types import SimpleNamespace
|
|
|
|
import pytest
|
|
|
|
import mpd_now_playable.cli as cli
|
|
|
|
|
|
def test_main_help_prints_usage(monkeypatch, capsys) -> None:
|
|
monkeypatch.setattr(cli.sys, "argv", ["mpd-now-playable", "--help"])
|
|
|
|
def fail_load() -> None:
|
|
raise AssertionError("loadConfig should not run for --help")
|
|
|
|
monkeypatch.setattr(cli, "loadConfig", fail_load)
|
|
|
|
with pytest.raises(SystemExit) as exc:
|
|
cli.main()
|
|
assert exc.value.code == 0
|
|
output = capsys.readouterr().out
|
|
|
|
assert "usage: mpd-now-playable" in output
|
|
assert "--version" in output
|
|
assert "install-launchagent" in output
|
|
assert "uninstall-launchagent" in output
|
|
|
|
|
|
def test_main_version_prints_version(monkeypatch, capsys) -> None:
|
|
monkeypatch.setattr(cli.sys, "argv", ["mpd-now-playable", "--version"])
|
|
|
|
def fail_load() -> None:
|
|
raise AssertionError("loadConfig should not run for --version")
|
|
|
|
monkeypatch.setattr(cli, "loadConfig", fail_load)
|
|
|
|
cli.main()
|
|
output = capsys.readouterr().out
|
|
|
|
assert output.strip() == f"mpd-now-playable v{cli.__version__}"
|
|
|
|
|
|
def test_main_starts_listener_and_receivers(monkeypatch) -> None:
|
|
receiver_cfg_1 = SimpleNamespace(kind="websockets")
|
|
receiver_cfg_2 = SimpleNamespace(kind="websockets")
|
|
config = SimpleNamespace(
|
|
cache="memory://",
|
|
mpd=SimpleNamespace(host="127.0.0.1", port=6600),
|
|
receivers=(receiver_cfg_1, receiver_cfg_2),
|
|
)
|
|
|
|
listener = object()
|
|
receiver_1 = object()
|
|
receiver_2 = object()
|
|
fake_coroutine = object()
|
|
|
|
seen = {}
|
|
|
|
def fake_load_config() -> SimpleNamespace:
|
|
return config
|
|
|
|
def fake_listener_ctor(cache: str) -> object:
|
|
seen["cache"] = cache
|
|
return listener
|
|
|
|
def fake_construct_receiver(rcfg: SimpleNamespace) -> object:
|
|
if rcfg is receiver_cfg_1:
|
|
return receiver_1
|
|
if rcfg is receiver_cfg_2:
|
|
return receiver_2
|
|
raise AssertionError(f"Unexpected receiver config: {rcfg!r}")
|
|
|
|
class FakeLoopFactory:
|
|
@staticmethod
|
|
def make_loop() -> object:
|
|
return object()
|
|
|
|
def fake_choose_loop_factory(receivers: tuple[object, ...]) -> type[FakeLoopFactory]:
|
|
seen["receivers"] = receivers
|
|
return FakeLoopFactory
|
|
|
|
def fake_listen(cfg: SimpleNamespace, lst: object, receivers: tuple[object, ...]) -> object:
|
|
seen["listen_args"] = (cfg, lst, receivers)
|
|
return fake_coroutine
|
|
|
|
def fake_run(coro: object, *, loop_factory: object, debug: bool) -> None:
|
|
seen["run"] = (coro, loop_factory, debug)
|
|
|
|
monkeypatch.setattr(cli.sys, "argv", ["mpd-now-playable"])
|
|
monkeypatch.setattr(cli, "loadConfig", fake_load_config)
|
|
monkeypatch.setattr(cli, "MpdStateListener", fake_listener_ctor)
|
|
monkeypatch.setattr(cli, "construct_receiver", fake_construct_receiver)
|
|
monkeypatch.setattr(cli, "choose_loop_factory", fake_choose_loop_factory)
|
|
monkeypatch.setattr(cli, "listen", fake_listen)
|
|
monkeypatch.setattr(cli.asyncio, "run", fake_run)
|
|
|
|
cli.main()
|
|
|
|
assert seen["cache"] == "memory://"
|
|
assert seen["receivers"] == (receiver_1, receiver_2)
|
|
assert seen["listen_args"] == (config, listener, (receiver_1, receiver_2))
|
|
assert seen["run"] == (fake_coroutine, FakeLoopFactory.make_loop, False)
|
|
|
|
|
|
def test_main_install_launchagent(monkeypatch, capsys) -> None:
|
|
seen = {}
|
|
|
|
def fake_install(*, label: str, force: bool) -> str:
|
|
seen["args"] = (label, force)
|
|
return "/tmp/example.plist"
|
|
|
|
monkeypatch.setattr(
|
|
cli.sys,
|
|
"argv",
|
|
["mpd-now-playable", "install-launchagent", "--label", "com.example.test", "--force"],
|
|
)
|
|
monkeypatch.setattr(cli, "install_launchagent", fake_install)
|
|
|
|
cli.main()
|
|
output = capsys.readouterr().out
|
|
assert seen["args"] == ("com.example.test", True)
|
|
assert "Installed LaunchAgent at /tmp/example.plist" in output
|
|
|
|
|
|
def test_main_uninstall_launchagent(monkeypatch, capsys) -> None:
|
|
seen = {}
|
|
|
|
def fake_uninstall(*, label: str) -> str:
|
|
seen["label"] = label
|
|
return "/tmp/example.plist"
|
|
|
|
monkeypatch.setattr(
|
|
cli.sys,
|
|
"argv",
|
|
["mpd-now-playable", "uninstall-launchagent", "--label", "com.example.test"],
|
|
)
|
|
monkeypatch.setattr(cli, "uninstall_launchagent", fake_uninstall)
|
|
|
|
cli.main()
|
|
output = capsys.readouterr().out
|
|
assert seen["label"] == "com.example.test"
|
|
assert "Uninstalled LaunchAgent at /tmp/example.plist" in output
|