rspec-cover_it 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7646cf84b3b24df43ca171de7628e65347bbdc8270dd4ff1de2c3bdc6d44596f
-  data.tar.gz: 47e86f853552fadfa9f64a478871fad62db530e4840d8887a6513593a6505032
+  metadata.gz: 66c4b3e0c88dffe53a1d435d40d4e1bdb52038661d8abb89ccc29ef6974398e2
+  data.tar.gz: 563d49e4e8656ea8da53defeb9e2d5f7ed9a91d36e5de0df126213f35dee6813
 SHA512:
-  metadata.gz: 114cb6e33e6867b1c4e4dadc43690e67feeb01cbcdab39bcd24d3d563e65323e43eb09dd2d78f8c5056f1b27faad3ba0f9383a31b5fb216674c92998dc718258
-  data.tar.gz: d89f76882f68ffbf6978407fa6f588e29fa87cf66f6cd64bc985af9510232168204d81925975f739e9cf2961e345b871a45c7ac8894e75f0ea0db6855a2d60d1
+  metadata.gz: e17ac79db233e4a1f9571f98d025c37cf2e3c304155838fc221507ef5ac6a24aecb47be2f0a6678423009254a0bb5eac442ce775f4ad8bb3f9a66554e9f09fd6
+  data.tar.gz: 0c9afcc6b5f019c185365e5c40925a3b96b4108133d1dcc6150edb182780c35e2a946693106fdf9299e507007a1adfb1dc33cbc341b95f729cea1faf202bbab2

data/README.md

@@ -126,3 +126,13 @@ aren't a lot of controls - it works, but it's a bit ugly and doesn't really
 give the right immediate impression. I'm contemplating using an `after(:suite)`
 hook and aggregating them myself, but at the end of the day RSpec is in control
 of the output stream, and we don't entirely fit its metaphor.
+
+We're using `Object.const_source_location` to find the path of the source file
+defining a given constant. That _mostly_ works, but it actually gives the path
+of the _first_ source file that defined that constant. So if your gem defines
+its version in `lib/foo/version.rb` (as an example), in a separate file from
+lib/foo.rb, the _path_ for `Foo` may end up being the former. Which is.. not
+going to have much coverable code, of course. This is an edge case, but one
+that is likely to occur fairly regularly. I haven't thought of a _good_ solution
+yet. Perhaps if the `covers` array includes a string, we should treat it as a
+relative path?

data/lib/rspec/cover_it/context.rb

@@ -1,12 +1,12 @@
 module RSpec
   module CoverIt
     class Context
-      def initialize(scope:, rspec_context:)
-        @scope, @rspec_context = scope, rspec_context
+      def initialize(scope:, rspec_context:, autoenforce:)
+        @scope, @rspec_context, @autoenforce = scope, rspec_context, autoenforce
       end
 
       def cover_it?
-        target_class && metadata.fetch(:cover_it, nil)
+        target_class && metadata.fetch(:cover_it, autoenforce?)
       end
 
       def target_path
@@ -25,6 +25,10 @@ module RSpec
 
       attr_reader :scope, :rspec_context
 
+      def autoenforce?
+        @autoenforce
+      end
+
       def metadata
         scope.metadata
       end

data/lib/rspec/cover_it/context_coverage.rb

@@ -25,7 +25,7 @@ module RSpec
       end
 
       def enforce!
-        return if local_coverage_rate >= 1.0
+        return if local_coverage.nil? || local_coverage_rate >= 1.0
         lines = local_coverage.each_with_index.select { |v, _i| v&.zero? }.map(&:last)
 
         summary =

data/lib/rspec/cover_it/coverage_state.rb

@@ -3,8 +3,8 @@ module RSpec
     class CoverageState
       attr_reader :filter
 
-      def initialize(filter:)
-        @filter = filter
+      def initialize(filter: nil, autoenforce: false)
+        @filter, @autoenforce = filter, autoenforce
         @pretest_results = nil
         @context_coverages = {}
       end
@@ -20,7 +20,7 @@ module RSpec
       end
 
       def start_tracking_for(scope, rspec_context)
-        context = Context.new(scope: scope, rspec_context: rspec_context)
+        context = context_for(scope, rspec_context)
         return unless context.cover_it?
 
         context_coverage_for(context).tap do |context_coverage|
@@ -29,7 +29,7 @@ module RSpec
       end
 
       def finish_tracking_for(scope, rspec_context)
-        context = Context.new(scope: scope, rspec_context: rspec_context)
+        context = context_for(scope, rspec_context)
         return unless context.cover_it?
 
         context_coverage_for(context).tap do |context_coverage|
@@ -42,6 +42,14 @@ module RSpec
 
       attr_reader :pretest_results
 
+      def autoenforce?
+        @autoenforce
+      end
+
+      def context_for(scope, rspec_context)
+        Context.new(scope: scope, rspec_context: rspec_context, autoenforce: autoenforce?)
+      end
+
       def context_coverage_for(context)
         @context_coverages[context.target_class] ||= ContextCoverage.new(
           context: context,

data/lib/rspec/cover_it/pretest_coverage.rb

@@ -4,7 +4,7 @@ module RSpec
       def initialize(filter:, results:)
         @filter = filter
         @results = results
-          .select { |k, _v| k.start_with?(filter) }
+          .select { |k, _v| filter.nil? || k.start_with?(filter) }
           .map { |k, v| [k, v.dup] }
           .to_h
       end

data/lib/rspec/cover_it/version.rb

@@ -1,5 +1,5 @@
 module RSpec
   module CoverIt
-    VERSION = "0.0.3"
+    VERSION = "0.0.4"
   end
 end

data/lib/rspec/cover_it.rb

@@ -16,8 +16,8 @@ module RSpec
       attr_accessor :state
     end
 
-    def self.setup(filter:)
-      RSpec::CoverIt.state = CoverageState.new(filter: filter)
+    def self.setup(filter: nil, autoenforce: false)
+      RSpec::CoverIt.state = CoverageState.new(filter: filter, autoenforce: autoenforce)
       RSpec::CoverIt.state.start_tracking
 
       RSpec.configure do |config|

metadata

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