Keep benchmark cases titles consistent

Before, we would call "capitalize" on the title of the benchmark case, which would change the original string the user provided. This fixes it, so the report always uses the original title.

Author: MatheusRich

CHANGELOG.md

@@ -6,8 +6,19 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+
 ---
 
+## [0.3.2] - 2021-09-10
+
+### Changed
+
+- Keep benchmark cases titles consistent
+
+Before, we would call "capitalize" on the title of the benchmark case, which
+would change the original string the user provided. This fixes it, so the report
+always uses the original title.
+
 ## [0.3.1] - 2021-09-10
 
 ### Changed
@@ -48,6 +59,7 @@ end
 ## [0.2.0] - 2020-10-23
 
 ### Added
+
 - Support for `Benchmark.memory` via [benchmark-memory](https://github.com/michaelherold/benchmark-memory).
 
 [unreleased]: https://github.com/MatheusRich/benchable/compare/v0.2.0...HEAD
@@ -56,10 +68,11 @@ end
 ## [0.1.0] - 2020-08-16
 
 ### Added
+
 - Support for `Benchmark.bm`, `Benchmark.bmbm` and `Benchmark.ips`.
 
 [unreleased]: https://github.com/MatheusRich/benchable/compare/v0.3.1...HEAD
 [0.1.0]: https://github.com/MatheusRich/benchable/releases/tag/v0.1.0
 [0.2.0]: https://github.com/MatheusRich/benchable/releases/tag/v0.2.0
 [0.3.0]: https://github.com/MatheusRich/benchable/releases/tag/v0.3.0
-[0.3.1]: https://github.com/MatheusRich/benchable/releases/tag/v0.3.1
+[0.3.1]: https://github.com/MatheusRich/benchable/releases/tag/v0.3.1

lib/benchable/benchmark.rb

@@ -24,7 +24,7 @@ def self.setup(&block)
     end
 
     def self.bench(name, &block)
-      define_method(method_name_for(name), &block)
+      define_method(:"bench_#{name}", &block)
     end
 
     def setup
@@ -39,10 +39,6 @@ def cases
       public_methods.grep(/\Abench_/)
     end
 
-    private_class_method def self.method_name_for(name)
-      "bench_#{name.to_s.tr(" ", "_").downcase}"
-    end
-
     private
 
     attr_reader :benchmark_type, :options
@@ -56,7 +52,7 @@ def run_benchmark
         b.config(options) if benchmark_type == :ips
 
         cases.each do |benchmark_case|
-          b.report(name_for(benchmark_case)) do
+          b.report(title_for(benchmark_case)) do
             method(benchmark_case).call
           end
         end
@@ -65,8 +61,8 @@ def run_benchmark
       end
     end
 
-    def name_for(benchmark_case)
-      benchmark_case.to_s.gsub("bench_", "").tr("_", " ").capitalize
+    def title_for(benchmark_case)
+      benchmark_case.to_s.delete_prefix("bench_")
     end
 
     def benchmark(&block)

spec/benchable/benchmark_spec.rb

@@ -55,7 +55,7 @@
     end
 
     it "creates a new bench method" do
-      expect { klass.new(:bm).bench_the_puts_method }.to output("Hello world!\n").to_stdout
+      expect { klass.new(:bm).public_send("bench_The puts method") }.to output("Hello world!\n").to_stdout
     end
   end
 

spec/benchable_spec.rb

@@ -62,8 +62,8 @@ def an_invalid_bench_method
     before do
       allow(bench_mock).to receive(:config)
       allow(bench_mock).to receive(:compare!)
-      allow(bench_mock).to receive(:report).with("Sum").and_yield
-      allow(bench_mock).to receive(:report).with("Multiplication").and_yield
+      allow(bench_mock).to receive(:report).with("sum").and_yield
+      allow(bench_mock).to receive(:report).with("multiplication").and_yield
       allow(Benchmark).to receive(:ips).and_yield(bench_mock)
     end
 
@@ -76,8 +76,8 @@ def an_invalid_bench_method
     it "builds and runs a benchmark", :aggregate_failures do
       run_benchmark
 
-      expect(bench_mock).to have_received(:report).with("Sum")
-      expect(bench_mock).to have_received(:report).with("Multiplication")
+      expect(bench_mock).to have_received(:report).with("sum")
+      expect(bench_mock).to have_received(:report).with("multiplication")
       expect(bench_mock).to have_received(:compare!)
     end