coverband 4.2.6 → 5.0.0.rc.5

data/lib/coverband/utils/railtie.rb

@@ -10,34 +10,51 @@ module Coverband
   Rails::Engine.prepend(RailsEagerLoad)
 
   class Railtie < Rails::Railtie
-    initializer 'coverband.configure' do |app|
+    initializer "coverband.configure" do |app|
       begin
         app.middleware.use Coverband::BackgroundMiddleware
 
         if Coverband.configuration.track_views
-          CoverbandViewTracker = Coverband::Collectors::ViewTracker.new
-          Coverband.configuration.view_tracker = CoverbandViewTracker
+          COVERBAND_VIEW_TRACKER = if Coverband.coverband_service?
+            Coverband::Collectors::ViewTrackerService.new
+          else
+            Coverband::Collectors::ViewTracker.new
+          end
+
+          Coverband.configuration.view_tracker = COVERBAND_VIEW_TRACKER
 
           ActiveSupport::Notifications.subscribe(/render_partial.action_view|render_template.action_view/) do |name, start, finish, id, payload|
-            CoverbandViewTracker.track_views(name, start, finish, id, payload) unless name.include?('!')
+            COVERBAND_VIEW_TRACKER.track_views(name, start, finish, id, payload) unless name.include?("!")
           end
         end
       rescue Redis::CannotConnectError => error
         Coverband.configuration.logger.info "Redis is not available (#{error}), Coverband not configured"
-        Coverband.configuration.logger.info 'If this is a setup task like assets:precompile feel free to ignore'
+        Coverband.configuration.logger.info "If this is a setup task like assets:precompile feel free to ignore"
       end
     end
 
     config.after_initialize do
       unless Coverband.tasks_to_ignore?
+        Coverband.configure
         Coverband.eager_loading_coverage!
         Coverband.report_coverage
         Coverband.runtime_coverage!
       end
     end
 
+    config.before_configuration do
+      unless ENV["COVERBAND_DISABLE_AUTO_START"]
+        begin
+          Coverband.start
+        rescue Redis::CannotConnectError => error
+          Coverband.configuration.logger.info "Redis is not available (#{error}), Coverband not configured"
+          Coverband.configuration.logger.info "If this is a setup task like assets:precompile feel free to ignore"
+        end
+      end
+    end
+
     rake_tasks do
-      load 'coverband/utils/tasks.rb'
+      load "coverband/utils/tasks.rb"
     end
   end
 end

data/lib/coverband/utils/relative_file_converter.rb

@@ -24,8 +24,8 @@ module Coverband
         @cache[file] ||= begin
                            relative_file = file
                            @roots.each do |root|
