Skip to content

Commit

Permalink
FIX: support case insensitive setting lookup (#795)
Browse files Browse the repository at this point in the history
  • Loading branch information
SamSaffron authored Sep 10, 2024
1 parent a5b5c3b commit 36ce88f
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/ai_bot/tools/setting_context.rb
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ def name
end

def setting_name
parameters[:setting_name]
@setting_name ||= parameters[:setting_name].to_s.downcase.gsub(" ", "_")
end

def invoke
Expand Down
9 changes: 9 additions & 0 deletions spec/lib/modules/ai_bot/tools/setting_context_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,15 @@ def setting_context(setting_name)
expect(result[:context]).to include("moderators_view_emails")
end

it "supports spaces and case insensitive setting name" do
result = setting_context("moderaTors View Emails").invoke

expect(result[:setting_name]).to eq("moderators_view_emails")

expect(result[:context]).to include("site_settings.yml")
expect(result[:context]).to include("moderators_view_emails")
end

it "returns the context for plugin setting" do
result = setting_context("ai_bot_enabled").invoke

Expand Down

0 comments on commit 36ce88f

Please sign in to comment.