From 8e7cb60beaa6fe01c1e4de57d90f434bcb3bd6a7 Mon Sep 17 00:00:00 2001 From: "Danilo M." Date: Tue, 30 Jun 2026 10:55:37 +0200 Subject: feat: config load/write with env override Co-Authored-By: Claude Opus 4.8 --- tests/unit/test_config.py | 35 +++++++++++++++++++++++++++++++++++ 1 file changed, 35 insertions(+) create mode 100644 tests/unit/test_config.py (limited to 'tests/unit/test_config.py') diff --git a/tests/unit/test_config.py b/tests/unit/test_config.py new file mode 100644 index 0000000..de90a3b --- /dev/null +++ b/tests/unit/test_config.py @@ -0,0 +1,35 @@ +import os, tempfile, unittest +from pathlib import Path +from firefly_cli import config +from firefly_cli.errors import ConfigError + +class TestConfig(unittest.TestCase): + def setUp(self): + self.dir = tempfile.TemporaryDirectory() + self.path = Path(self.dir.name) / "config.toml" + def tearDown(self): + self.dir.cleanup() + for k in ("FIREFLY_URL", "FIREFLY_TOKEN"): + os.environ.pop(k, None) + + def test_write_then_read_roundtrip(self): + config.write("https://f.example/", "tok123", path=self.path) + cfg = config.load(path=self.path, env={}) + self.assertEqual(cfg["url"], "https://f.example") # trailing slash trimmed + self.assertEqual(cfg["token"], "tok123") + + def test_env_overrides_file(self): + config.write("https://file/", "filetok", path=self.path) + cfg = config.load(path=self.path, + env={"FIREFLY_URL": "https://env", "FIREFLY_TOKEN": "envtok"}) + self.assertEqual(cfg["url"], "https://env") + self.assertEqual(cfg["token"], "envtok") + + def test_missing_everything_raises_configerror(self): + with self.assertRaises(ConfigError): + config.load(path=self.path, env={}) + + def test_env_only_no_file(self): + cfg = config.load(path=self.path, + env={"FIREFLY_URL": "https://env/", "FIREFLY_TOKEN": "t"}) + self.assertEqual(cfg["url"], "https://env") -- cgit v1.2.3