rspec-cover_it 0.0.5 → 0.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b3efab53f0b9f05be2f5a8ae7f170e4a3a8e886439a751e025926c61bf9ac8f1
-  data.tar.gz: 569fbdcc5f328d5d0656eb3fe20265c9cd164455cfbf290c7764b92d15454fbf
+  metadata.gz: 63e1b79c2f0b0d52869caa48e5d7dcfee100a726f1fb4698bcbef5d228179ec4
+  data.tar.gz: 24d36270a6320b920f30134c8bb354b77325b16e4a311b1fb018470c72f0c5a7
 SHA512:
-  metadata.gz: e9d198749c87a093cf8cbf5d2e5a4f20bfdb1a4c312579a541d8335fa4ccab88781fdbee073195cf8d5c682d921019c9fb86ede9fe951e57f83df24722934c59
-  data.tar.gz: 604d4808f4a3d8afacf52cfe0651d4bbae46ee510ca95e123cd14304b071cbf1c726bf37741002efdd0d9dc636625b47d149f0a17226c26f42cdad0ba531be7f
+  metadata.gz: 3f8774a17ea6e1170471182cf768cbd7fc0e3f69a8cf2d04445d7a7ae0e7e04947b6a75205aa2d53eaaeef104597447608b3873413b73bfda2d9b83ce015bec6
+  data.tar.gz: 3cf13d18a7658e6769d672e1d130ace9e3842ce1e3e7edfdf21e3c64753fb47668da6c7fa16690bc78e6741aa58cd03bdf1344d7c98b50a6fdbf51de03ba5ec1

data/README.md

@@ -33,10 +33,7 @@ You set up `RSpec::CoverIt` in your `spec_helper.rb` - require `rspec/cover_it`,
 and then `RSpec::CoverIt.setup(**config_options)` (before loading your code, as
 you would with any other coverage library). It's.. _semi_ compatible with other
 coverage systems - it only starts Coverage if it's not already running, and it
-only uses `peek_result`, so it doesn't affect other systems outcomes. But if you
-setup SimpleCov to do branch-coverage or something, that will break this gem,
-for now (we assume basic line-coverage style results).
-
+only uses `peek_result`, so it doesn't affect other systems outcomes.
 Rough configuration options:
 
 * `filter`: Only paths starting with this (matching this regex?) can matter
@@ -64,6 +61,10 @@ In example groups, you can use metadata to control the behavior of
   the fact that we can't perfectly infer the location of the source code in
   some cases - in particular, `lib/foo/version.rb` tends to cause a problem
   for specs on `foo.rb`, since the version file is invariably loaded first.
+  Note - in gems, this _frequently_ also happens when you glob-load a directory
+  _before_ defining the namespace they are all loading objects into. Then the
+  first file in that directory that loads ends up being the one that actually
+  creates the namespace.
 * `covers`: An array of classes and modules that this example groups _thinks
   it is completely testing_. Ideally, you'd have a separate test file for each,
   but sometimes that's hard to do - you can let one spec claim responsibility

data/lib/rspec/cover_it/context_coverage.rb

@@ -57,8 +57,11 @@ module RSpec
           else
             "on #{lines.length} lines, including #{lines.first(10).map(&:to_s).join(", ")}"
           end
-        message = "Missing coverage in #{context.target_path} #{summary}"
-        fail(MissingCoverage, message)
+
+        fail(MissingCoverage, <<~MESSAGE.tr("\n", " "))
+          Example group `#{context.scope_name}` is missing coverage on
+          `#{context.target_class}` in `#{context.target_path}` #{summary}
+        MESSAGE
       end
 
       attr_reader :context, :pretest_results

data/lib/rspec/cover_it/coverage_state.rb

@@ -16,7 +16,7 @@ module RSpec
       end
 
       def finish_load_tracking
-        @pretest_results = PretestCoverage.new(filter: filter, results: Coverage.peek_result)
+        @pretest_results = PretestCoverage.new(filter: filter, results: get_current_coverage)
       end
 
       def start_tracking_for(scope, rspec_context)
@@ -24,7 +24,7 @@ module RSpec
         return unless context.cover_it?
 
         context_coverage_for(context).tap do |context_coverage|
-          context_coverage.precontext_coverage = Coverage.peek_result[context.target_path]
+          context_coverage.precontext_coverage = get_current_coverage(context.target_path)
         end
       end
 
@@ -33,7 +33,7 @@ module RSpec
         return unless context.cover_it?
 
         context_coverage_for(context).tap do |context_coverage|
-          context_coverage.postcontext_coverage = Coverage.peek_result[context.target_path]
+          context_coverage.postcontext_coverage = get_current_coverage(context.target_path)
           context_coverage.enforce!(default_threshold: default_threshold_rate)
         end
       end
@@ -60,6 +60,21 @@ module RSpec
           pretest_results: pretest_results
         )
       end
+
+      def get_current_coverage(path = nil)
+        result = Coverage.peek_result
+
+        if path
+          value = result[path]
+          value.is_a?(Hash) ? value.fetch(:lines) : value
+        else
+          if result.any? { |_k, v| v.is_a?(Hash) }
+            result.transform_values { |v| v.fetch(:lines) }
+          else
+            result
+          end
+        end
+      end
     end
   end
 end

data/lib/rspec/cover_it/version.rb

@@ -1,5 +1,5 @@
 module RSpec
   module CoverIt
-    VERSION = "0.0.5"
+    VERSION = "0.0.7"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rspec-cover_it
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.7
 platform: ruby
 authors:
 - Eric Mueller