Skip to content

Commit 5f59fa8

Browse files
Fix rubocop offenses: shorten find_by_class, disable OneClassPerFile in spec
Made-with: Cursor
1 parent 4668783 commit 5f59fa8

2 files changed

Lines changed: 2 additions & 9 deletions

File tree

lib/sidekiq/throttled/registry.rb

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -106,20 +106,13 @@ def find(name)
106106
# @param name [Class, #to_s]
107107
# @return [Strategy, nil]
108108
def find_by_class(name)
109-
const = if name.is_a?(Class)
110-
name
111-
elsif Object.const_defined?(name)
112-
Object.const_get(name)
113-
end
114-
109+
const = name.is_a?(Class) ? name : (Object.const_get(name) if Object.const_defined?(name))
115110
return unless const.is_a?(Class)
116111

117112
const.ancestors.each do |m|
118113
strategy = find(m.name)
119114
return strategy if strategy
120115
end
121-
122-
nil
123116
rescue NameError
124117
nil
125118
end

spec/support/sidekiq.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ def jid
7373
end
7474
end
7575

76-
class PseudoLogger < Logger
76+
class PseudoLogger < Logger # rubocop:disable Style/OneClassPerFile
7777
include Singleton
7878

7979
def initialize

0 commit comments

Comments
 (0)