tests: import-validation docstring cites brave default_params.time_period, which never gained an options list
#5.676 aberto em 22 de ago. de 2026
Métricas do repositório
- Stars
- (8.787 estrelas)
- Métricas de merge de PR
- (Mesclagem média 2d 11h) (263 fundiu PRs em 30d)
Description
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.