coverband 5.0.0.rc.4 → 5.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c83a78e28c79671ef144ffa38e40ee061a873ef6a207437f01923188c17d3cf0
-  data.tar.gz: 80a3c0a46cfb55a5972139e034733d9fb6541f0d547caff1f004203b9ddb0484
+  metadata.gz: f73e90b7f1451c8f7a47355b680b361fa96e7ab665c1b135282e1cf2f87db8ac
+  data.tar.gz: e0bd9b0e10bd500dcb814091a048902e4123fa09a3532e8bffb7108cf69bea53
 SHA512:
-  metadata.gz: 0b2f9965e62d911acc562c73df79f4ba13ca0df48bb18258a9069b71d3323f9167385b11fe25772d32d23575338e98e04f721dd57f6562ca40171e39f1cae591
-  data.tar.gz: 6ca2447bbddf8ac8bd6a92e4544a2e38b5a7bf84d49e9e02230ef3a2e0044436ce70d9907dad8a7d163cf909369ad4f089af49bb6a50da5b62982e3001aa48bd
+  metadata.gz: 67e8d3f7f1c925f030b828eb4a4b37a1b35e887341da8cc6956d4c84ec71cb2a31395f656d48ad3d1782e81404624ae95604a4d0198af4cef2e2c43fc7e08578
+  data.tar.gz: 3d4a7456beb6adfa1e6eeec4bacb35ef4f3b3a25049b9540b603e5f9bcd65aba1d370c1cbf446c7835106d8ccc536209fbdc74f77949847e91262cdffa0c715c

data/.standard.yml

@@ -13,6 +13,7 @@ ignore: # default: []
       - Style/RedundantRegexpEscape # fix later, enforcement changed
       - Layout/ArrayAlignment # WTF all of master broken from a few changes in rubo
       - Performance/RegexpMatch # Rubocop / standardrb have this WRONG for Ruby 2.3/2.4 not compatiable
+      - Style/GlobalStdStream # Rubocop / standardrb have this WRONG for Ruby 2.3/2.4 not compatiable
   - "vendor/**/*"
   - "pkg/**/*"
   - "test/**/*":

data/README.md

@@ -10,7 +10,7 @@
 <p align="center">
   <a href="#key-features">Key Features</a> •
   <a href="#installation">Installation</a> •
-  <a href="#coverage-report">Coverage Report</a> •
+  <a href="#coverband-web-ui">Coverband Web UI</a> •
   <a href="#advanced-config">Advanced Config</a> •
   <a href="#license">License</a> •
   <a href="/changes.md">Change Log / Roadmap</a> •
@@ -28,7 +28,7 @@ The primary goal of Coverband is giving deep insight into your production runtim
 - Low performance overhead
 - Simple setup and configuration
 - Out of the box support for all standard code execution paths (web, cron, background jobs, rake tasks, etc)
-- Splits load time (Rails eager load) and Run time metrics
+- Splits load time (Rails eager load) and runtime metrics
 - Easy to understand actionable insights from the report
 - Development mode, offers deep insight of code usage details (number of LOC execution during single request, etc) during development.
 - Mountable web interface to easily share reports
@@ -41,7 +41,7 @@ Coverband stores coverage data in Redis. The Redis endpoint is looked for in thi
 
 1. `ENV['COVERBAND_REDIS_URL']`
 2. `ENV['REDIS_URL']`
