coverband 4.1.1 → 4.2.0.alpha

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA256:
-  metadata.gz: 0e029eb4a5ce229ca0ce9dd7ac6638bc3b9776e07a724269a495b2832228119d
-  data.tar.gz: 0f37d401fafe727024665b096e337138e53f8be139521dea9e149695b44f39ce
+SHA1:
+  metadata.gz: 40372eb3dc52edf4440af29a183eea4354a1695b
+  data.tar.gz: 47215556426f24df1aa12a26dadfb97418f59ce9
 SHA512:
-  metadata.gz: c0d51d0263c37369abffc5839a367cdebd5fad7d8132362044163614fca33e2bc8617770bb36d3dd268eb34a4f0243e5fe4a1fad516f724a9da122aa5739a8be
-  data.tar.gz: db40dfd936b39d159e7d92b470b186761cfea6ce944e55ff8079d70279af8c84a3602c5c873317c58d76b08aa0a654e653eb10df8a608ba646e4bd37b006105b
+  metadata.gz: da9ddb6c8ad13165d7fca9332e23d10a69fe96d6eb4aa2589f206375f4f87943a878d16858408118a32cb0cb37c33e647b3da581f78792fb0fe84ee3fe36b83b
+  data.tar.gz: 29f91f8e81dd7f00c3b3fa3718b5d8734cad42c7feb4dbd83b930b3cf7e0e649e3ecb34de4efef5c43c15bf37b4d25c6622bc6118755b413ecce1914ac88aed6

data/.gitignore

@@ -16,10 +16,7 @@ rdoc
 spec/reports
 test/tmp
 test/version_tmp
-/tmp/
-temp_results
+tmp
 .byebug_history
 .env
 log/
-test/unique_files
-test/rails5_dummy/tmp

data/.travis.yml

@@ -3,7 +3,7 @@ rvm:
   - "2.3"
   - "2.4"
   - "2.5"
-  - "2.6.1"
+  - "2.6"
 gemfile:
   - Gemfile
   - Gemfile.rails4
@@ -11,9 +11,7 @@ services:
   - redis-server
 before_install:
   - echo 'this is a hack to clear default bundler and force bundler 1.17.3'
-  - ls /home/travis/.rvm/gems/
   - rm /home/travis/.rvm/gems/ruby-2.3.7@global/specifications/bundler-2.0.1.gemspec || true
-  - rm /home/travis/.rvm/gems/ruby-2.5.4@global/specifications/bundler-2.0.1.gemspec || true
   - gem uninstall bundler || true
   - gem install bundler -v '1.17.3'
-  - bundler --version
+  - bundler --version

data/README.md

@@ -38,8 +38,6 @@ Coverband stores coverage data in Redis. The Redis endpoint is looked for in thi
 2. `ENV['REDIS_URL']`
 3. `localhost`
 
