You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Test coverage is somewhat spotty right now. Plenty of the core pieces are covered, along with actual bugs that were discovered by usage, but there are plenty of obvious gaps in our current verification.
Any contribution to increase coverage of the pytest tests is welcome.
Note that these tests do not have a usable py command, and so need to be structured to not rely on that - plenty of mocks, monkey-patching and directly importing commands is intended!
(For tests that can only be run after the full app is compiled/installed, create a new issue describing the scenario and why you think it wouldn't be valid as a unit test.)
I added unit tests for config_bool, _expand_vars, config_append, config_dict_merge, config_split, and config_split_append. These tests increased coverage for config.py from 17% to 30%. I deliberately limited this PR to the dependency-free helpers, with resolve_config and merge_config as natural follow-up work. PR: (#392)
Activity
pythongh-7: Add unit tests for pure configuration helpers.
breadonb commentedon Aug 1, 2026
I added unit tests for
config_bool,_expand_vars,config_append,config_dict_merge,config_split, andconfig_split_append. These tests increased coverage forconfig.pyfrom 17% to 30%. I deliberately limited this PR to the dependency-free helpers, withresolve_configandmerge_configas natural follow-up work. PR: (#392)gh-7: Add some unit tests for config helpers. (#392)