minitest-holdify 1.3.4 → 1.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 91ac2c93846bb953d73a2cee3f89f8985b5ed1596b13995423212e91020cf52f
-  data.tar.gz: c3d196b9a65791d40aa3255a2c11345bebecf989a7c154d0e7d93cb4b9302294
+  metadata.gz: d19382125e8e102ff0d8b01ca8227e68b44c7d0a532e99b5b2e7236a37a37407
+  data.tar.gz: cc34821ba517f701180baa40391421cc6d888d18a079be4c1445d6bda43578dc
 SHA512:
-  metadata.gz: 20007012180156142177f200c8874325e446638ab65fd87ecd6aa7f3377473fce0ff8dee1c88da698dc5aff77318cd66d6ad2bf3c419fabbc9c77170ff9b909d
-  data.tar.gz: aad5cb0fd0683f63fe3754c39745e87674e0df1c98b2babc4facc926d79bf1d0c9d3f643dd72d156acf29b18a159c0ea79e21b4e0e68ff384bc7c1402a2ae60e
+  metadata.gz: '0981c21b5932c8931c467642c6b1575bd3c61f34430f9330eb3c8c22d6712de5dba67ce6b31b8f8f59ed8e2778ddf3d393570721eda84496a35001e394e22588'
+  data.tar.gz: 5a73f8ab4dedddfe7694651d9a582641926648810c8dc30bec911a10c0f60dd63fa14b2b38422fd861b8abe18e9890b86de182cb73a28b7026b1bc8e7428912d

data/lib/holdify/feedback.rb

@@ -8,12 +8,8 @@ module Holdify
   # Feedback report on failure
   class Feedback
     def initialize(hold, hold_ref, *args)
-      test_lno  = hold.test_loc.lineno
-      index     = hold.session[test_lno].size - 1   # current index
-      xxh       = hold.store.xxh(test_lno)
       yaml_path = hold.store.path
-
-      @yaml_lno = find_yaml_lno(yaml_path, test_lno, xxh, index)
+      @yaml_lno = find_yaml_lno(hold, yaml_path)
       @yaml_ref = Holdify.relativize("#{yaml_path}:#{@yaml_lno}")
 
       @hold_ref = Holdify.relativize(hold_ref)
@@ -34,7 +30,7 @@ module Holdify
       extend Color::GitDiff
     end
 
-    def message = [@message, *file_refs, *diff, ''].join("\n")
+    def render = [@message, *file_refs, *diff, ''].join("\n")
 
     def file_refs
       ["--- @stored --> #{@yaml_ref}", "+++ @tested --> #{@hold_ref}"].tap do |refs|
@@ -46,7 +42,11 @@ module Holdify
 
     private
 
-    def find_yaml_lno(yaml_path, test_lno, xxh, index)
+    def find_yaml_lno(hold, yaml_path)
+      test_lno = hold.test_loc.lineno
+      index    = hold.session[test_lno].size - 1   # current index
+      xxh      = hold.store.xxh(test_lno)
+
       found = false
       count = -1
       File.foreach(yaml_path).with_index(1) do |line, ln|
@@ -91,8 +91,8 @@ module Holdify
       end
 
       def process_lines(lines)
-        width   = 0
-        lineno  = [@yaml_lno - 1]
+        width  = 0
+        lineno = [@yaml_lno - 1]
         lines.map.with_index do |line, i|
           if i.zero? # @@ ... @@
             width, line = render_hunk(line)
@@ -105,14 +105,16 @@ module Holdify
         end.compact
       end
 
+      # Reduce the hunk lines by 1 and calculate the gutter width
       def render_hunk(line)
         width = 0
         hunk  = line.gsub(/([-+]\d+),(\d+)\s+([-+]\d+),(\d+)/) do
-          v1, = $2.to_i - 1          # rubocop:disable Style/PerlBackrefs
-          v2  = $4.to_i - 1          # rubocop:disable Style/PerlBackrefs
-          width = (@yaml_lno + [v1, v2].max).to_s.length
-          "#{$1},#{v1} #{$3},#{v2}"  # rubocop:disable Style/PerlBackrefs
-        end
+                  v1,   = $2.to_i - 1          # rubocop:disable Style/PerlBackrefs
+                  v2    = $4.to_i - 1          # rubocop:disable Style/PerlBackrefs
+                  width = (@yaml_lno + [v1, v2].max).to_s.length
+                  "#{$1},#{v1} #{$3},#{v2}"    # rubocop:disable Style/PerlBackrefs
+                end
+
         [width, hunk]
       end
 
