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

Commit 424091d

Browse files
committed
fix specs
1 parent ca6b87a commit 424091d

File tree

2 files changed

+29
-1
lines changed

2 files changed

+29
-1
lines changed

lib/completions/endpoints/open_ai.rb

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,19 @@ def provider_id
2727
AiApiAuditLog::Provider::OpenAI
2828
end
2929

30+
def perform_completion!(
31+
dialect,
32+
user,
33+
model_params = {},
34+
feature_name: nil,
35+
feature_context: nil,
36+
partial_tool_calls: false,
37+
&blk
38+
)
39+
@disable_native_tools = dialect.disable_native_tools?
40+
super
41+
end
42+
3043
private
3144

3245
def model_uri

spec/lib/modules/ai_bot/personas/persona_spec.rb

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ def system_prompt
4646
}
4747
end
4848

49+
fab!(:admin)
4950
fab!(:user)
5051
fab!(:upload)
5152

@@ -238,6 +239,20 @@ def system_prompt
238239

239240
# should be ordered by priority and then alpha
240241
expect(DiscourseAi::AiBot::Personas::Persona.all(user: user)).to eq(
242+
[
243+
DiscourseAi::AiBot::Personas::General,
244+
DiscourseAi::AiBot::Personas::Artist,
245+
DiscourseAi::AiBot::Personas::Creative,
246+
DiscourseAi::AiBot::Personas::DiscourseHelper,
247+
DiscourseAi::AiBot::Personas::GithubHelper,
248+
DiscourseAi::AiBot::Personas::Researcher,
249+
DiscourseAi::AiBot::Personas::SettingsExplorer,
250+
DiscourseAi::AiBot::Personas::SqlHelper,
251+
],
252+
)
253+
254+
# it should allow staff access to WebArtifactCreator
255+
expect(DiscourseAi::AiBot::Personas::Persona.all(user: admin)).to eq(
241256
[
242257
DiscourseAi::AiBot::Personas::General,
243258
DiscourseAi::AiBot::Personas::Artist,
@@ -256,7 +271,7 @@ def system_prompt
256271
SiteSetting.ai_google_custom_search_api_key = ""
257272
SiteSetting.ai_artifact_security = "disabled"
258273

259-
expect(DiscourseAi::AiBot::Personas::Persona.all(user: user)).to contain_exactly(
274+
expect(DiscourseAi::AiBot::Personas::Persona.all(user: admin)).to contain_exactly(
260275
DiscourseAi::AiBot::Personas::General,
261276
DiscourseAi::AiBot::Personas::SqlHelper,
262277
DiscourseAi::AiBot::Personas::SettingsExplorer,

0 commit comments

Comments
 (0)