Skip to content
This repository was archived by the owner on Jul 22, 2025. It is now read-only.

Commit fccd072

Browse files
authored
DEV: Don't use delays for streaming summaries. (#1244)
We started used a callback as a buffer in FoldContent, so the Fake endpoint is attempting to emulate delays in the streaming. However, we don't care about that in these specs.
1 parent 0d60aca commit fccd072

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

spec/jobs/scheduled/summaries_backfill_spec.rb

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@
77
let(:limit) { 24 } # guarantee two summaries per batch
88
let(:intervals) { 12 } # budget is split into intervals. Job runs every five minutes.
99

10+
before { DiscourseAi::Completions::Endpoints::Fake.delays = [] }
11+
after { DiscourseAi::Completions::Endpoints::Fake.reset! }
12+
1013
before do
1114
assign_fake_provider_to(:ai_summarization_model)
1215
SiteSetting.ai_summarization_enabled = true

spec/lib/modules/summarization/fold_content_spec.rb

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@
1010

1111
before { SiteSetting.ai_summarization_enabled = true }
1212

13+
before { DiscourseAi::Completions::Endpoints::Fake.delays = [] }
14+
after { DiscourseAi::Completions::Endpoints::Fake.reset! }
15+
1316
describe "#summarize" do
1417
before do
1518
# Make sure each content fits in a single chunk.

0 commit comments

Comments
 (0)