Skip to content

test: cover project config precedence#272

Closed
haosenwang1018 wants to merge 1 commit intozilliztech:mainfrom
haosenwang1018:test/config-project-precedence
Closed

test: cover project config precedence#272
haosenwang1018 wants to merge 1 commit intozilliztech:mainfrom
haosenwang1018:test/config-project-precedence

Conversation

@haosenwang1018
Copy link
Copy Markdown
Contributor

Summary

  • add direct coverage proving project TOML overrides global TOML during config resolution
  • verify set_config_value(..., project=True) writes to PROJECT_CONFIG_PATH instead of the global config file
  • keep the change focused on real config precedence behavior shared by multiple CLI entrypoints

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 — the project config precedence tests are covered more comprehensively in #280. 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