Use Chat class for real chat reqeusts
What does this MR do and why?
Instead of calling zero shot agent directly when testing real requests, we rather call upper Chat class. This will allow us to test also newly added slash commands.
Specs should behave same as before, this just prepares the spec for adding tests for slash commands in a next MR.
Related to #429915 (closed)
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
How to set up and validate locally
- follow the steps in https://docs.gitlab.com/ee/development/ai_features/duo_chat.html#testing-gitlab-duo-chat-against-real-llms-locally to test it, but with this change applied, the location of spec changes
REAL_AI_REQUEST=1 bundle exec rspec ee/spec/lib/gitlab/llm/completions/chat_real_requests_spec.rb
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.