@@ -150,7 +152,8 @@ module Holdify
       # Methods enabling the git-diff ANSI feedback
       module GitDiff
         def git_command
-          re = '[[:alnum:]:._-]+|[^[:alnum:]:._-]|[[:space:]]+' # YAML and HTML friendly: avoid long continuous tokens
+          # Words/ruby entities | indent | other single-chars
+          re = '[[:alnum:]_:]+|[[:space:]]+|.'
           "git diff --no-index --color-words='#{re}' --unified=1000 #{@exp_path} #{@act_path}"
         end
 
@@ -167,7 +170,7 @@ module Holdify
                      "#{sgr}#{type}#{' ' * width}#{SGR[:clear]}"
                    else
                      lineno[0] += 1
-                     "#{sgr}#{type || ' '}#{lineno[0].to_s.rjust(width)}#{SGR[:clear]}"
+                     "#{sgr}#{type || ' '}#{lineno[0].to_s.rjust(width)}#{SGR[:clear] if sgr}"
                    end
 
           "#{gutter} #{line}"

data/lib/holdify/hold.rb

@@ -39,6 +39,10 @@ module Holdify
       @session.each { |lineno, values| @store.set_values(lineno, values) }
     end
 
+    def feedback(*) = Feedback.new(self, *).render
+
+    private
+
     # Find the triggering LOC inside the test block/method
     def find_test_loc
       caller_locations(2).find do |location|
@@ -48,7 +52,5 @@ module Holdify
         label == @test.name || label == '<top (required)>' || label == '<main>' || label.start_with?('<class:', '<module:')
       end
     end
-
-    def feedback(*) = Feedback.new(self, *).message
   end
 end

data/lib/holdify.rb

@@ -7,7 +7,7 @@ require_relative 'holdify/store'
 
 # The container module
 module Holdify
-  VERSION = '1.3.4'
+  VERSION = '1.3.5'
 
   @fresh_mutex  = Mutex.new
   @fresh        = []
@@ -23,7 +23,7 @@ module Holdify
       end
     end
 
-    def fresh_report
+    def warn_fresh_values
       return if quiet
 
       @fresh_mutex.synchronize do

data/lib/minitest/holdify_plugin.rb

@@ -19,8 +19,8 @@ module Minitest
     opts.on '--holdify-no-color', 'Disable colored output' do
       options[:holdify_no_color] = true
     end
-    opts.on '--holdify-no-relative-paths', 'Disable relative paths in file references' do
-      options[:holdify_no_relative_paths] = true
+    opts.on '--holdify-absolute-paths', 'Disable relative paths in file references' do
+      options[:holdify_absolute_paths] = true
     end
     opts.on '--holdify-store-ext EXT', 'The yaml store extension (default .yaml)' do |ext|
       options[:holdify_store_ext] = ext
@@ -38,19 +38,19 @@ module Minitest
     Holdify.quiet     = options[:holdify_quiet]
     Holdify.git_diff  = git unless options[:holdify_no_git_diff]
     Holdify.color     = !ENV.key?('NO_COLOR') unless options[:holdify_no_color]
-    Holdify.rel_paths = true unless options[:holdify_no_relative_paths]
+    Holdify.rel_paths = true unless options[:holdify_absolute_paths]
     Holdify.store_ext = options[:holdify_store_ext] || '.yaml'
 
     Minitest.after_run do
       Holdify.persist_stores!
-      Holdify.fresh_report
+      Holdify.warn_fresh_values
     end
   end
 
   # Patching Minitest::Assertion
   class Assertion
     remove_const :RE
-    RE = /in [`'](?:[^']+[#.])?(?:assert|refute|flunk|pass|fail|raise|must|wont|to)/ # :nodoc:
+    RE = /in [`'](?:[^']+[#.])?(?:assert|refute|flunk|pass|fail|raise|must|wont|to)/ # add to (to_hold)
   end
 
   # Reopen the minitest class

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: minitest-holdify
 version: !ruby/object:Gem::Version
-  version: 1.3.4
+  version: 1.3.5
 platform: ruby
 authors:
 - Domizio Demichelis