coverband 5.0.0.rc.4 → 5.0.0.rc.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c83a78e28c79671ef144ffa38e40ee061a873ef6a207437f01923188c17d3cf0
-  data.tar.gz: 80a3c0a46cfb55a5972139e034733d9fb6541f0d547caff1f004203b9ddb0484
+  metadata.gz: b829c0b4e513f6f04dbc4e4459a2bfbbbfdf5ecb563f7d8e584c6c628082d875
+  data.tar.gz: d3931c93df0c2add11d5cebf98fe26d6df14b4ae987b58edb80d2891f38782f3
 SHA512:
-  metadata.gz: 0b2f9965e62d911acc562c73df79f4ba13ca0df48bb18258a9069b71d3323f9167385b11fe25772d32d23575338e98e04f721dd57f6562ca40171e39f1cae591
-  data.tar.gz: 6ca2447bbddf8ac8bd6a92e4544a2e38b5a7bf84d49e9e02230ef3a2e0044436ce70d9907dad8a7d163cf909369ad4f089af49bb6a50da5b62982e3001aa48bd
+  metadata.gz: bbae35d24375200cc62e3f191ca3dd001f3be9ef24383ec5508c14dec0e5830dd23107031a8fe093f52615c06b292a91a38ef9fb68d5d54a7834d1b9097dd0ae
+  data.tar.gz: 2e085a9d3a5cd3e23c87508d0e36b4119a6027fe10a164714b4de8b934142a981691214f493e169c6167288cd666e7d7c024a605423597fec1fb6d8e72bc1e89

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/changes.md

@@ -61,7 +61,11 @@ 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
 - improved resque patching pattern
 - improved default ignores
 - additional adapters

data/coverband.gemspec

@@ -30,6 +30,7 @@ Gem::Specification.new do |spec|
   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 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.is_a?(Coverband::Adapters::WebServiceStore)
+      raise "invalid configuration: coverband service shouldn't have redis url set" if redis_url && store.is_a?(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/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.rc.5"
 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,44 @@ 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 redis_url" 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"
+      end
+    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/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

@@ -45,6 +45,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, {})

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.rc.5
 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-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
@@ -165,6 +165,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