diff options
| author | Andrew Branson <andrew.branson@jolla.com> | 2026-08-09 17:37:37 +0200 |
|---|---|---|
| committer | Andrew Branson <andrew.branson@jolla.com> | 2026-08-09 17:37:37 +0200 |
| commit | ffb64c45b42dda1edf44a0fbaf42c78f790d6869 (patch) | |
| tree | 290ee2607c5679f628cd673b3dfb875aefd41f1b /tests | |
| parent | b9746bfa320cdd1464eb6c91fbadad35c84d2459 (diff) | |
Track a generic config.py.template and ignore the operational config.py so
machine-specific build host defaults remain local.
Document the bootstrap copy and replace personal OBS and Android test
fixtures with neutral examples.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/test_server.py | 22 |
1 files changed, 18 insertions, 4 deletions
diff --git a/tests/test_server.py b/tests/test_server.py index 232d411..c5534a1 100644 --- a/tests/test_server.py +++ b/tests/test_server.py @@ -13,6 +13,7 @@ from sailfish_devel_mcp.config import ( AndroidBuildHostConfig, BUNDLED_BUILD_HELPER, Config, + DEFAULT_ANDROID_BUILD_HOST, DEFAULT_ANDROID_BUILD_PROJECT_DIR, DEFAULT_ANDROID_BUILD_SSH_TARGET, DeviceConfig, @@ -250,16 +251,29 @@ class McpServerTests(unittest.TestCase): self.assertIsNone(config.paths.local_sdk) self.assertTrue(config.paths.build_sailfishos.exists()) self.assertNotIn("build-sailfishos-skill", str(config.paths.build_sailfishos)) - self.assertEqual(config.default_android_build_host, "android-builder") + self.assertEqual(config.default_android_build_host, DEFAULT_ANDROID_BUILD_HOST) self.assertEqual( - config.android_build_hosts["android-builder"].ssh_target, + config.android_build_hosts[DEFAULT_ANDROID_BUILD_HOST].ssh_target, DEFAULT_ANDROID_BUILD_SSH_TARGET, ) self.assertEqual( - config.android_build_hosts["android-builder"].project_dir, + config.android_build_hosts[DEFAULT_ANDROID_BUILD_HOST].project_dir, DEFAULT_ANDROID_BUILD_PROJECT_DIR, ) + def test_config_template_contains_only_generic_build_host_defaults(self) -> None: + template = ( + Path(__file__).resolve().parents[1] + / "src" + / "sailfish_devel_mcp" + / "config.py.template" + ).read_text(encoding="utf-8") + self.assertIn('DEFAULT_ANDROID_BUILD_HOST = "android-builder"', template) + self.assertIn( + 'DEFAULT_ANDROID_BUILD_SSH_TARGET = "builder@example.invalid"', + template, + ) + def test_config_loads_local_sdk_from_paths(self) -> None: with tempfile.TemporaryDirectory() as tmp: root = Path(tmp) @@ -1010,7 +1024,7 @@ class McpServerTests(unittest.TestCase): self.assertEqual(response["result"]["structuredContent"]["returncode"], 7) argv = list(mocked_run.call_args.args[0]) self.assertEqual(argv[:3], ["ssh", "-F", str(root / "ssh_config")]) - self.assertIn("builder@example.invalid", argv) + self.assertIn(DEFAULT_ANDROID_BUILD_SSH_TARGET, argv) remote = argv[-1] self.assertIn("tail -n \"$lines\" \"$log_path\"", remote) self.assertIn("lines=25", remote) |
