Follow-up from "Simulate warden session for Duo Chat"
The following discussion from !158022 (merged) should be addressed:
-
@jessieay started a discussion: Suggestion (non-blocking but would like to see in follow up): there is untested logic added in this MR.
In that case where the feature flag is enabled, I would also expect to see test cases for:
- When session is nil
- When
ai_prompt_message.user
is nil
These are both checks in
ee/app/workers/llm/completion_worker.rb
I will add a follow-up issue and assign you @lulalala