single_cov 1.10.0 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8d35277b97cbcabd001c11de169153d1eda320dfcfbd7ad516406ee811e623eb
-  data.tar.gz: c5d9fb7c34b7cb1a67a772a8ce870dbfc40cc6b1ae9ee859a951bbc3cbf93120
+  metadata.gz: e47ea219fd0ed2b3d8911ca9679c4de43988ba900072e7a3efe349275d230f8b
+  data.tar.gz: 24410aaf39a37f4301723bc53b76cbccfeb596ceed364a4ebe4dfbdde54a3556
 SHA512:
-  metadata.gz: 64b8de808afd738fae4817c0701c12905882a4122ac50b53cc3e13694f78a831bcbaa433a7e45115682ef3ea2beefd7b59d2442872ad2ce11a82999007e3c6c7
-  data.tar.gz: 7637465f16cadc6a8beec088abd9fff899a3d9052617dd9c89d786ee24e565cb6fa8a8508a69407d63c013088aa11a3b765ac152198fc395460a490536525295
+  metadata.gz: 0dd92b86bfa607cfb908cfaa7fc2a5b8e74da99a92d6a116cc88dbf2b241cc27664c027814989cb6e7d180bed5d7dff65e181c76cb71181340e0d947d7a0d183
+  data.tar.gz: c42f4b53dfc9e16cb72b57e8180cf81c9cad57dbf1be0f549ceb480e16dd31d7030130dfca58e731dc3398ff61a82d05eeafb9b5fa2a7b905dd43ca6ab60bfa4

data/lib/single_cov/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module SingleCov
-  VERSION = "1.10.0"
+  VERSION = "2.0.0"
 end

data/lib/single_cov.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 module SingleCov
   COVERAGES = []
-  MAX_OUTPUT = 40
+  MAX_OUTPUT = Integer(ENV["SINGLE_COV_MAX_OUTPUT"] || "40")
   RAILS_APP_FOLDERS = ["models", "serializers", "helpers", "controllers", "mailers", "views", "jobs", "channels"]
   UNCOVERED_COMMENT_MARKER = /#.*uncovered/
   PREFIXES_TO_IGNORE = [] # things to not prefix with lib/ etc
@@ -32,7 +32,7 @@ module SingleCov
 
     def all_covered?(result)
       errors = COVERAGES.flat_map do |file, expected_uncovered|
-        next no_coverage_error(file) unless coverage = result["#{root}/#{file}"]
+        next no_coverage_error(file) unless (coverage = result["#{root}/#{file}"])
 
         uncovered = uncovered(coverage)
         next if uncovered.size == expected_uncovered
@@ -50,7 +50,9 @@ module SingleCov
 
       return true if errors.empty?
 
-      errors[MAX_OUTPUT..-1] = "... coverage output truncated" if errors.size >= MAX_OUTPUT
+      if errors.size >= MAX_OUTPUT
+        errors[MAX_OUTPUT..-1] = "... coverage output truncated (use SINGLE_COV_MAX_OUTPUT=999 to expand)"
+      end
       @error_logger.puts errors
 
       errors.all? { |l| warning?(l) }
@@ -117,8 +119,7 @@ module SingleCov
 
       case framework
       when :minitest
-        minitest_should_not_be_running!
-        return if minitest_running_subset_of_tests?
+        return if minitest_running_subset_of_tests?(ARGV)
       when :rspec
         return if rspec_running_subset_of_tests?
       else
@@ -127,15 +128,35 @@ module SingleCov
 
       start_coverage_recording
 
-      override_at_exit do |status, _exception|
-        if enabled? && main_process? && status == 0
-          results = coverage_results
-          generate_report results
-          exit 1 unless SingleCov.all_covered?(results)
+      # minitest overrides at_exit, so we need to get into it to execute after it finishes
+      # so when using the `minitest` executable or loading minitest before SingleCov the first branch is used
+      if defined?(Minitest)
+        (class << Minitest; self; end).prepend(Module.new do
+          def run(args = [])
+            original_args = args.dup # minitest modified them
+            result = super
+            if result && !SingleCov.send(:minitest_running_subset_of_tests?, original_args)
+              return SingleCov.report_at_exit
+            end
+            result
+          end
+        end)
+      else
+        override_at_exit do |status, _exception|
+          if main_process? && status == 0 && !report_at_exit
+            exit 1
+          end
         end
       end
     end
 
+    def report_at_exit
+      return unless enabled?
+      results = coverage_results
+      generate_report results
+      SingleCov.all_covered?(results)
+    end
+
     # use this in forks when using rspec to silence duplicated output
     def disable
       @disabled = true
@@ -157,7 +178,7 @@ module SingleCov
     end
 
     def enabled?
-      (!defined?(@disabled) || !@disabled)
+      !defined?(@disabled) || !@disabled
     end
 
     # assuming that the main process will load all the files, we store it's pid
@@ -166,7 +187,7 @@ module SingleCov
     end
 
     def main_process?
