tz icon indicating copy to clipboard operation
tz copied to clipboard

Configuration file support (TOML) tests

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

This PR adds unit-test coverage for the recently-added TOML config parsing, on top of the bugfixes in PR #67. It raises the overall unit test coverage from 46% to 73% (fix #68).

However, this PR does not test the correct semantic mapping of every individual key configuration, so there is still further improvement to be done in future. For example, it’s still possible for someone to add a new keystroke but forget to (or make a mistake) update some code paths or documentation (perhaps a new issue: ā€œConfiguration testing is incompleteā€). In future, it may even be good to change the Keymap to a simple map[string][]string to reduce the amount of code paths and testing required.

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

This PR is based on #67 which should be merged first.

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