rspec-cover_it 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 528de4fe406f539d93d8632624714721d8cded1463a35776c16423d387c13c30
-  data.tar.gz: 57fc958451fb2dfeafb94aaaed56aba38befd938a4ea24d2f82d0ac09a46ee4e
+  metadata.gz: 7646cf84b3b24df43ca171de7628e65347bbdc8270dd4ff1de2c3bdc6d44596f
+  data.tar.gz: 47e86f853552fadfa9f64a478871fad62db530e4840d8887a6513593a6505032
 SHA512:
-  metadata.gz: 6c636b00bd14fd976d53f9f97cbd2d53334ecb5677ad480ef00e737b51144da2f0ae76fffc1f6d1fb0366381961311e8aec93dfe6453cc2942c10dc652d87a88
-  data.tar.gz: 93c2432a8ce2725b2ca3ff9f1c8115a38154dc7c4426f7aa5667d12ed9546b9890d83885bcbe727064ea78972e58ab4e721b6dcef82d93c004d95f8a8ae6609f
+  metadata.gz: 114cb6e33e6867b1c4e4dadc43690e67feeb01cbcdab39bcd24d3d563e65323e43eb09dd2d78f8c5056f1b27faad3ba0f9383a31b5fb216674c92998dc718258
+  data.tar.gz: d89f76882f68ffbf6978407fa6f588e29fa87cf66f6cd64bc985af9510232168204d81925975f739e9cf2961e345b871a45c7ac8894e75f0ea0db6855a2d60d1

data/README.md

@@ -78,7 +78,30 @@ We use `Object.const_source_location` to find the file that defines the
 means that, if you are reopening the class somewhere else, that coverage won't
 be checked; if you are including 15 Concerns, and don't intend to write separate
 specs for them, be sure to list them as `covers:` metadata on the test. Also,
-shame!.
+shame!
+
+## Output
+
+When there's missing coverage on the class under test, you'll currently see
+output like this:
+
+```
+❯ rspec
+
+Randomized with seed 29392
+...............................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................
+An error occurred in an `after(:context)` hook.
+Failure/Error: fail(MissingCoverage, message)
+  Missing coverage in /Users/emueller/src/quiet_quality/lib/quiet_quality/message.rb on line 7
+# /Users/emueller/src/rspec-cover_it/lib/rspec/cover_it/context_coverage.rb:40:in `enforce!'
+# /Users/emueller/src/rspec-cover_it/lib/rspec/cover_it/coverage_state.rb:37:in `block in finish_tracking_for'
+# /Users/emueller/src/rspec-cover_it/lib/rspec/cover_it/coverage_state.rb:35:in `finish_tracking_for'
+# /Users/emueller/src/rspec-cover_it/lib/rspec/cover_it.rb:26:in `block (2 levels) in setup'
+.....................................................................................................................................................................................................................................................................................................
+
+Finished in 1.06 seconds (files took 0.28925 seconds to load)
+852 examples, 0 failures, 1 error occurred outside of examples
+```
 
 ## Drawbacks and Shortcomings
 

data/lib/rspec/cover_it/context_coverage.rb

@@ -8,6 +8,10 @@ module RSpec
 
       attr_accessor :precontext_coverage, :postcontext_coverage
 
+      def pretest_coverage
+        @_pretest_coverage ||= pretest_results[target_path]
+      end
+
       def local_coverage
         return nil unless precontext_coverage && postcontext_coverage
         @_local_coverage ||= pretest_coverage
@@ -20,18 +24,30 @@ module RSpec
         covered_line_count.to_f / coverable_line_count.to_f
       end
 
+      def enforce!
+        return if local_coverage_rate >= 1.0
+        lines = local_coverage.each_with_index.select { |v, _i| v&.zero? }.map(&:last)
+
+        summary =
+          if lines.length == 1
+            "on line #{lines.first}"
+          elsif lines.length <= 10
+            "on lines #{lines.map(&:to_s).join(", ")}"
+          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)
+      end
+
       private
 
-      attr_reader :context, :pretest_results, :precontext_coverage, :postcontext_coverage
+      attr_reader :context, :pretest_results
 
       def target_path
         @_target_path ||= context.target_path
       end
 
-      def pretest_coverage
-        @_pretest_coverage ||= pretest_results[target_path]
-      end
-
       # Really, we shouldn't see nil for any of these values unless they are all
       # nil. We want the coverage we'd expect to have seen if we ran _just_ this
       # groups of examples, which ought boe the pretest coverage, plus the

data/lib/rspec/cover_it/coverage_state.rb

@@ -34,7 +34,7 @@ module RSpec
 
         context_coverage_for(context).tap do |context_coverage|
           context_coverage.postcontext_coverage = Coverage.peek_result[context.target_path]
-          fail("missing coverage!") if context_coverage.local_coverage_rate < 1.0
+          context_coverage.enforce!
         end
       end
 

data/lib/rspec/cover_it/version.rb

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

metadata

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