Skip to content

Commit d2ff29d

Browse files
TonsOfFunclaude
andcommitted
Fix rubocop linting issues in chat provider test
- Remove extra empty line at class body beginning - Add spaces inside array brackets per rubocop style rules - Add missing space after comma 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 219e3e7 commit d2ff29d

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

test/providers/open_ai/chat_provider_test.rb

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ module Providers
1515
module OpenAI
1616
module Chat
1717
class ChatProviderTest < ActiveSupport::TestCase
18-
1918
include WebMock::API
2019

2120
setup do
@@ -31,7 +30,7 @@ class ChatProviderTest < ActiveSupport::TestCase
3130
stub_streaming_response(tool_calls_sse_response)
3231

3332
stream = @client.chat.completions.stream(
34-
messages: [{ content: "What's the weather in Boston?", role: :user }],
33+
messages: [ { content: "What's the weather in Boston?", role: :user } ],
3534
model: "qwen-plus",
3635
tools: weather_tool
3736
)
@@ -59,7 +58,7 @@ class ChatProviderTest < ActiveSupport::TestCase
5958
}
6059

6160
assert_equal(
62-
[expected_message],
61+
[ expected_message ],
6362
chat_provider.send(:message_stack),
6463
"message_stack should contain one assistant message with merged tool_calls"
6564
)
@@ -69,7 +68,7 @@ class ChatProviderTest < ActiveSupport::TestCase
6968

7069
def stub_streaming_response(response_body, request_options = {})
7170
default_request = {
72-
messages: [{ content: "What's the weather in Boston?",role: "user" }],
71+
messages: [ { content: "What's the weather in Boston?", role: "user" } ],
7372
model: "qwen-plus",
7473
stream: true
7574
}
@@ -112,7 +111,7 @@ def weather_tool
112111
city: { type: "string" },
113112
units: { type: "string" }
114113
},
115-
required: ["city", "units"],
114+
required: [ "city", "units" ],
116115
additionalProperties: false
117116
},
118117
strict: true

0 commit comments

Comments
 (0)