Skip to content

Commit

Permalink
Merge pull request #913 from cucumber/fix-new-rubocop-offenses
Browse files Browse the repository at this point in the history
Fix some new RuboCop offenses
  • Loading branch information
mvz authored Dec 3, 2023
2 parents 43ab140 + ac25182 commit a332436
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion lib/aruba/basic_configuration.rb
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ def ==(other)

# Set if name is option
def set_if_option(name, *args)
public_send("#{name}=".to_sym, *args) if option? name
public_send(:"#{name}=", *args) if option? name
end

private
Expand Down
6 changes: 4 additions & 2 deletions lib/aruba/command.rb
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,9 @@ def initialize(command, opts = {})
launchers << Processes::InProcess
launchers << Processes::SpawnProcess

launcher = launchers.find { |l| l.match? opts[:mode] }
klass = launchers.find { |l| l.match? opts[:mode] }

super launcher.new(
launcher = klass.new(
command,
opts.fetch(:exit_timeout),
opts.fetch(:io_wait_timeout),
Expand All @@ -41,6 +41,8 @@ def initialize(command, opts = {})
opts.fetch(:startup_wait_time)
)

super(launcher)

@event_bus = opts.fetch(:event_bus)
rescue KeyError => e
raise ArgumentError, e.message
Expand Down

0 comments on commit a332436

Please sign in to comment.