Skip to content

test: cover invalid config keys#273

Closed
haosenwang1018 wants to merge 1 commit intozilliztech:mainfrom
haosenwang1018:test/config-invalid-key-errors
Closed

test: cover invalid config keys#273
haosenwang1018 wants to merge 1 commit intozilliztech:mainfrom
haosenwang1018:test/config-invalid-key-errors

Conversation

@haosenwang1018
Copy link
Copy Markdown
Contributor

Summary

  • add coverage for set_config_value() rejecting unknown config sections
  • add coverage for set_config_value() rejecting unknown fields inside valid sections
  • keep the change focused on real error branches in the shared config write path

Testing

  • uv run ruff format tests/test_config.py
  • uv run ruff check tests/test_config.py
  • uv run python -m pytest tests/test_config.py -q

Closes #114

@zc277584121
Copy link
Copy Markdown
Collaborator

Closing — this is superseded by #280 which covers the same invalid key validation with a more comprehensive parametrized approach. Thanks!

@zc277584121 zc277584121 closed this Apr 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: improve test coverage across the project

2 participants