rspec-cover_it 0.0.6 → 0.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5989ad87b5dccfd49f0e4074fde2aa5ee796af31c93d7a8c3b081fd5283db3aa
-  data.tar.gz: ef49e7f6d941accf93973aaf56da57a6ae60db7232042b9d06edad5d1845dc5e
+  metadata.gz: 63e1b79c2f0b0d52869caa48e5d7dcfee100a726f1fb4698bcbef5d228179ec4
+  data.tar.gz: 24d36270a6320b920f30134c8bb354b77325b16e4a311b1fb018470c72f0c5a7
 SHA512:
-  metadata.gz: 0d64ad3f4a25abd3a5af67ab7e6ff3f548e7c43004c25c6c7dba5da4de81d5217a2160ae7c725c50af9fb9caecad57105c4d0f18cafb5b12a4968c63de849993
-  data.tar.gz: e0dee5c4480e921645112dcd5bfcbe303f144a90564d03542fe6b84b11159566807555b06c45382eb5a0302faba6470b786bcd38c4e5ba16563184a0bad52259
+  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/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.6"
+    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.6
+  version: 0.0.7
 platform: ruby
 authors:
 - Eric Mueller