-                             relative_file = file.gsub(/^#{root}/, '.')
-                             break relative_file if relative_file.start_with?('.')
+                             relative_file = file.gsub(/^#{root}/, ".")
+                             break relative_file if relative_file.start_with?(".")
                            end
                            relative_file
                          end

data/lib/coverband/utils/result.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'digest/sha1'
-require 'forwardable'
+require "digest/sha1"
+require "forwardable"
 
 ####
 # Thanks for all the help SimpleCov https://github.com/colszowka/simplecov
@@ -31,9 +31,9 @@ module Coverband
       def initialize(original_result)
         @original_result = (original_result || {}).freeze
 
-        @files = Coverband::Utils::FileList.new(@original_result.map do |filename, coverage|
+        @files = Coverband::Utils::FileList.new(@original_result.map { |filename, coverage|
           Coverband::Utils::SourceFile.new(filename, coverage) if File.file?(filename)
-        end.compact.sort_by(&:short_name))
+        }.compact.sort_by(&:short_name))
       end
 
       # Returns all filenames for source files contained in this result
@@ -41,11 +41,6 @@ module Coverband
         files.map(&:filename)
       end
 
-      # Returns a Hash of groups for this result. Define groups using Coverband.add_group 'Models', 'app/models'
-      def groups
-        @groups ||= FileGroups.new(files).grouped_results
-      end
-
       # Defines when this result has been created. Defaults to Time.now
       def created_at
         @created_at ||= Time.now
@@ -60,8 +55,8 @@ module Coverband
           Dir[tracked_files].each do |file|
             absolute = File.expand_path(file)
             result[absolute] ||= {
-              'data' => Array.new(File.foreach(absolute).count) { 0 },
-              'never_loaded' => true
+              "data" => Array.new(File.foreach(absolute).count) { 0 },
+              "never_loaded" => true
             }
           end
         end

data/lib/coverband/utils/results.rb

@@ -45,16 +45,6 @@ module Coverband
         eager_file.relevant_lines - eager_file.covered_lines_count
       end
 
-      ###
-      # TODO: Groups still have some issues, this should be generic for groups, but right now gem_name
-      # is specifically called out, need to revisit all gorups code.
-      ###
-      def group_file_list_with_type(group, file_list, results_type)
-        return unless get_results(results_type)
-
-        get_results(results_type).groups[group].find { |gem_files| gem_files.first.gem_name == file_list.first.gem_name }
-      end
-
       def file_from_path_with_type(full_path, results_type = :merged)
         return unless get_results(results_type)
 

data/lib/coverband/utils/source_file.rb

@@ -32,14 +32,14 @@ module Coverband
         alias number line_number
 
         def initialize(src, line_number, coverage)
-          raise ArgumentError, 'Only String accepted for source' unless src.is_a?(String)
-          raise ArgumentError, 'Only Integer accepted for line_number' unless line_number.is_a?(Integer)
-          raise ArgumentError, 'Only Integer and nil accepted for coverage' unless coverage.is_a?(Integer) || coverage.nil?
+          raise ArgumentError, "Only String accepted for source" unless src.is_a?(String)
+          raise ArgumentError, "Only Integer accepted for line_number" unless line_number.is_a?(Integer)
+          raise ArgumentError, "Only Integer and nil accepted for coverage" unless coverage.is_a?(Integer) || coverage.nil?
 
-          @src         = src
+          @src = src
           @line_number = line_number
-          @coverage    = coverage
-          @skipped     = false
+          @coverage = coverage
+          @skipped = false
         end
 
         # Returns true if this is a line that should have been covered, but was not
@@ -71,10 +71,10 @@ module Coverband
         # The status of this line - either covered, missed, skipped or never. Useful i.e. for direct use
         # as a css class in report generation
         def status
-          return 'skipped' if skipped?
-          return 'never' if never?
-          return 'missed' if missed?
-          return 'covered' if covered?
+          return "skipped" if skipped?
+          return "never" if never?
+          return "missed" if missed?
+          return "covered" if covered?
         end
       end
 
@@ -89,17 +89,17 @@ module Coverband
       attr_reader :last_updated_at
       # meta data that the file was never loaded during boot or runtime
       attr_reader :never_loaded
-      NOT_AVAILABLE = 'not available'
+      NOT_AVAILABLE = "not available"
 
       def initialize(filename, file_data)
         @filename = filename
         @runtime_relavant_lines = nil
         if file_data.is_a?(Hash)
-          @coverage = file_data['data']
+          @coverage = file_data["data"]
           @first_updated_at = @last_updated_at = NOT_AVAILABLE
-          @first_updated_at = Time.at(file_data['first_updated_at']) if file_data['first_updated_at']
-          @last_updated_at =  Time.at(file_data['last_updated_at']) if file_data['last_updated_at']
-          @never_loaded = file_data['never_loaded'] || false
+          @first_updated_at = Time.at(file_data["first_updated_at"]) if file_data["first_updated_at"]
+          @last_updated_at = Time.at(file_data["last_updated_at"]) if file_data["last_updated_at"]
+          @never_loaded = file_data["never_loaded"] || false
         else
           # TODO: Deprecate this code path this was backwards compatability from 3-4
           @coverage = file_data
@@ -117,14 +117,14 @@ module Coverband
 
       # The path to this source file relative to the projects directory
       def project_filename
-        @filename.sub(/^#{Coverband.configuration.root}/, '')
+        @filename.sub(/^#{Coverband.configuration.root}/, "")
       end
 
       # The source code for this file. Aliased as :source
       def src
         # We intentionally read source code lazily to
         # suppress reading unused source code.
-        @src ||= File.open(filename, 'rb', &:readlines)
+        @src ||= File.open(filename, "rb", &:readlines)
       end
       alias source src
 
@@ -138,9 +138,9 @@ module Coverband
       def build_lines
         coverage_exceeding_source_warn if coverage.size > src.size
 
-        lines = src.map.with_index(1) do |src, i|
+        lines = src.map.with_index(1) { |src, i|
           Coverband::Utils::SourceFile::Line.new(src, i, coverage[i - 1])
-        end
+        }
 
         process_skipped_lines(lines)
       end
@@ -241,23 +241,14 @@ module Coverband
       # was at the start of the file name
       # I had previously patched this in my local Rails app
       def short_name
-        filename.sub(/^#{Coverband.configuration.root}/, '.')
-                .sub(%r{^.*\/gems\/}, '.')
-                .gsub(%r{^\.\/}, '')
+        filename.sub(/^#{Coverband.configuration.root}/, ".")
+          .gsub(%r{^\.\/}, "")
       end
 
       def relative_path
         RelativeFileConverter.convert(filename)
       end
 
-      def gem?
-        filename =~ %r{^.*\/gems\/}
-      end
-
-      def gem_name
-        gem? ? short_name.split('/').first.gsub(%r{^\.}, '') : nil
-      end
-
       private
 
       # ruby 1.9 could use Float#round(places) instead

data/lib/coverband/utils/tasks.rb

@@ -4,25 +4,22 @@ namespace :coverband do
   # handles configuring in require => false and COVERBAND_DISABLE_AUTO_START cases
   Coverband.configure unless Coverband.configured?
 
-  desc 'report runtime Coverband code coverage'
+  desc "report runtime Coverband code coverage"
   task :coverage do
-    if Coverband.configuration.reporter == 'scov'
-      Coverband::Reporters::HTMLReport.new(Coverband.configuration.store).report
-    else
-      Coverband::Reporters::ConsoleReport.report(Coverband.configuration.store)
-    end
+    Coverband::Reporters::ConsoleReport.report(Coverband.configuration.store)
   end
 
-  desc 'report runtime Coverband code coverage'
+  desc "report runtime Coverband code coverage"
   task :coverage_server do
-    Rake.application['environment'].invoke if Rake::Task.task_defined?('environment')
-    Rack::Server.start app: Coverband::Reporters::Web.new, Port: ENV.fetch('COVERBAND_COVERAGE_PORT', 1022).to_i
+    Rake.application["environment"].invoke if Rake::Task.task_defined?("environment")
+    Coverband.configuration.store.merge_mode = true if Coverband.configuration.store.is_a?(Coverband::Adapters::FileStore)
+    Rack::Server.start app: Coverband::Reporters::Web.new, Port: ENV.fetch("COVERBAND_COVERAGE_PORT", 1022).to_i
   end
 
   ###
   # clear data helpful for development or after configuration issues
   ###
-  desc 'reset Coverband coverage data, helpful for development, debugging, etc'
+  desc "reset Coverband coverage data, helpful for development, debugging, etc"
   task :clear do
     Coverband.configuration.store.clear!
   end
@@ -30,7 +27,7 @@ namespace :coverband do
   ###
   # Updates the data in the coverband store from one format to another
   ###
-  desc 'upgrade previous Coverband datastore to latest format'
+  desc "upgrade previous Coverband datastore to latest format"
   task :migrate do
     Coverband.configuration.store.migrate!
   end

data/lib/coverband/version.rb

@@ -5,5 +5,5 @@
 # use format '4.2.1.rc.1' ~> 4.2.1.rc to prerelease versions like v4.2.1.rc.2 and v4.2.1.rc.3
 ###
 module Coverband
-  VERSION = '4.2.6'
+  VERSION = "5.0.0.rc.5"
 end

data/public/application.js

@@ -33,23 +33,6 @@ $(document).ready(function() {
     ]
   });
 
-  // Configuration for fancy sortable tables for source file groups
-  $(".gem_list").dataTable({
-    aaSorting: [[1, "asc"]],
-    bPaginate: false,
-    bJQueryUI: true,
-    aoColumns: [
-      null,
-      { sType: "percent" },
-      { sType: "percent" },
-      null,
-      null,
-      null,
-      null,
-      null
-    ]
-  });
-
   // Syntax highlight all files up front - deactivated
   // $('.source_table pre code').each(function(i, e) {hljs.highlightBlock(e, '  ')});
 
@@ -184,17 +167,6 @@ $(document).ready(function() {
     return false;
   });
 
-  $("a.gem-link").live("click", function() {
-    $(".file_list_container").hide();
-    $(".file_list_container" + $(this).attr("href")).show();
-    window.location.href =
-      window.location.href.split("#")[0] +
-      $(this)
-        .attr("href")
-        .replace("#", "#_");
-    return false;
-  });
-
   if (jQuery.url.attr("anchor")) {
     var anchor = jQuery.url.attr("anchor");
     // source file hash
@@ -203,8 +175,6 @@ $(document).ready(function() {
     } else {
       if ($(".group_tabs a." + anchor.replace("_", "")).length > 0) {
         $(".group_tabs a." + anchor.replace("_", "")).click();
-      } else {
-        $("a.gem-link[href=#" + anchor.replace("_", "") + "]").click();
       }
     }
   } else {

data/test/benchmarks/benchmark.rake

@@ -14,9 +14,11 @@ namespace :benchmarks do
       run_gc
     end
 
-    def warmup_stats(*); end
+    def warmup_stats(*)
+    end
 
-    def add_report(*); end
+    def add_report(*)
+    end
 
     private
 
@@ -29,46 +31,47 @@ namespace :benchmarks do
 
   # desc 'setup standard benchmark'
   task :setup do
-    require 'benchmark'
-    require 'benchmark/ips'
+    require "benchmark"
+    require "benchmark/ips"
+    require "redis"
 
-    if ENV['COVERAGE'] || ENV['ONESHOT']
-      require 'coverage'
-      ::Coverage.start(oneshot_lines: !!ENV['ONESHOT'])
+    if ENV["COVERAGE"] || ENV["ONESHOT"]
+      require "coverage"
+      ::Coverage.start(oneshot_lines: !!ENV["ONESHOT"])
     end
-    require 'redis'
-    require 'coverband'
-    require File.join(File.dirname(__FILE__), 'dog')
+    require "coverband"
+
+    require File.join(File.dirname(__FILE__), "dog")
   end
 
   def benchmark_redis_store
-    redis = if ENV['REDIS_TEST_URL']
-              Redis.new(url: ENV['REDIS_TEST_URL'])
-            else
-              Redis.new
-            end
+    redis = if ENV["REDIS_TEST_URL"]
+      Redis.new(url: ENV["REDIS_TEST_URL"])
+    else
+      Redis.new
+    end
     Coverband::Adapters::RedisStore.new(redis,
-                                        redis_namespace: 'coverband_bench')
+      redis_namespace: "coverband_bench")
   end
 
   # desc 'set up coverband with Redis'
   task :setup_redis do
     Coverband.configure do |config|
-      config.root                = Dir.pwd
-      config.logger              = $stdout
-      config.store               = benchmark_redis_store
-      config.use_oneshot_lines_coverage = true if ENV['ONESHOT']
-      config.simulate_oneshot_lines_coverage = true if ENV['SIMULATE_ONESHOT']
+      config.root = Dir.pwd
+      config.logger = $stdout
+      config.store = benchmark_redis_store
+      config.use_oneshot_lines_coverage = true if ENV["ONESHOT"]
+      config.simulate_oneshot_lines_coverage = true if ENV["SIMULATE_ONESHOT"]
     end
   end
 
   # desc 'set up coverband with filestore'
   task :setup_file do
     Coverband.configure do |config|
-      config.root                = Dir.pwd
-      config.logger              = $stdout
-      file_path                  = '/tmp/benchmark_store.json'
-      config.store               = Coverband::Adapters::FileStore.new(file_path)
+      config.root = Dir.pwd
+      config.logger = $stdout
+      file_path = "/tmp/benchmark_store.json"
+      config.store = Coverband::Adapters::FileStore.new(file_path)
     end
   end
 
@@ -83,14 +86,14 @@ namespace :benchmarks do
     suite = GCSuite.new
     Benchmark.ips do |x|
       x.config(time: 12, warmup: 5, suite: suite)
-      x.report 'coverband' do
+      x.report "coverband" do
         work
         Coverband.report_coverage
       end
-      x.report 'no coverband' do
+      x.report "no coverband" do
         work
       end
-      x.hold! 'temp_results' if hold_work
+      x.hold! "temp_results" if hold_work
       x.compare!
     end
     Coverband::Collectors::Coverage.instance.reset_instance
@@ -100,7 +103,7 @@ namespace :benchmarks do
   NON_NIL_LINES = 18
   def fake_line_numbers
     LINES.times.map do |line|
-      coverage = (line < NON_NIL_LINES) ? rand(5) : nil
+      coverage = line < NON_NIL_LINES ? rand(5) : nil
     end
   end
 
@@ -141,18 +144,18 @@ namespace :benchmarks do
     5.times { store.save_report(report) }
     Benchmark.ips do |x|
       x.config(time: 15, warmup: 5)
-      x.report('store_reports_all') { store.save_report(report) }
+      x.report("store_reports_all") { store.save_report(report) }
     end
     keys_subset = report.keys.first(100)
     report_subset = report.select { |key, _value| keys_subset.include?(key) }
     Benchmark.ips do |x|
       x.config(time: 20, warmup: 5)
-      x.report('store_reports_subset') { store.save_report(report_subset) }
+      x.report("store_reports_subset") { store.save_report(report_subset) }
     end
   end
 
   def measure_memory
-    require 'memory_profiler'
+    require "memory_profiler"
     report = fake_report
     store = benchmark_redis_store
     store.clear!
@@ -165,21 +168,21 @@ namespace :benchmarks do
     capture = StringIO.new
     $stdout = capture
 
-    MemoryProfiler.report do
+    MemoryProfiler.report {
       10.times { store.save_report(report) }
-    end.pretty_print
+    }.pretty_print
     data = $stdout.string
     $stdout = previous_out
-    unless data.match('Total retained:  0 bytes')
+    unless data.match?("Total retained:  0 bytes")
       puts data
-      raise 'leaking memory!!!'
+      raise "leaking memory!!!"
     end
   ensure
     $stdout = previous_out
   end
 
   def measure_memory_report_coverage
-    require 'memory_profiler'
+    require "memory_profiler"
     report = fake_report
     store = benchmark_redis_store
     store.clear!
@@ -192,7 +195,7 @@ namespace :benchmarks do
     capture = StringIO.new
     $stdout = capture
 
-    MemoryProfiler.report do
+    MemoryProfiler.report {
       10.times do
         Coverband.report_coverage
         ###
@@ -202,12 +205,12 @@ namespace :benchmarks do
         ###
         Coverband::Collectors::Delta.class_variable_set(:@@previous_coverage, nil)
       end
-    end.pretty_print
+    }.pretty_print
     data = $stdout.string
     $stdout = previous_out
-    unless data.match('Total retained:  0 bytes')
+    unless data.match?("Total retained:  0 bytes")
       puts data
-      raise 'leaking memory!!!'
+      raise "leaking memory!!!"
     end
   ensure
     $stdout = previous_out
@@ -219,7 +222,7 @@ namespace :benchmarks do
   # not including in test suite but we can try to figure it out and fix.
   ###
   def measure_configure_memory
-    require 'memory_profiler'
+    require "memory_profiler"
     # warmup
     3.times { Coverband.configure }
 
@@ -227,31 +230,31 @@ namespace :benchmarks do
     capture = StringIO.new
     $stdout = capture
 
-    MemoryProfiler.report do
+    MemoryProfiler.report {
       10.times do
         Coverband.configure do |config|
-          redis_url = ENV['CACHE_REDIS_URL'] || ENV['REDIS_URL']
-          config.store = Coverband::Adapters::RedisStore.new(Redis.new(url: redis_url), redis_namespace: 'coverband_bench_data')
+          redis_url = ENV["CACHE_REDIS_URL"] || ENV["REDIS_URL"]
+          config.store = Coverband::Adapters::RedisStore.new(Redis.new(url: redis_url), redis_namespace: "coverband_bench_data")
         end
       end
-    end.pretty_print
+    }.pretty_print
     data = $stdout.string
     $stdout = previous_out
-    unless data.match('Total retained:  0 bytes')
+    unless data.match?("Total retained:  0 bytes")
       puts data
-      raise 'leaking memory!!!'
+      raise "leaking memory!!!"
     end
   ensure
     $stdout = previous_out
   end
 
-  desc 'checks memory of collector'
+  desc "checks memory of collector"
   task memory_check: [:setup] do
     # require 'pry-byebug'
-    require 'objspace'
-    puts 'memory load check'
+    require "objspace"
+    puts "memory load check"
     puts(ObjectSpace.memsize_of_all / 2**20)
-    data = File.read('./tmp/debug_data.json')
+    data = File.read("./tmp/debug_data.json")
     # about 2mb
     puts(ObjectSpace.memsize_of(data) / 2**20)
 
@@ -288,65 +291,67 @@ namespace :benchmarks do
     json_data = nil
     GC.start
     puts(ObjectSpace.memsize_of_all / 2**20)
-    debugger
-    puts 'done'
+
+    # dig in some more...
+    # debugger
+    puts "done"
   end
 
-  desc 'runs memory reporting on Redis store'
+  desc "runs memory reporting on Redis store"
   task memory_reporting: [:setup] do
-    puts 'runs memory benchmarking to ensure we dont leak'
+    puts "runs memory benchmarking to ensure we dont leak"
     measure_memory
   end
 
-  desc 'runs memory reporting on report_coverage'
+  desc "runs memory reporting on report_coverage"
   task memory_reporting_report_coverage: [:setup] do
-    puts 'runs memory benchmarking on report_coverage to ensure we dont leak'
+    puts "runs memory benchmarking on report_coverage to ensure we dont leak"
     measure_memory_report_coverage
   end
 
-  desc 'runs memory reporting on configure'
+  desc "runs memory reporting on configure"
   task memory_configure_reporting: [:setup] do
-    puts 'runs memory benchmarking on configure to ensure we dont leak'
+    puts "runs memory benchmarking on configure to ensure we dont leak"
     measure_configure_memory
   end
 
-  desc 'runs memory leak check via Rails tests'
+  desc "runs memory leak check via Rails tests"
   task memory_rails: [:setup] do
-    puts 'runs memory rails test to ensure we dont leak'
+    puts "runs memory rails test to ensure we dont leak"
     puts `COVERBAND_MEMORY_TEST=true bundle exec test/forked/rails_full_stack_test.rb`
   end
 
-  desc 'runs memory leak checks'
+  desc "runs memory leak checks"
   task memory: %i[memory_reporting memory_reporting_report_coverage memory_rails] do
-    puts 'done'
+    puts "done"
   end
 
-  desc 'runs benchmarks on reporting large sets of files to redis'
+  desc "runs benchmarks on reporting large sets of files to redis"
   task redis_reporting: [:setup] do
-    puts 'runs benchmarks on reporting large sets of files to redis'
+    puts "runs benchmarks on reporting large sets of files to redis"
     reporting_speed
   end
 
   # desc 'runs benchmarks on default redis setup'
   task run_redis: %i[setup setup_redis] do
-    puts 'Coverband configured with default Redis store'
+    puts "Coverband configured with default Redis store"
     run_work(true)
   end
 
   def run_big
-    require 'memory_profiler'
-    require './test/unique_files'
+    require "memory_profiler"
+    require "./test/unique_files"
 
-    4000.times { |index| require_unique_file('big_dog.rb.erb', dog_number: index) }
+    4000.times { |index| require_unique_file("big_dog.rb.erb", dog_number: index) }
     # warmup
     3.times { Coverband.report_coverage }
     dogs = 400.times.map { |index| Object.const_get("Dog#{index}") }
-    MemoryProfiler.report do
+    MemoryProfiler.report {
       10.times do
         dogs.each(&:bark)
         Coverband.report_coverage
       end
-    end.pretty_print
+    }.pretty_print
   end
 
   task run_big: %i[setup setup_redis] do
@@ -358,18 +363,18 @@ namespace :benchmarks do
 
   # desc 'runs benchmarks file store'
   task run_file: %i[setup setup_file] do
-    puts 'Coverband configured with file store'
+    puts "Coverband configured with file store"
     run_work(true)
   end
 
-  desc 'benchmarks external requests to coverband_demo site'
+  desc "benchmarks external requests to coverband_demo site"
   task :coverband_demo do
     # for local testing
     # puts `ab -n 500 -c 5 "http://127.0.0.1:3000/posts"`
     puts `ab -n 2000 -c 10 "https://coverband-demo.herokuapp.com/posts"`
   end
 
-  desc 'benchmarks external requests to coverband_demo site'
+  desc "benchmarks external requests to coverband_demo site"
   task :coverband_demo_graph do
     # for local testing
     # puts `ab -n 200 -c 5 "http://127.0.0.1:3000/posts"`
@@ -379,38 +384,38 @@ namespace :benchmarks do
     `open tmp/timeseries.jpg`
   end
 
-  desc 'benchmark initialization of rails'
+  desc "benchmark initialization of rails"
   task :init_rails do
-    require 'benchmark'
-    require 'benchmark/ips'
+    require "benchmark"
+    require "benchmark/ips"
     Benchmark.ips do |x|
       x.config(time: 60, warmup: 0)
-      x.report('init_rails') do
-        system('bundle exec rake init_rails -f ./test/benchmarks/init_rails.rake')
+      x.report("init_rails") do
+        system("bundle exec rake init_rails -f ./test/benchmarks/init_rails.rake")
       end
     end
   end
 
-  desc 'compare Coverband Ruby Coverage with Filestore with normal Ruby'
+  desc "compare Coverband Ruby Coverage with Filestore with normal Ruby"
   task :compare_file do
-    puts 'comparing Coverage loaded/not, this takes some time for output...'
-    puts 'coverage loaded'
+    puts "comparing Coverage loaded/not, this takes some time for output..."
+    puts "coverage loaded"
     puts `COVERAGE=true rake benchmarks:run_file`
-    puts 'just the work'
+    puts "without coverage"
     puts `rake benchmarks:run_file`
   end
 
-  desc 'compare Coverband Ruby Coverage with Redis and normal Ruby'
+  desc "compare Coverband Ruby Coverage with Redis and normal Ruby"
   task :compare_redis do
-    puts 'comparing Coverage loaded/not, this takes some time for output...'
-    puts 'coverage loaded'
+    puts "comparing Coverage loaded/not, this takes some time for output..."
+    puts "coverage loaded"
     puts `COVERAGE=true rake benchmarks:run_redis`
-    puts 'just the work'
+    puts "without coverage"
     puts `rake benchmarks:run_redis`
   end
 end
 
-desc 'runs benchmarks'
-task benchmarks: ['benchmarks:redis_reporting',
-                  'benchmarks:compare_file',
-                  'benchmarks:compare_redis']
+desc "runs benchmarks"
+task benchmarks: ["benchmarks:redis_reporting",
+  "benchmarks:compare_file",
+  "benchmarks:compare_redis"]