-      (!defined?(@main_process_pid) || @main_process_pid == Process.pid)
+      !defined?(@main_process_pid) || @main_process_pid == Process.pid
     end
 
     # {[branch_id] => {[branch_part] => coverage}} --> uncovered location
@@ -228,56 +249,26 @@ module SingleCov
       COVERAGES.size == 1
     end
 
-    # we cannot insert our hooks when minitest is already running
-    def minitest_should_not_be_running!
-      return unless defined?(Minitest)
-      return unless Minitest.class_variable_defined?(:@@installed_at_exit)
-      return unless Minitest.class_variable_get(:@@installed_at_exit)
-
-      # untested
-      # https://github.com/rails/rails/pull/26515 rails loads autorun before test
-      # but it works out for some reason
-      return if Minitest.extensions.include?('rails')
-
-      # untested
-      # forking test runner does some hacky acrobatics to fake minitest status
-      # and the resets it ... works out ok in the end ...
-      return if faked_by_forking_test_runner?
-
-      # ... but only if it's used with `--merge-coverage` otherwise the coverage reporting is useless
-      if $0.end_with?("/forking-test-runner")
-        raise "forking-test-runner only work with single_cov when using --merge-coverage"
-      end
-
-      raise "Load minitest after setting up SingleCov"
-    end
-
-    # ForkingTestRunner fakes an initialized minitest to avoid multiple hooks being installed
-    # so hooks still get added in order https://github.com/grosser/forking_test_runner/pull/4
-    def faked_by_forking_test_runner?
-      defined?(Coverage) && Coverage.respond_to?(:capture_coverage!)
-    end
-
     # do not record or verify when only running selected tests since it would be missing data
-    def minitest_running_subset_of_tests?
+    def minitest_running_subset_of_tests?(argv)
       # via direct option (ruby test.rb -n /foo/)
-      (ARGV.map { |a| a.split('=', 2).first } & ['-n', '--name', '-l', '--line']).any? ||
+      # or `minitest foo.rb:123` which is translated to -n
+      argv.map { |a| a.split('=', 2).first }.intersect?(['-n', '--name', '-l', '--line']) ||
 
-      # via testrbl or mtest or rails with direct line number (mtest test.rb:123)
-      (ARGV.first =~ /:\d+\Z/) ||
+      # via testrbl, mtest, or rails with direct line number (mtest test.rb:123)
+      (argv.first =~ /:\d+\Z/) ||
 
       # via rails test which preloads mintest, removes ARGV and fills options
       (
-        defined?(Minitest) &&
         defined?(Minitest.reporter) &&
         Minitest.reporter &&
         (reporter = Minitest.reporter.reporters.first) &&
-        reporter.options[:filter]
+        (reporter.options[:filter] || reporter.options[:include]) # MT 5 vs MT 6
       )
     end
 
     def rspec_running_subset_of_tests?
-      (ARGV & ['-t', '--tag', '-e', '--example']).any? || ARGV.any? { |a| a =~ /:\d+$|\[[\d:]+\]$/ }
+      ARGV.intersect?(['-t', '--tag', '-e', '--example']) || ARGV.any? { |a| a =~ /:\d+$|\[[\d:]+\]$/ }
     end
 
     # code stolen from SimpleCov
@@ -398,7 +389,7 @@ module SingleCov
           end
 
         # remove test extension
-        if !file_part.sub!(/_(?:test|spec)\.rb\b.*/, '.rb') && !file_part.sub!(/\/test_/, "/")
+        if !file_part.sub!(/_(?:test|spec)\.rb\b.*/, '.rb') && !file_part.sub!('/test_', "/")
           raise "Unable to remove test extension from #{file} ... /test_, _test.rb and _spec.rb are supported"
         end
       end
@@ -416,14 +407,14 @@ module SingleCov
     end
 
     def generate_report(results)
-      return unless report = coverage_report
+      return unless (report = coverage_report)
 
       # not a hard dependency for the whole library
       require "json"
       require "fileutils"
 
       used = COVERAGES.map { |f, _| "#{root}/#{f}" }
-      covered = results.select { |k, _| used.include?(k) }
+      covered = results.slice(*used)
 
       if coverage_report_lines
         covered = covered.transform_values { |v| v.is_a?(Hash) ? v.fetch(:lines) : v }

metadata

@@ -1,16 +1,98 @@
 --- !ruby/object:Gem::Specification
 name: single_cov
 version: !ruby/object:Gem::Version
-  version: 1.10.0
+  version: 2.0.0
 platform: ruby
 authors:
 - Michael Grosser
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-05-20 00:00:00.000000000 Z
-dependencies: []
-description:
+date: 1980-01-02 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: bump
+  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: minitest
+  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: rake
+  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: rspec
+  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: rubocop
+  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: simplecov
+  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'
 email: michael@grosser.it
 executables: []
 extensions: []
@@ -23,7 +105,6 @@ homepage: https://github.com/grosser/single_cov
 licenses:
 - MIT
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -31,15 +112,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.7.0
+      version: 3.2.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.3
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Actionable code coverage.
 test_files: []