-3. `localhost`
+3. `localhost:6379`
 
 The redis store can also be explicitly defined within the coverband.rb. See [advanced config](#advanced-config).
 
@@ -53,8 +53,6 @@ Add this line to your application's `Gemfile`, remember to `bundle install` afte
 gem 'coverband'
 ```
 
-## Upgrading to Latest
-
 ### No custom code or middleware required
 
 With older versions of coverband, projects would report to redis using rack or sidekiq middleware. After coverband 4.0, this should no longer be required and could cause performance issues. Reporting to redis is now automatically done within a background thread with no custom code needed.
@@ -67,7 +65,7 @@ The Railtie integration means you shouldn't need to do anything else other than
 
 ## Sinatra
 
-For the best coverage you want this loaded as early as possible. I have been putting it directly in my `config.ru` but you could use an initializer, though you may end up missing some boot up coverage. To start collection require Coverband as early as possible.
+For the best coverage you want this loaded as early as possible. We recommend requiring cover band directly in the `config.ru`.  Requiring coverband within an initializer could also work, but you may end up missing some boot up coverage. To start collection require Coverband as early as possible.
 
 ```ruby
 require 'coverband'
@@ -77,7 +75,33 @@ use Coverband::BackgroundMiddleware
 run ActionController::Dispatcher.new
 ```
 
-# Coverage Report
+## Coverband Web UI
+
+![image](https://raw.github.com/danmayer/coverband/master/docs/coverband_web_ui.png)
+
+> The web index as available on the [Coverband Demo site](https://coverband-demo.herokuapp.com/coverage?#_Coverage)
+
+- View overall coverage information
+
+- Drill into individual file coverage
+
+- View individual file details
+
+- Clear Coverage - disabled by default as it could be considered a dangerous operation in production. Enable with `config.web_enable_clear` or leave off and clear from [rake task](#clear-coverage).
+
+  - Clear coverage report
+
+    This will clear the coverage data. This wipes out all collected data. 
+
+  - Clear individual file coverage
+
+    This will clear the details of the file you are looking at. This is helpful if you don't want to lose all Coverage data but made a change that you expect would impact a particular file.
+
+- Force coverage collection
+
+  This triggers coverage collection on the current webserver process. Useful in development but confusing in production environments where many ruby processes are usually running.
+
+### Mounting as a Rack App
 
 Coverband comes with a mountable rack app for viewing reports. For Rails this can be done in `config/routes.rb` with:
 
@@ -97,49 +121,23 @@ Rails.application.routes.draw do
 end
 ```
 
-or you can enable basic auth by setting `ENV['COVERBAND_PASSWORD']` or via your configuration `config.password = 'my_pass'`
+or you can enable basic auth by setting `ENV['COVERBAND_PASSWORD']` or via your configuration `config.password = <YOUR_COMPLEX_UNGUESSABLE_PASSWORD>`
 
-### Coverband Web Endpoint
+### Standalone
 
-The web endpoint is a barebones endpoint that you can either expose direct (after authentication) or you can just link to the actions you wish to expose. The index is intended as a example to showcase all the features.
+The coverage server can also be started standalone with a rake task:
 
-![image](https://raw.github.com/danmayer/coverband/master/docs/coverband_web_ui.png)
-
-> The web index as available on the Coverband Demo site
-
-- **force coverage collection:** This triggers coverage collection on the current webserver process
-- **clear coverage report:** This will clear the coverage data. This wipes out all collected data (**dangerous**)
-- View individual file details
-- **clear individual file coverage:** This will clear the details of the file you are looking at. This is helpful if you don't want to lose all Coverage data but made a change that you expect would impact a particular file.
-
-### JRuby Support
-
-Coverband is compatible with JRuby. If you want to run on JRuby note that I haven't benchmarked and I believe the perf impact on older versions of JRuby could be significant, improved Coverage support is in [JRuby master](https://github.com/jruby/jruby/pull/6180), and will be in the next release.
-
-- older versions of JRuby need tracing enabled to work (and this could cause bad performance)
-  - run Jruby with the `--debug` option
-  - add into your `.jrubyrc` the `debug.fullTrace=true` setting
-- For best performance the `oneshot_lines` is recommended, and in the latest releases should have very low overhead
-- See JRuby support in a Rails app configured to run via JRuby, in [Coverband Demo](https://github.com/coverband-service/coverband_demo)
-- JRuby is tested via CI against Rails 5 and 6
-
-### Rake Tasks
-
-The rake task generates a report locally and opens a browser pointing to `coverage/index.html`.
-
-`rake coverband:coverage`
-
-This is mostly useful in your local development environment.
-
-##### Example Output
+```
+ bundle exec rake coverband:coverage_server
+```
 
-Since Coverband is [Simplecov](https://github.com/colszowka/simplecov) output compatible it should work with any of the `SimpleCov::Formatter`'s available. The output below is produced using the default Simplecov HTML formatter.
+The web UI should then be available here: http://localhost:1022/
 
-Index Page
-![image](https://raw.github.com/danmayer/coverband/master/docs/coverband_index.png)
+This is especially useful for projects that are api only and cannot support the mounted rack app. To get production coverage, point coverband at your production redis server and ensure to checkout the production version of your project code locally.
 
-Details on an example Sinatra app
-![image](https://raw.github.com/danmayer/coverband/master/docs/coverband_details.png)
+```
+ COVERBAND_REDIS_URL=redis://username:password:redis_production_server:2322 bundle exec rake coverband:coverage_server
+```
 
 # Coverband Demo
 
@@ -245,7 +243,7 @@ end
 
 ### Avoiding Cache Stampede
 
-If you have many servers and they all hit Redis at the same time you can see spikes in your Redis CPU, and memory. This is do to a concept called [cache stampede](https://en.wikipedia.org/wiki/Cache_stampede). It is better to spread out the reporting across your servers. A simple way to do this is to add a random wiggle on your background reporting. This configuration option allows a wiggle. The right amount of wiggle depends on the numbers of servers you have and how willing you are to have delays in your coverage reporting. I would recommend at least 1 second per server.
+If you have many servers and they all hit Redis at the same time you can see spikes in your Redis CPU, and memory. This is do to a concept called [cache stampede](https://en.wikipedia.org/wiki/Cache_stampede). It is better to spread out the reporting across your servers. A simple way to do this is to add a random wiggle on your background reporting. This configuration option allows a wiggle. The right amount of wiggle depends on the numbers of servers you have and how willing you are to have delays in your coverage reporting. I would recommend at least 1 second per server. Note, the default wiggle is set to 30 seconds.
 
 Add a wiggle (in seconds) to the background thread to avoid all your servers reporting at the same time:
 
@@ -253,11 +251,12 @@ Add a wiggle (in seconds) to the background thread to avoid all your servers rep
 
 ### Redis Hash Store
 
-Coverband on very high volume sites with many server processes reporting can have a race condition. To resolve the race condition and reduce Ruby memory overhead we have introduced a new Redis storage option. This moves the some of the work from the Ruby processes to Redis. It is worth noting because of this, it has a larger demands on the Redis server. So adjust your Redis instance accordingly. To help reduce the extra redis load you can also change the background reporting time period.
+Coverband on very high volume sites with many server processes reporting can have a race condition which can cause hit counts to be inaccurate. To resolve the race condition and reduce Ruby memory overhead we have introduced a new Redis storage option. This moves the some of the work from the Ruby processes to Redis. It is worth noting because of this, it has a larger demands on the Redis server. So adjust your Redis instance accordingly. To help reduce the extra redis load you can also change the background reporting frequency.
 
-- set the new Redis store: `config.store = Coverband::Adapters::HashRedisStore.new(Redis.new(url: redis_url))`
-- adjust from default 30s reporting `config.background_reporting_sleep_seconds = 120`
-- reminder it is recommended to have a unique Redis per workload (background jobs, caching, Coverband), for this store, it may be more important to have a dedicated Redis.
+- Use a dedicated coverband redis instance: `config.store = Coverband::Adapters::HashRedisStore.new(Redis.new(url: redis_url))`
+- Adjust from default 30s reporting `config.background_reporting_sleep_seconds = 120`
+
+See more discussion [here](https://github.com/danmayer/coverband/issues/384).
 
 ### Clear Coverage
 
@@ -265,6 +264,8 @@ Now that Coverband uses MD5 hashes there should be no reason to manually clear c
 
 `rake coverband:clear`
 
+This can also be done through the web if `config.web_enable_clear` is enabled.
+
 ### Coverage Data Migration
 
 Between the release of 4.0 and 4.1 our data format changed. This resets all your coverage data. If you want to restore your previous coverage data, feel free to migrate.
@@ -339,6 +340,17 @@ We will match Heroku & Ruby's support lifetime, supporting the last 3 major Ruby
 
 For Rails, we will follow the policy of the [Rails team maintenance policy](https://guides.rubyonrails.org/maintenance_policy.html). We officially support the last two major release versions, while providing minimal support (major bugs / security fixes) for an additional version. This means at the moment we primaryly target Rails 6.x, 5.x, and will try to keep current functionality working for Rails 4.x but may release new features that do not work on that target.
 
+### JRuby Support
+
+Coverband is compatible with JRuby. If you want to run on JRuby note that I haven't benchmarked and I believe the perf impact on older versions of JRuby could be significant. Improved Coverage support is in [JRuby master](https://github.com/jruby/jruby/pull/6180), and will be in the next release.
+
+- older versions of JRuby need tracing enabled to work (and this could cause bad performance)
+  - run Jruby with the `--debug` option
+  - add into your `.jrubyrc` the `debug.fullTrace=true` setting
+- For best performance the `oneshot_lines` is recommended, and in the latest releases should have very low overhead
+- See JRuby support in a Rails app configured to run via JRuby, in [Coverband Demo](https://github.com/coverband-service/coverband_demo)
+- JRuby is tested via CI against Rails 5 and 6
+
 # Contributing To Coverband
 
 If you are working on adding features, PRs, or bugfixes to Coverband this section should help get you going.

data/changes.md

@@ -61,13 +61,21 @@ Will be the fully modern release that drops maintenance legacy support in favor
   - drops static report support
   - drops gem support
   - only loaded web reporter files when required
-- improved load order allowing more time for ENV vars (better dotenv, figaro, rails secrets support)
+- configuration improvements
+	- improved load order allowing more time for ENV vars (better dotenv, figaro, rails secrets support)
+	- all config options can be set via coverband config, not requiring ENV var support
+	- deprecation notices on soon to be removed config options
+	- config exceptions on invalid configuration combinations
+	- additional testing around configurations
+	- improved defaults and reduced configuration options
 - improved resque patching pattern
 - improved default ignores
 - additional adapters
   - supports web-service adapter for http coverage collection
   - support log/file adapter
+  - extendable pattern to support any additional backends
 - reduce logs / errors / alerts on bad startup configurations
+- fix: #301 runtime vs eagerload should always remain correct
 
 # Released
 

data/coverband.gemspec

@@ -25,11 +25,13 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "memory_profiler"
   spec.add_development_dependency "minitest"
   spec.add_development_dependency "minitest-fork_executor"
+  spec.add_development_dependency "minitest-stub-const"
   spec.add_development_dependency "mocha", "~> 1.7.0"
   spec.add_development_dependency "rack"
   spec.add_development_dependency "rack-test"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "resque"
+  spec.add_development_dependency "standard", "= 0.2.5"
   spec.add_development_dependency "standardrb"
 
   spec.add_development_dependency "coveralls"

data/lib/coverband/adapters/web_service_store.rb

@@ -3,8 +3,7 @@
 module Coverband
   module Adapters
     ###
-    # WebServiceStore store a merged coverage file to local disk
-    # TODO: add webmock tests
+    # WebServiceStore: store a checkpoint of coverage to a remote service
     ###
     class WebServiceStore < Base
       attr_reader :coverband_url, :process_type, :runtime_env, :hostname, :pid
@@ -56,8 +55,7 @@ module Coverband
         res = Net::HTTP.start(uri.hostname, uri.port, use_ssl: uri.scheme == "https") do |http|
           http.request(req)
         end
-        coverage_data = JSON.parse(res.body)
-        coverage_data
+        JSON.parse(res.body)
       rescue => e
         logger&.error "Coverband: Error while retrieving coverage #{e}" if Coverband.configuration.verbose || Coverband.configuration.service_dev_mode
       end

data/lib/coverband/configuration.rb

@@ -3,7 +3,7 @@
 module Coverband
   class Configuration
     attr_accessor :root_paths, :root,
-      :additional_files, :verbose,
+      :verbose,
       :reporter, :redis_namespace, :redis_ttl,
       :background_reporting_enabled,
       :test_env, :web_enable_clear, :gem_details, :web_debug, :report_on_exit,
@@ -11,7 +11,7 @@ module Coverband
       :view_tracker
     attr_writer :logger, :s3_region, :s3_bucket, :s3_access_key_id,
       :s3_secret_access_key, :password, :api_key, :service_url, :coverband_timeout, :service_dev_mode,
-      :service_test_mode, :proces_type, :report_period, :track_views,
+      :service_test_mode, :process_type, :track_views, :redis_url,
       :background_reporting_sleep_seconds, :reporting_wiggle
 
     attr_reader :track_gems, :ignore, :use_oneshot_lines_coverage
@@ -20,7 +20,6 @@ module Coverband
     # TODO: This is is brittle and not a great solution to avoid deploy time
     # actions polluting the 'runtime' metrics
     #
-    # * should we skip /bin/rails webpacker:compile ?
     # * Perhaps detect heroku deployment ENV var opposed to tasks?
     #####
     IGNORE_TASKS = ["coverband:clear",
@@ -28,6 +27,7 @@ module Coverband
                     "coverband:coverage_server",
                     "coverband:migrate",
                     "assets:precompile",
+                    "webpacker:compile",
                     "db:version",
                     "db:create",
                     "db:drop",
@@ -56,7 +56,6 @@ module Coverband
       @root_paths = []
       @ignore = IGNORE_DEFAULTS.dup
       @search_paths = TRACKED_DEFAULT_PATHS.dup
-      @additional_files = []
       @verbose = false
       @reporter = "scov"
       @logger = nil
@@ -65,8 +64,6 @@ module Coverband
       @background_reporting_sleep_seconds = nil
       @test_env = nil
       @web_enable_clear = false
-      @track_gems = false
-      @gem_details = false
       @track_views = true
       @view_tracker = nil
       @web_debug = false
@@ -85,17 +82,19 @@ module Coverband
       @service_dev_mode = nil
       @service_test_mode = nil
       @proces_type = nil
-      @report_period = nil
+
+      @redis_url = nil
+      @redis_namespace = nil
+      @redis_ttl = 2_592_000 # in seconds. Default is 30 days.
+      @reporting_wiggle = nil
 
       # TODO: these are deprecated
       @s3_region = nil
       @s3_bucket = nil
       @s3_access_key_id = nil
       @s3_secret_access_key = nil
-
-      @redis_namespace = nil
-      @redis_ttl = 2_592_000 # in seconds. Default is 30 days.
-      @reporting_wiggle = nil
+      @track_gems = false
+      @gem_details = false
     end
 
     def logger
@@ -110,25 +109,16 @@ module Coverband
       @password || ENV["COVERBAND_PASSWORD"]
     end
 
-    def s3_bucket
-      puts "deprecated, s3 is no longer support"
-    end
-
-    def s3_region
-      puts "deprecated, s3 is no longer support"
-    end
-
-    def s3_access_key_id
-      puts "deprecated, s3 is no longer support"
-    end
-
-    def s3_secret_access_key
-      puts "deprecated, s3 is no longer support"
-    end
-
+    # The adjustments here either protect the redis or service from being overloaded
+    # the tradeoff being the delay in when reporting data is available
+    # if running your own redis increasing this number reduces load on the redis CPU
     def background_reporting_sleep_seconds
-      @background_reporting_sleep_seconds ||= if Coverband.coverband_service?
-        Coverband.configuration.coverband_env == "production" ? Coverband.configuration.report_period : 60
+      @background_reporting_sleep_seconds ||= if service?
+        # default to 10m for service
+        Coverband.configuration.coverband_env == "production" ? 600 : 60
+      elsif store.is_a?(Coverband::Adapters::HashRedisStore)
+        # Default to 5 minutes if using the hash redis store
+        300
       else
         60
       end
@@ -139,7 +129,8 @@ module Coverband
     end
 
     def store
-      @store ||= if Coverband.coverband_service?
+      @store ||= if service?
+        raise "invalid configuration: unclear default store coverband expects either api_key or redis_url" if ENV["COVERBAND_REDIS_URL"]
         require "coverband/adapters/web_service_store"
         Coverband::Adapters::WebServiceStore.new(service_url)
       else
@@ -147,21 +138,18 @@ module Coverband
       end
     end
 
-    def track_views
-      @track_views ||= service_disabled_dev_test_env? ? false : true
-    end
-
     def store=(store)
       raise "Pass in an instance of Coverband::Adapters" unless store.is_a?(Coverband::Adapters::Base)
-
-      # Default to 5 minutes if using the hash redis store
-      # This is a safer default for the high server volumes that need the hash store
-      # it should avoid overloading the redis with lots of load
-      @background_reporting_sleep_seconds = 300 if store.is_a?(Coverband::Adapters::HashRedisStore)
+      raise "invalid configuration: only coverband service expects an API Key" if api_key && store.class.to_s != "Coverband::Adapters::WebServiceStore"
+      raise "invalid configuration: coverband service shouldn't have redis url set" if ENV["COVERBAND_REDIS_URL"] && store.class.to_s == "Coverband::Adapters::WebServiceStore"
 
       @store = store
     end
 
+    def track_views
+      @track_views ||= service_disabled_dev_test_env? ? false : true
+    end
+
     ###
     # Search Paths
     ###
@@ -183,10 +171,6 @@ module Coverband
       @ignore = (@ignore + ignored_array).uniq
     end
 
-    def track_gems=(_value)
-      puts "gem tracking is deprecated, setting this will be ignored"
-    end
-
     def current_root
       @current_root ||= File.expand_path(Coverband.configuration.root).freeze
     end
@@ -203,7 +187,7 @@ module Coverband
       @all_root_patterns ||= all_root_paths.map { |path| /^#{path}/ }.freeze
     end
 
-    SKIPPED_SETTINGS = %w[@s3_secret_access_key @store]
+    SKIPPED_SETTINGS = %w[@s3_secret_access_key @store @api_key @password]
     def to_h
       instance_variables
         .each_with_object({}) do |var, hash|
@@ -221,6 +205,10 @@ module Coverband
       Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("2.6.0")
     end
 
+    def redis_url
+      @redis_url ||= ENV["COVERBAND_REDIS_URL"] || ENV["REDIS_URL"]
+    end
+
     def api_key
       @api_key ||= ENV["COVERBAND_API_KEY"]
     end
@@ -245,12 +233,12 @@ module Coverband
       @service_dev_mode ||= ENV["COVERBAND_ENABLE_TEST_MODE"] || false
     end
 
-    def proces_type
+    def process_type
       @process_type ||= ENV["PROCESS_TYPE"] || "unknown"
     end
 
-    def report_period
-      @process_type ||= (ENV["COVERBAND_REPORT_PERIOD"] || 600).to_i
+    def service?
+      Coverband.coverband_service? || !api_key.nil?
     end
 
     def service_disabled_dev_test_env?
@@ -258,12 +246,28 @@ module Coverband
         (coverband_env == "development" && !Coverband.configuration.service_dev_mode)
     end
 
-    private
+    def s3_bucket
+      puts "deprecated, s3 is no longer support"
+    end
 
-    def redis_url
-      ENV["COVERBAND_REDIS_URL"] || ENV["REDIS_URL"]
+    def s3_region
+      puts "deprecated, s3 is no longer support"
+    end
+
+    def s3_access_key_id
+      puts "deprecated, s3 is no longer support"
+    end
+
+    def s3_secret_access_key
+      puts "deprecated, s3 is no longer support"
+    end
+
+    def track_gems=(_value)
+      puts "gem tracking is deprecated, setting this will be ignored"
     end
 
+    private
+
     def redis_store_options
       {ttl: Coverband.configuration.redis_ttl,
        redis_namespace: Coverband.configuration.redis_namespace}

data/lib/coverband/reporters/base.rb

@@ -101,8 +101,7 @@ module Coverband
             end
           end
 
-          scov_style_report = fix_file_names(scov_style_report, roots)
-          scov_style_report
+          fix_file_names(scov_style_report, roots)
         end
       end
     end

data/lib/coverband/reporters/console_report.rb

@@ -8,14 +8,8 @@ module Coverband
     class ConsoleReport < Base
       def self.report(store, options = {})
         scov_style_report = super(store, options)
-
-        scov_style_report.each_pair do |file, usage|
-          # TODO: In Coverband 5 deprecate none hash format
-          if usage.is_a?(Hash)
-            Coverband.configuration.logger.info "#{file}: #{usage["data"]}"
-          else
-            Coverband.configuration.logger.info "#{file}: #{usage}"
-          end
+        scov_style_report[:merged].each_pair do |file, usage|
+          Coverband.configuration.logger.info "#{file}: #{usage["data"]}"
         end
         scov_style_report
       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 = "5.0.0.rc.4"
+  VERSION = "5.0.0"
 end

data/test/coverband/configuration_test.rb

@@ -4,6 +4,7 @@ require File.expand_path("../test_helper", File.dirname(__FILE__))
 
 class BaseTest < Minitest::Test
   def setup
+    Coverband.configuration.reset
     super
     Coverband.configuration.reset
     Coverband.configure do |config|
@@ -52,7 +53,7 @@ class BaseTest < Minitest::Test
     assert_equal current_paths, Coverband.configuration.root_paths
   end
 
-  test "store raises issues" do
+  test "store raises when not set to supported adapter" do
     Coverband::Collectors::Coverage.instance.reset_instance
     assert_raises RuntimeError do
       Coverband.configure do |config|
@@ -61,6 +62,56 @@ class BaseTest < Minitest::Test
     end
   end
 
+  test "store defaults to redis store" do
+    Coverband::Collectors::Coverage.instance.reset_instance
+    assert_equal Coverband.configuration.store.class, Coverband::Adapters::RedisStore
+  end
+
+  test "store is a service store when api_key is set" do
+    Coverband::Collectors::Coverage.instance.reset_instance
+    Coverband.configuration.reset
+    Coverband.configure do |config|
+      config.redis_url = nil
+      config.api_key = "test-key"
+    end
+    assert_equal Coverband.configuration.store.class.to_s, "Coverband::Adapters::WebServiceStore"
+  end
+
+  test "store raises when api key set but not set to service" do
+    Coverband::Collectors::Coverage.instance.reset_instance
+    Coverband.configuration.reset
+    assert_raises RuntimeError do
+      Coverband.configure do |config|
+        config.api_key = "test-key"
+        config.redis_url = "redis://localhost:3333"
+        config.store = Coverband::Adapters::RedisStore.new(Coverband::Test.redis, redis_namespace: "coverband_test")
+      end
+    end
+  end
+
+  test "store raises when api key and coverband redis env" do
+    Coverband::Collectors::Coverage.instance.reset_instance
+    Coverband.configuration.reset
+
+    env = ENV.to_hash.merge("COVERBAND_REDIS_URL" => "redis://localhost:3333")
+    Object.stub_const(:ENV, env) do
+      assert_raises RuntimeError do
+        Coverband.configure do |config|
+          config.api_key = "test-key"
+        end
+      end
+    end
+  end
+
+  test "store doesnt raises when api key and redis_url" do
+    Coverband::Collectors::Coverage.instance.reset_instance
+    Coverband.configuration.reset
+    Coverband.configure do |config|
+      config.api_key = "test-key"
+      config.redis_url = "redis://localhost:3333"
+    end
+  end
+
   test "use_oneshot_lines_coverage" do
     refute Coverband.configuration.use_oneshot_lines_coverage
 

data/test/coverband/integrations/background_test.rb

@@ -12,18 +12,29 @@ class BackgroundTest < Minitest::Test
     end
   end
 
+  def setup
+    Coverband.configuration.reset
+    super
+    Coverband.configure do |config|
+      config.background_reporting_sleep_seconds = 60
+      Coverband.configuration.reporting_wiggle = 0
+    end
+  end
+
   def test_start
+    sleep_seconds = Coverband.configuration.background_reporting_sleep_seconds.to_i
     Thread.expects(:new).yields.returns(ThreadDouble.new(true))
     Coverband::Background.expects(:loop).yields
-    Coverband::Background.expects(:sleep).with(60)
+    Coverband::Background.expects(:sleep).with(sleep_seconds)
     Coverband::Collectors::Coverage.instance.expects(:report_coverage).once
     2.times { Coverband::Background.start }
   end
 
   def test_start_with_wiggle
+    sleep_seconds = Coverband.configuration.background_reporting_sleep_seconds.to_i
     Thread.expects(:new).yields.returns(ThreadDouble.new(true))
     Coverband::Background.expects(:loop).yields
-    Coverband::Background.expects(:sleep).with(65)
+    Coverband::Background.expects(:sleep).with(sleep_seconds + 5)
     Coverband::Background.expects(:rand).with(10).returns(5)
     Coverband.configuration.reporting_wiggle = 10
     Coverband::Collectors::Coverage.instance.expects(:report_coverage).once

data/test/coverband/reporters/web_test.rb

@@ -22,13 +22,13 @@ if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("2.2.0")
       test "renders index content" do
         get "/"
         assert last_response.ok?
-        assert_match "Coverband Admin", last_response.body
+        assert_match "Coverband Home", last_response.body
       end
 
       test "renders index content for empty path" do
         get ""
         assert last_response.ok?
-        assert_match "Coverband Admin", last_response.body
+        assert_match "Coverband Home", last_response.body
       end
 
       test "renders 404" do

data/test/jruby_check.rb

@@ -2,7 +2,7 @@ require "coverage"
 
 Coverage.start
 
-require "./test/dog.rb"
+require "./test/dog"
 
 puts Coverage.peek_result
 

data/test/test_helper.rb

@@ -6,6 +6,7 @@ require "rubygems"
 require "simplecov"
 require "coveralls"
 require "minitest/autorun"
+require "minitest/stub_const"
 require "mocha/minitest"
 require "ostruct"
 require "json"
@@ -45,6 +46,7 @@ module Coverband
     end
 
     def self.reset
+      Coverband.configuration.reset
       Coverband.configuration.redis_namespace = "coverband_test"
       Coverband.configuration.store.instance_variable_set(:@redis_namespace, "coverband_test")
       Coverband.configuration.store.class.class_variable_set(:@@path_cache, {})

data/views/nav.erb

@@ -1,7 +1,7 @@
 
 <div id="header">
-  <a href='<%= base_path %>'>Coverband Admin</a> &nbsp;
-  <a href='<%= base_path %>settings'>Info</a> &nbsp;
+  <a href='<%= base_path %>'>Coverband Home</a> &nbsp;
+  <a href='<%= base_path %>settings'>Configuration</a> &nbsp;
   <% if Coverband.configuration.web_enable_clear %>
     <%= button("#{base_path}clear", 'clear coverage report', delete: true) %>
   <% end %>

data/views/settings.erb

@@ -11,8 +11,8 @@
   <body>
     <div id="wrapper" style="">
       <div id="header">
-        <a href='<%= base_path %>'>Coverband Admin</a> &nbsp;
-        <a href='<%= base_path %>settings'>Info</a> &nbsp;
+        <a href='<%= base_path %>'>Coverband Home</a> &nbsp;
+        <a href='<%= base_path %>settings'>Configuration</a> &nbsp;
       </div>
       <div id="content">
         <dl>

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coverband
 version: !ruby/object:Gem::Version
-  version: 5.0.0.rc.4
+  version: 5.0.0
 platform: ruby
 authors:
 - Dan Mayer
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-08-17 00:00:00.000000000 Z
+date: 2020-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
@@ -95,6 +95,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest-stub-const
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
@@ -165,6 +179,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: standard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.2.5
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.2.5
 - !ruby/object:Gem::Dependency
   name: standardrb
   requirement: !ruby/object:Gem::Requirement
@@ -434,11 +462,11 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Rack middleware to measure production code usage (LOC runtime usage)