fix(env): default INCLUDE_SOURCES to empty string (#223)
* fix(env): default INCLUDE_SOURCES to empty string * test(env): patch resolved config path in include sources test
This commit is contained in:
+1
-1
@@ -264,7 +264,7 @@ def get_config() -> dict[str, Any]:
|
|||||||
('XQUIK_API_KEY', None),
|
('XQUIK_API_KEY', None),
|
||||||
('FROM_BROWSER', None),
|
('FROM_BROWSER', None),
|
||||||
('SETUP_COMPLETE', None),
|
('SETUP_COMPLETE', None),
|
||||||
('INCLUDE_SOURCES', None),
|
('INCLUDE_SOURCES', ''),
|
||||||
]
|
]
|
||||||
|
|
||||||
for key, default in keys:
|
for key, default in keys:
|
||||||
|
|||||||
@@ -0,0 +1,14 @@
|
|||||||
|
from scripts.lib import env
|
||||||
|
|
||||||
|
|
||||||
|
def test_include_sources_defaults_to_empty_string(monkeypatch, tmp_path):
|
||||||
|
# Ensure the env var is not set
|
||||||
|
monkeypatch.delenv("INCLUDE_SOURCES", raising=False)
|
||||||
|
|
||||||
|
# Avoid reading any real user config file by patching the resolved module path directly
|
||||||
|
monkeypatch.setattr(env, "CONFIG_FILE", tmp_path / "does-not-exist.env")
|
||||||
|
|
||||||
|
cfg = env.get_config()
|
||||||
|
|
||||||
|
assert "INCLUDE_SOURCES" in cfg
|
||||||
|
assert cfg["INCLUDE_SOURCES"] == ""
|
||||||
Reference in New Issue
Block a user