mirror of
https://github.com/discourse/discourse-ai.git
synced 2025-10-27 12:38:45 +00:00
f4708d4178
* DEV: Ai summary utilizing name instead of username * utilize username if name not available * test added * test added 2
98 lines
2.8 KiB
Ruby
98 lines
2.8 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
RSpec.describe DiscourseAi::Summarization::Strategies::TopicSummary do
|
|
subject(:topic_summary) { described_class.new(topic) }
|
|
|
|
fab!(:topic) { Fabricate(:topic, highest_post_number: 25) }
|
|
fab!(:post_1) { Fabricate(:post, topic: topic, post_number: 1) }
|
|
fab!(:post_2) { Fabricate(:post, topic: topic, post_number: 2) }
|
|
|
|
describe "#targets_data" do
|
|
shared_examples "includes only public-visible topics" do
|
|
it "only includes visible posts" do
|
|
post_2.update!(hidden: true)
|
|
|
|
post_numbers = topic_summary.targets_data.map { |c| c[:id] }
|
|
|
|
expect(post_numbers).to contain_exactly(1)
|
|
end
|
|
|
|
it "doesn't include posts without users" do
|
|
post_2.update!(user_id: nil)
|
|
|
|
post_numbers = topic_summary.targets_data.map { |c| c[:id] }
|
|
|
|
expect(post_numbers).to contain_exactly(1)
|
|
end
|
|
|
|
it "doesn't include whispers" do
|
|
post_2.update!(post_type: Post.types[:whisper])
|
|
|
|
post_numbers = topic_summary.targets_data.map { |c| c[:id] }
|
|
|
|
expect(post_numbers).to contain_exactly(1)
|
|
end
|
|
end
|
|
|
|
context "when the topic has a best replies summary" do
|
|
before { topic.update(has_summary: true) }
|
|
|
|
it_behaves_like "includes only public-visible topics"
|
|
end
|
|
|
|
context "when the topic doesn't have a best replies summary" do
|
|
before { topic.update(has_summary: false) }
|
|
|
|
it_behaves_like "includes only public-visible topics"
|
|
end
|
|
|
|
context "when the topic has embed content cached" do
|
|
it "embed content is used instead of the raw text" do
|
|
topic_embed =
|
|
Fabricate(
|
|
:topic_embed,
|
|
topic: topic,
|
|
embed_content_cache: "<p>hello world new post :D</p>",
|
|
)
|
|
|
|
content = topic_summary.targets_data
|
|
op_content = content.first[:text]
|
|
|
|
expect(op_content).to include(topic_embed.embed_content_cache)
|
|
end
|
|
end
|
|
|
|
context "when enable_names enabled and prioritize_username_in_ux disabled" do
|
|
fab!(:user) { Fabricate(:user, name: "test") }
|
|
|
|
it "includes the name" do
|
|
SiteSetting.enable_names = true
|
|
SiteSetting.prioritize_username_in_ux = false
|
|
|
|
post_1.update!(user: user)
|
|
|
|
content = topic_summary.targets_data
|
|
poster_name = content.first[:poster]
|
|
|
|
expect(poster_name).to eq("test")
|
|
end
|
|
end
|
|
|
|
context "when enable_names enabled and prioritize_username_in_ux enabled" do
|
|
fab!(:user) { Fabricate(:user, username: "test") }
|
|
|
|
it "includes the username" do
|
|
SiteSetting.enable_names = true
|
|
SiteSetting.prioritize_username_in_ux = true
|
|
|
|
post_1.update!(user: user)
|
|
|
|
content = topic_summary.targets_data
|
|
poster_name = content.first[:poster]
|
|
|
|
expect(poster_name).to eq("test")
|
|
end
|
|
end
|
|
end
|
|
end
|