tz icon indicating copy to clipboard operation
tz copied to clipboard

Configuration file support (TOML) tests are missing

Open jnd-au opened this issue 1 year ago • 1 comments

TOML config file support (and associated logging) was recently added to tz (#1) but it lacks unit tests:

  1. It reduced HEAD unit-test coverage from 42% to 31%, and reduced coverage from 84% to 73% in PR #62.
  2. It introduced user-impacting changes of behaviour, bugs, or blockers like: #63 #65 #66

Therefore, config file parsing is a significant surface area for unit testing.

Furthermore, testing is needed for the configuration environment-variable support, but this is already included with bugfixes in PR #62.

jnd-au avatar Oct 17 '24 03:10 jnd-au

Furthermore, the config file functions need to be parameterised to unblock unit testing.

jnd-au avatar Oct 17 '24 03:10 jnd-au