mirror of
https://github.com/discourse/discourse-ai.git
synced 2025-08-03 19:53:28 +00:00
0634b85a81
* UX: Validations to Llm-backed features (except AI Bot) This change is part of an ongoing effort to prevent enabling a broken feature due to lack of configuration. We also want to explicit which provider we are going to use. For example, Claude models are available through AWS Bedrock and Anthropic, but the configuration differs. Validations are: * You must choose a model before enabling the feature. * You must turn off the feature before setting the model to blank. * You must configure each model settings before being able to select it. * Add provider name to summarization options * vLLM can technically support same models as HF * Check we can talk to the selected model * Check for Bedrock instead of anthropic as a site could have both creds setup
89 lines
2.9 KiB
Ruby
89 lines
2.9 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
RSpec.describe Jobs::StreamPostHelper do
|
|
subject(:job) { described_class.new }
|
|
|
|
before { SiteSetting.ai_helper_model = "fake:fake" }
|
|
|
|
describe "#execute" do
|
|
fab!(:topic) { Fabricate(:topic) }
|
|
fab!(:post) do
|
|
Fabricate(
|
|
:post,
|
|
topic: topic,
|
|
raw:
|
|
"I like to eat pie. It is a very good dessert. Some people are wasteful by throwing pie at others but I do not do that. I always eat the pie.",
|
|
)
|
|
end
|
|
fab!(:user) { Fabricate(:leader) }
|
|
|
|
before do
|
|
Group.find(Group::AUTO_GROUPS[:trust_level_3]).add(user)
|
|
SiteSetting.composer_ai_helper_enabled = true
|
|
end
|
|
|
|
describe "validates params" do
|
|
it "does nothing if there is no post" do
|
|
messages =
|
|
MessageBus.track_publish("/discourse-ai/ai-helper/explain/#{post.id}") do
|
|
job.execute(post_id: nil, user_id: user.id, term_to_explain: "pie")
|
|
end
|
|
|
|
expect(messages).to be_empty
|
|
end
|
|
|
|
it "does nothing if there is no user" do
|
|
messages =
|
|
MessageBus.track_publish("/discourse-ai/ai-helper/explain/#{post.id}") do
|
|
job.execute(post_id: post.id, user_id: nil, term_to_explain: "pie")
|
|
end
|
|
|
|
expect(messages).to be_empty
|
|
end
|
|
|
|
it "does nothing if there is no term to explain" do
|
|
messages =
|
|
MessageBus.track_publish("/discourse-ai/ai-helper/explain/#{post.id}") do
|
|
job.execute(post_id: post.id, user_id: user.id, term_to_explain: nil)
|
|
end
|
|
|
|
expect(messages).to be_empty
|
|
end
|
|
end
|
|
|
|
it "publishes updates with a partial result" do
|
|
explanation =
|
|
"In this context, \"pie\" refers to a baked dessert typically consisting of a pastry crust and filling."
|
|
|
|
partial_explanation = "I"
|
|
|
|
DiscourseAi::Completions::Llm.with_prepared_responses([explanation]) do
|
|
messages =
|
|
MessageBus.track_publish("/discourse-ai/ai-helper/explain/#{post.id}") do
|
|
job.execute(post_id: post.id, user_id: user.id, term_to_explain: "pie")
|
|
end
|
|
|
|
partial_result_update = messages.first.data
|
|
expect(partial_result_update[:done]).to eq(false)
|
|
expect(partial_result_update[:result]).to eq(partial_explanation)
|
|
end
|
|
end
|
|
|
|
it "publishes a final update to signal we're done" do
|
|
explanation =
|
|
"In this context, \"pie\" refers to a baked dessert typically consisting of a pastry crust and filling."
|
|
|
|
DiscourseAi::Completions::Llm.with_prepared_responses([explanation]) do
|
|
messages =
|
|
MessageBus.track_publish("/discourse-ai/ai-helper/explain/#{post.id}") do
|
|
job.execute(post_id: post.id, user_id: user.id, term_to_explain: "pie")
|
|
end
|
|
|
|
final_update = messages.last.data
|
|
expect(final_update[:result]).to eq(explanation)
|
|
expect(final_update[:done]).to eq(true)
|
|
end
|
|
end
|
|
end
|
|
end
|