Skip to content

Commit 757f84f

Browse files
rosaclaude
andcommitted
Address review feedback: add missing require, fix redundant test
- Add explicit require for check/result in client_stub.rb so the file is standalone-loadable - Rename test_accepts_string_keys to test_accepts_symbol_keys and actually pass symbol keys to provide distinct coverage Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent b188850 commit 757f84f

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

lib/rspamd/client_stub.rb

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
require "rspamd/check/result"
2+
13
module Rspamd
24
class ClientStub
35
HAM_RESULT = Check::Result.new(

test/rails_test.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@ def test_reset_clears_config_and_clients
4848
assert_instance_of Rspamd::ClientStub, client
4949
end
5050

51-
def test_accepts_string_keys
52-
Rspamd::Rails.setup("enabled" => true, "outbound" => { "host" => "localhost", "port" => 11334 })
51+
def test_accepts_symbol_keys
52+
Rspamd::Rails.setup(enabled: true, outbound: { host: "localhost", port: 11334 })
5353

5454
client = Rspamd::Rails.client_for(:outbound)
5555
assert_instance_of Rspamd::Client, client

0 commit comments

Comments
 (0)