ManageIQ/manageiq

View on GitHub
tools/memcached_ping.rb

Summary

Maintainability
A
0 mins
Test Coverage

Use Array.new(10) with a block instead of .times.map.
Open

  avg = 10.times.map { Benchmark.realtime { client.get("test") } }.inject(:+) / 10.0
Severity: Minor
Found in tools/memcached_ping.rb by rubocop

This cop checks for .times.map calls. In most cases such calls can be replaced with an explicit array creation.

Example:

# bad
9.times.map do |i|
  i.to_s
end

# good
Array.new(9) do |i|
  i.to_s
end

Use sum instead of inject(:+), unless calling inject(:+) on an empty array.
Open

  avg = 10.times.map { Benchmark.realtime { client.get("test") } }.inject(:+) / 10.0
Severity: Minor
Found in tools/memcached_ping.rb by rubocop

There are no issues that match your filters.

Category
Status