It's a good start, thanks!

But given that you want to find what are the slowest tests ran, we could maybe add a --time option that displays all tests sorted by how much time they took. I'd actually use that a lot, for example to find what are the slowest specs on the compiler itself and try to reduce their them.

Otherwise you'll have to manually sort the times from the output.

rspec has a --profile mode which outputs the top 10 slowest specs in the suite. I think that's a better approach than printing the time in verbose mode: you typically want to see the slowest tests, not just all runtimes. Agreed?

Sounds good. Can you choose how many slowest specs to show?

What do you think about --slow=20 as the parameter to show the slowest 20 specs, defaulting to 10?

--profile reads better. We could do --profile=10 or we can start with just --profile and a fixed 10 and later improve on that, if ever needed :-)

Any tips on how to add an option default, so the user can pass --profile or -p 20, where the first version would default to 10?

Maybe:

require "option_parser"

OptionParser.parse! do |parser|
  parser.on("-p", "--profile N", "desc") do |value|
    p value # empty string means nothing -p was used
  end
end

Or:

require "option_parser"

OptionParser.parse! do |parser|
  parser.on("-p", "--profile", "desc") do
    puts "bare"
  end
  parser.on("--profile N", "desc") do |value|
    p value
  end
end

It seems you can't pass "-p=10" nor "-p 10", but you can pass "-p10" as command line args. I can't remember now why that's like that. Otherwise you can use:

require "option_parser"

OptionParser.parse! do |parser|
  parser.on("-p", "--profile", "desc") do
    puts "bare"
  end
  parser.on("-p N", "--profile N", "desc") do |value|
    p value
  end
end

but for the second case you must use -p10 or --profile 10 or --profile=10, but you can't use -p=10.

Too many options to choose now 😊

Updated

Merged as ab52258

Thank you!