Skip to content

test: cover config dict conversion edge cases#278

Merged
zc277584121 merged 1 commit intozilliztech:mainfrom
haosenwang1018:test/config-dict-to-config-coverage
Apr 2, 2026
Merged

test: cover config dict conversion edge cases#278
zc277584121 merged 1 commit intozilliztech:mainfrom
haosenwang1018:test/config-dict-to-config-coverage

Conversation

@haosenwang1018
Copy link
Copy Markdown
Contributor

Summary

  • add coverage for _dict_to_config() ignoring unknown fields and unknown sections
  • verify non-dict section payloads are skipped instead of crashing
  • verify explicit empty section dicts still fall back to dataclass defaults

Testing

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

Refs #114

@zc277584121 zc277584121 merged commit aaf09ec into zilliztech:main Apr 2, 2026
5 of 6 checks passed
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.

2 participants