-The redis store can also be explicitly defined within the coverband.rb. See [advanced config](#advanced-config).
-
 ## Gem Installation
 
 Add this line to your application's `Gemfile`, remember to `bundle install` after updating:
@@ -48,11 +46,9 @@ Add this line to your application's `Gemfile`, remember to `bundle install` afte
 gem 'coverband'
 ```
 
-If [tracking gem usage](#collecting-gem--library-usage), be sure to include coverband before other gems you would like to track.
-
 ## Rails
 
-The Railtie integration means you shouldn't need to do anything anything else other than ensure coverband is required after rails within your Gemfile.
+The Railtie integration means you shouldn't need to do anything anything else. If you have an issue with that, please [file an issue](https://github.com/danmayer/coverband/issues).
 
 
 ## Sinatra
@@ -202,15 +198,6 @@ ENV['AWS_ACCESS_KEY_ID']
 ENV['AWS_SECRET_ACCESS_KEY']
 ```
 
-### 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.
-
-`rake coverband:migrate`
-
-* We will be working to support migrations going forward, when possible
-
-
 ### Clear Coverage
 
 Now that Coverband uses MD5 hashes there should be no reason to manually clear coverage unless one is testing, changing versions, possibly debugging Coverband itself.
@@ -255,28 +242,16 @@ By adding any files above you will get reporting on those files as part of your
 
 ### Collecting Gem / Library Usage
 
-Gem usage can be tracked by enabling the `track_gems` config.
-
-```
-Coverband.configure do |config|
-  config.track_gems = true
-end
-```
-
-The `track_gems` feature exposes a Gems tab in the report which prints out the percentage usage of each Gem. See demo [here](https://coverband-demo.herokuapp.com/coverage?#_Gems).
-
-When tracking gems, it is important that `Coverband#start` is called before the gems to be tracked are required. Since `Coverband#start` is automatically called by default when coverband is required, list coverband before the other gems to be tracked within your Gemfile. The exception to this are gems like rails and resque. Since coverband has some specific intergrations for these frameworks, these two gems should be required first.
+By default Coverband has assumed you are trying to track your application code usage and not all the supporting framework and library (Gems) code usage. There are reasons to track library usage though such as finding out which Gems aren't actually being used within production. See some of the discussion on [issue 21](https://github.com/danmayer/coverband/issues/21).
 
-The track_gems config only exposes the overall usage of a gem. In order to see the detail of each file, enable the `gem_details` flag.
+How to collect gem usage with Coverband:
 
-```
-Coverband.configure do |config|
-  config.track_gems = true
-  config.gem_details = true
-end
-```
+* use the `safe_reload_files` feature to add the path of all gem files you wish to track
+* --- or ---
+* ensure you call `require 'coverband'` which triggers `Coverband.start` before loading all your gems
+   * while possible this is currently hard as Rails and most environments load your whole Gemfile
+   * we are looking for an improve and easier way to support this.
 
-This flag exposes line by line usage of gem files. Unfortunately due to the way the coverband report is currently rendered, enabling `gem_details` slows down viewing of the coverage report in the browser and is not yet recommended.
 
 ### Verbose Debug / Development Mode
 

data/Rakefile

@@ -13,7 +13,7 @@ Rake::TestTask.new(:test) do |test|
   # exclude benchmark from the tests as the way it functions resets code coverage during executions
   # test.pattern = 'test/unit/*_test.rb'
   # using test files opposed to pattern as it outputs which files are run
-  test.test_files = FileList['test/integration/**/*_test.rb', 'test/coverband/**/*_test.rb']
+  test.test_files = FileList['test/unit/**/*_test.rb']
   test.verbose = true
 end
 

data/changes.md

@@ -36,7 +36,7 @@ Will be the fully modern release that drops maintenance legacy support in favor
 - Make good video on setup, install, usage
 - See if we can add support for views / templates
   - using this technique https://github.com/ioquatix/covered
-- Better default grouping (could use groups features for gems for rails controllers, models, lib, etc)
+- Better default grouping (could use groups features for gems for rails controllers, models, lib, etc)  	
 
 ### Coverband_jam_session
 
@@ -63,36 +63,20 @@ Feature Ideas:
 
 # Alpha
 
-### Coverband 4.2.0.rc
+### Coverband 4.2.0.alpha
 
-- loadtime vs runtime coverage
-- view coverage
-- perf speedup on gem details views
+- pilot release of Gems tracking
+	- todos
+	  - support multiple gem paths (various version managers setup multiple gem paths)
+	  - speed up page load by allowing multiple pages
 
-# Released
-
-### Coverband 4.1.1
-
-- fix bug on info page when using namespaces
-- fix bug related to forking Ruby processes (Resque for example)
-  - bug caused negative line hits
-  - bug seemed to cause memory leaks (not entirely sure how)
-- various test improvements
-- fix bad requests on coverband web admin to `/coverage/favicon.png`
-- improved documentation
-
-### Coverband 4.1.0
+### Coverband 4.1.0.alpha
 
 - default disabled web clear, add config option to allow it
 - out of the box support for resque
 - readme improvements
-- fix on regression of merging directory changing deployments
-  - fixes for duplicate root paths
-- pilot release of Gems tracking (disabled by default) - todos - support multiple gem paths (various version managers setup multiple gem paths) - speed up page load by allowing multiple pages
-- added web settings and debug views
-  - added support for seeing coverage data size consumed in redis
-- support coverage data migrations from 4.0.x to 4.1.0
-- fixes for heroku /tmp asset building
+
+# Released
 
 ### Coverband 4.0.1
 
@@ -102,7 +86,7 @@ Feature Ideas:
 - reduced the S3 dependencies to minimal set (not loading all of aws-sdk, only aws-sdk-s3), ensured they are optional
 - Improved Coverband web admin
 - Coverage reports include timestamps of Coverage collection
-- Added Coveralls to the dev process thanks @dondonz
+- Added Coveralls to the dev process thanks @dondonz 
 - now tested under Ruby 2.6.0 thanks @Kbaum
 - improved full stack testing for Rails 5 & 4
 - warning before clear coverage on coverband web

data/coverband.gemspec

@@ -33,8 +33,6 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rack-test'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'resque'
-  # temporarily needed to fun tests for classifier-reborn as part of benchmarks
-  spec.add_development_dependency 'minitest-reporters'
 
   # TODO: used for benchmarking and tests I think we have other better benchmarking
   # perhaps time to drop this and refactor.
@@ -44,8 +42,6 @@ Gem::Specification.new do |spec|
   # require 'byebug'; byebug
   spec.add_development_dependency 'pry-byebug'
 
-  spec.add_development_dependency 'minitest-profile'
-
   # TODO: Remove when other production adapters exist
   # because the default configuration of redis store, we really do require
   # redis now. I was reluctant to add this, but until we offer another production

data/lib/coverband.rb

@@ -7,7 +7,6 @@ require 'redis'
 require 'coverband/version'
 require 'coverband/at_exit'
 require 'coverband/configuration'
-require 'coverband/utils/file_path_helper'
 require 'coverband/adapters/base'
 require 'coverband/adapters/redis_store'
 require 'coverband/adapters/file_store'
@@ -50,10 +49,6 @@ module Coverband
     end
   end
 
-  def self.report_coverage(force_report = false)
-    Coverband::Collectors::Coverage.instance.report_coverage(force_report)
-  end
-
   def self.configuration
     self.configuration_data ||= Configuration.new
   end

data/lib/coverband/adapters/base.rb

@@ -3,8 +3,6 @@
 module Coverband
   module Adapters
     class Base
-      include Coverband::Utils::FilePathHelper
-
       def initialize
         @file_hash_cache = {}
       end
@@ -13,25 +11,13 @@ module Coverband
         raise 'abstract'
       end
 
-      def migrate!
-        raise 'abstract'
-      end
-
-      def size
-        raise 'abstract'
-      end
-
-      def size_in_mib
-        format('%.2f', (size.to_f / 2**20))
-      end
-
       # Note: This could lead to slight race on redis
       # where multiple processes pull the old coverage and add to it then push
       # the Coverband 2 had the same issue,
       # and the tradeoff has always been acceptable
       def save_report(report)
         data = report.dup
-        data = merge_reports(data, get_report)
+        merge_reports(data, get_report)
         save_coverage(data)
       end
 
@@ -63,7 +49,6 @@ module Coverband
       end
 
       def expand_report(report)
-        expanded = {}
         report_time = Time.now.to_i
         report.each_pair do |key, line_data|
           extended_data = {
@@ -72,13 +57,12 @@ module Coverband
             'file_hash' => file_hash(key),
             'data' => line_data
           }
-          expanded[full_path_to_relative(key)] = extended_data
+          report[key] = extended_data
         end
-        expanded
       end
 
-      def merge_reports(new_report, old_report, options = {})
-        new_report = expand_report(new_report) unless options[:skip_expansion]
+      def merge_reports(new_report, old_report)
+        new_report = expand_report(new_report)
         keys = (new_report.keys + old_report.keys).uniq
         keys.each do |file|
           new_report[file] = if new_report[file] &&

data/lib/coverband/adapters/file_store.rb

@@ -20,14 +20,6 @@ module Coverband
         File.delete(path) if File.exist?(path)
       end
 
-      def size
-        File.size?(path).to_i
-      end
-
-      def migrate!
-        raise NotImplementedError, "FileStore doesn't support migrations"
-      end
-
       private
 
       attr_accessor :path

data/lib/coverband/adapters/redis_store.rb

@@ -11,56 +11,25 @@ module Coverband
       # used to store data to redis. It is changed only when breaking changes to our
       # redis format are required.
       ###
-      REDIS_STORAGE_FORMAT_VERSION = 'coverband_3_2'
+      REDIS_STORAGE_FORMAT_VERSION = 'coverband3_1'
 
       def initialize(redis, opts = {})
         super()
         @redis           = redis
         @ttl             = opts[:ttl]
         @redis_namespace = opts[:redis_namespace]
-        @format_version  = REDIS_STORAGE_FORMAT_VERSION
       end
 
       def clear!
         @redis.del(base_key)
       end
 
-      def size
-        @redis.get(base_key).bytesize
-      end
-
-      ###
-      # Current implementation moves from coverband3_1 to coverband_3_2
-      # In the future this can be made more general and support a more specific
-      # version format.
-      ###
-      def migrate!
-        reset_base_key
-        @format_version = 'coverband3_1'
-        previous_data = get_report
-        if previous_data.empty?
-          puts 'no previous data to migrate found'
-          exit 0
-        end
-        relative_path_report = previous_data.each_with_object({}) do |(key, vals), fixed_report|
-          fixed_report[full_path_to_relative(key)] = vals
-        end
-        clear!
-        reset_base_key
-        @format_version = REDIS_STORAGE_FORMAT_VERSION
-        save_coverage(merge_reports(get_report, relative_path_report, skip_expansion: true))
-      end
-
       private
 
       attr_reader :redis
 
-      def reset_base_key
-        @base_key = nil
-      end
-
       def base_key
-        @base_key ||= [@format_version, @redis_namespace].compact.join('.')
+        @base_key ||= [REDIS_STORAGE_FORMAT_VERSION, @redis_namespace].compact.join('.')
       end
 
       def save_coverage(data)

data/lib/coverband/at_exit.rb

@@ -11,7 +11,7 @@ module Coverband
         @at_exit_registered = true
         at_exit do
           ::Coverband::Background.stop
-          Coverband.report_coverage(true)
+          Coverband::Collectors::Coverage.instance.report_coverage(true)
           Coverband.configuration.logger&.debug('Coverband: Reported coverage before exit')
         end
       end

data/lib/coverband/collectors/coverage.rb

@@ -87,13 +87,7 @@ module Coverband
       end
 
       def array_diff(latest, original)
-        latest.map.with_index do |v, i|
-          if (v && original[i])
-            [0, v - original[i]].max
-          else
-            nil
-          end
-        end
+        latest.map.with_index { |v, i| (v && original[i]) ? v - original[i] : nil }
       end
 
       def previous_results

data/lib/coverband/configuration.rb

@@ -8,7 +8,7 @@ module Coverband
                   :redis_namespace, :redis_ttl,
                   :safe_reload_files, :background_reporting_enabled,
                   :background_reporting_sleep_seconds, :test_env,
-                  :web_enable_clear, :gem_details, :web_debug
+                  :web_enable_clear, :gem_details
 
     attr_writer :logger, :s3_region, :s3_bucket, :s3_access_key_id, :s3_secret_access_key
     attr_reader :track_gems
@@ -20,10 +20,7 @@ module Coverband
     def reset
       @root = Dir.pwd
       @root_paths = []
-      # Heroku when building assets runs code from a dynamic directory
-      # /tmp was added to avoid coverage from /tmp/build directories during
-      # heroku asset compilation
-      @ignore = %w[vendor .erb$ .slim$ /tmp]
+      @ignore = %w(vendor .erb$ .slim$)
       @additional_files = []
       @reporting_frequency = 0.0
       @verbose = false
@@ -37,7 +34,6 @@ module Coverband
       @track_gems = false
       @gem_details = false
       @groups = {}
-      @web_debug = false
 
       # TODO: should we push these to adapter configs
       @s3_region = nil
@@ -87,11 +83,6 @@ module Coverband
     def track_gems=(value)
       @track_gems = value
       return unless @track_gems
-      # by default we ignore vendor where many deployments put gems
-      # we will remove this default if track_gems is set
-      @ignore.delete('vendor')
-      # while we want to allow vendored gems we don't want to track vendored ruby STDLIB
-      @ignore << 'vendor/ruby-*'
       add_group('App', root)
       # TODO: rework support for multiple gem paths
       # currently this supports GEM_HOME (which should be first path)
@@ -124,25 +115,6 @@ module Coverband
       Gem::PathSupport.new(ENV).path.select { |path| File.exist?(path) }
     end
 
-    def current_root
-      File.expand_path(Coverband.configuration.root)
-    end
-
-    def all_root_paths
-      roots = Coverband.configuration.root_paths.dup
-      roots += Coverband.configuration.gem_paths.dup if Coverband.configuration.track_gems
-      roots << "#{Coverband.configuration.current_root}/"
-      roots
-    end
-
-    SKIPPED_SETTINGS = %w(@s3_secret_access_key @store)
-    def to_h
-      instance_variables
-        .each_with_object('gem_paths': gem_paths) do |var, hash|
-          hash[var.to_s.delete('@')] = instance_variable_get(var) unless SKIPPED_SETTINGS.include?(var.to_s)
-        end
-    end
-
     private
 
     def redis_url