tests: import-validation docstring cites brave default_params.time_period, which never gained an options list
#5,676 创建于 2026年8月22日
仓库指标
- 星标
- (8,787 个星标)
- PR 合并指标
- (平均合并 2天 11小时) (30 天内合并 263 个 PR)
描述
Follow-up to #5597 (fixes #5589).
The module docstring of tests/settings/test_import_settings_validation.py:6 cites the motivating example as:
a free-text
search.engine.web.brave.default_params.time_periodvalue from before the field gained an options list
But that key has not gained an options list. In the current defaults:
| key | ui_element |
options |
|---|---|---|
search.engine.web.brave.default_params.time_period |
text |
null |
search.engine.web.serpapi.default_params.time_period |
text |
null |
search.engine.web.serper.default_params.time_period |
select |
day / week / month / year / null |
search.engine.web.tavily.default_params.time_period |
select |
d / w / m / y |
Impact: a maintainer tracing the regression to the cited key finds brave's time_period is still free-text, which undercuts the documented motivation and sends reproduction effort to the wrong setting.
Scope: docstring only. The tests themselves correctly exercise search.time_period and the validation logic is unaffected.
Suggested fix: point the example at a key that actually carries an options list — search.engine.web.serper.default_params.time_period or the tavily equivalent — or drop the engine qualifier so the example matches the changelog's generic default_params.time_period phrasing.