Name:
Surface L4: drop cancelled viin_ai_api (+ webhook), ship viin_ai_livechat (ADR-027)
State:
Killed
finished in 303m
PR State:
merged
PR Author:
David Tran
PR Author Email:
PR:
#100
Committer:
David Tran
Committer Email:
davidtran.hp@gmail.com
Commit:
30c98d00e89c3bbfa9d4ac0ffdeb23d71ce6de1f
Description:
[FIX] viin_ai_livechat: assert+mute the expected guard WARNINGs in the two refusal tests
The visitor-path guards (tool-bearing agent / no least-privilege service user) correctly log a WARNING in production - those are genuine misconfigurations an admin should fix. But the two negative-path tests that deliberately construct such agents leaked that WARNING to the Runbot / go-live log.
Wrap each guard-triggering call in assertLogs: this BINDS the 'refuse + warn + hand to human' contract (the test now reds if a future change drops the guard warning) AND keeps the expected warning out of the log (assertLogs sets the logger's propagate=False for the context). Production behavior is unchanged - test-only.
Branch:
17.0
Instance ID:
0
Age:
Up-time: