PR State:
merged
PR:
#22
Commit:
9eabf767bfd1ffa3c1889f6e223fd499b2acbcb4
[FIX] viin_ai_agent,viin_ai_search: mute expected warnings in negative-path tests
Four post-install tests intentionally exercise error paths to verify
graceful handling, but lack a @mute_logger wrapper, so runbot logs
their expected warnings as test-suite noise:
- test_run_ai_prompt_missing_model_skips_gracefully and
test_run_ai_prompt_missing_prompt_skips_gracefully create server
actions named 'WI17 No Model' / 'WI17 No Prompt' to drive
ir_actions_server._run_action_ai_prompt down its skip branch, which
logs at WARNING from
odoo.addons.viin_ai_agent.models.ir_actions_server.
- test_missing_model_name_raises_error and
test_unknown_model_raises_error post a JSON-RPC request with no /
invalid model_name; the controller raises ValidationError, which
Odoo's HTTP dispatcher logs at WARNING from odoo.http.
Wrap each test with the canonical @mute_logger pattern from commit
7d34381 (viin_ai_chat: mute expected log output in negative-path
controller tests). Production behaviour is unchanged — the warning
still fires when an admin actually misconfigures a server action or a
caller passes a bad model_name; the decorator only suppresses the line
within the test scope so the runbot output stays signal-rich.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Branch:
17.0
| Subbuilds | Name | State | Detail |
|---|---|---|---|
| Build 381284 | Check the quality with Pylint: from test_lint to test_pylint 14m | Succeed | Log |
| Build 381285 | At install tests: from account to website_twitter 120m | Succeed | Log |
| Build 381286 | Post install tests: step install all modules 60m | Succeed | Log |
| Build 381287 | Post install tests: from account to payment_alipay | Killed | Log |
| Build 381288 | Post install tests: from payment_aps to test_viin_web_cohort | Killed | Log |
| Build 381289 | Post install tests: from test_website to to_website_apps_store 69m | Succeed | Log |
| Build 381290 | Post install tests: from to_website_apps_store_loc to viin_estimate_sale 79m | Failed | Log |
| Build 381291 | Post install tests: from viin_event_checkin_website to viin_project_gantt 33m | Succeed | Log |
| Build 381292 | Post install tests: from viin_project_progress to website_twitter 44m | Succeed | Log |
| Build 381293 | Test install all modules without demo data: from account to website_twitter 49m | Succeed | Log |
| Create Date | Level | Message |
|---|---|---|
| 05/02/2026 08:00:12 | INFO |
Updated repository Viindoo-odoo
|
| 05/02/2026 08:00:12 | INFO |
Updated repository Viindoo-tvtmaaddons
|
| 05/02/2026 08:00:12 | INFO |
Updated repository Viindoo-erponline-enterprise
|
| 05/02/2026 08:00:12 | INFO |
Updated repository Viindoo-branding
|
| 05/02/2026 08:00:12 | INFO |
Cloned repository Viindoo-ai
|
| 05/02/2026 10:21:00 | ERROR |
Subbuild # 381290:
ERROR: duplicate key value violates unique constraint "viin_ai_tool_viin_ai_tool_technical_name_unique"
|
| 05/02/2026 10:21:00 | ERROR |
Subbuild # 381290:
odoo.sql_db: bad query: INSERT INTO "viin_ai_tool" ("action_id", "active", "create_date", "create_uid", "description", "name", "parameters_schema", "requires_confirmation", "runs_as_sudo", "sequence", "technical_name", "write_date", "write_uid") VALUES (2076, true, '2026-05-02 02:52:14.849814', 1, 'B', 'Tool alpha duplicate', '{}', false, false, 10, 'alpha', '2026-05-02 02:52:14.849814', 1) RETURNING "id"
DETAIL: Key (technical_name)=(alpha) already exists.
|