minitest-allow 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f24ec8511a2369224f41c4a036d52efa431c102690aa1f933097a863e13df8e0
-  data.tar.gz: b5e459178923fd55debdc76f1bd8e0647ababe7fccb457839c2f2599c0bb8f7c
+  metadata.gz: 5561e1b5e01bbc7bc937a0532707bb5c03ae369c66f39fa9a6d345f33ae5fb21
+  data.tar.gz: f6c2ee76c8e82fd30ad9ea3a248f83fb0a8e605cd130bb0c0b20dec1807a465c
 SHA512:
-  metadata.gz: a4e5a3e8bfda70b669f185b92280ea1146a18a61533d5508d256ea72f04672e1f3691505dad3b0c643d46ec98577bf681170cc7605c0db1edc82f1d60861eaa3
-  data.tar.gz: e9d36c2adcb5fa8005b4e2a7eeeecfe35b20bc0822f02ecccd71b419f6e63ffa46a7d544363ffffcf917d4105cc3e3bb23a9b8e36f3248a47ff088702cd46b21
+  metadata.gz: 2dcd4745beb67f57cd66bd973c19429497b0ae0021af438f3540ba6bb0566e5eca7d6b8e4fc3a7362d9626e662c10eedb3e938ac38fb61b770e15464523741e3
+  data.tar.gz: bb735fe2bb7b7ec75b700e0d28c39e1d26466502638c1a086048c35243a10d7314224ea01f51ac346d3229dc097bb8a9990c65514e778c99696353e26d666421

data/History.rdoc

@@ -1,3 +1,16 @@
+=== 1.2.0 / 2022-05-12
+
+* 1 major enhancement:
+
+  * The allow file allows for regexps to match errors and names by pattern.
+
+* 5 minor enhancements:
+
+  * Don't print allow fixes unless allow_save
+  * Explicity use psych. Don't ask.
+  * Output of good/bad tests is now in YAML format with easily searchable headers.
+  * Use safe_load_file if it is available and allow Regexps.
+
 === 1.1.0 / 2021-04-19
 
 * 2 minor enhancements:
@@ -10,4 +23,3 @@
 * 1 major enhancement
 
   * Birthday!
-

data/Rakefile

@@ -15,4 +15,16 @@ Hoe.spec "minitest-allow" do
   dependency "minitest", "~> 5.0"
 end
 
+Rake.application["test"].clear # hack? is there a better way?
+
+task :test => "test:filtered"
+
+namespace "test" do
+  Minitest::TestTask.create "filtered" do |t|
+    t.extra_args << "--allow=allow.yml"
+  end
+
+  Minitest::TestTask.create "unfiltered"
+end
+
 # vim: syntax=ruby

data/allow.yml

@@ -1,2 +1,3 @@
 ---
 - TestMinitest::TestAllow#test_sanity
+- !ruby/regexp /This is bad/

data/lib/minitest/allow_plugin.rb

@@ -3,12 +3,18 @@ module Minitest
     @allow = @allow_save = false
 
     opts.on "-a", "--allow=path", String, "Allow listed tests to fail." do |f|
-      require "yaml"
-      @allow = YAML.load File.read f
+      # don't ask why I'm using this specifically:
+      require "psych"
+
+      @allow = if Psych.respond_to? :safe_load_file then
+                 Psych.safe_load_file f, permitted_classes: [Regexp]
+               else
+                 Psych.load_file f
+               end
     end
 
     opts.on "-A", "--save-allow=path", String, "Save failing tests." do |f|
-      require "yaml"
+      require "psych"
       @allow_save = f
     end
   end
@@ -35,7 +41,7 @@ module Minitest
   end
 
   module Allow
-    VERSION = "1.1.0"
+    VERSION = "1.2.0"
 
     attr_accessor :allow, :allow_save, :allow_seen
 
@@ -57,25 +63,82 @@ module Minitest
         .uniq
         .sort
 
-      File.write allow_save, data.to_yaml
+      File.write allow_save, Psych.dump(data, line_width:-1)
     end
 
+    # Test runs call record, we put everything ran in allow_seen.
+    # This means allow_seen has everything RAN, regardless of pass/fail
+    # We use this intersected with the allowed file to determine what to report
+    # on... if it wasn't seen, we don't say add/remove at all.
+    #
+    # `allow` is all the stuff from the allow file. Test names and
+    # regexps that excuse failures.
+    #
+    # `allow_results` is an array of arrays of results: [[result, ...], ...]
+    #
+    # When the run is done, we want to tell the user what tests to
+    # remove or add from the allowed list:
+    #
+    # * If a test passed that is in the allowed list, it should be removed.
+    # * If a test failed that is NOT in the allowed list, it should be added.
+    # * If a test failed that is matched by a regexp, it's NAME should be removed
+    #   if it is listed.
+
     def filter_allow
-      maybe_bad = allow_results.flatten.map(&:full_name).uniq
-      to_remove = maybe_bad & allow
-      extra_bad = maybe_bad - to_remove
+      # 1. split allow into strings and regexps
+      allowed_REs, allowed_names = allow.partition { |a| Regexp === a }
+
+      allowed_names = allowed_names.map { |x| [x, x] }.to_h
 
-      self.allow -= to_remove
+      # 2. remove items from allow_results whose full_name matches the strings
+      # 3. remove items from allow_results whose message matches the regexps
+      # 4. remove items from allow_results whose full_name matches the regexps?
 
+      hit = {}
+      allow_results = self.allow_results
       allow_results.each do |results|
-        results.delete_if { |r| to_remove.include? r.full_name }
+        results.delete_if { |r|
+          name = r.full_name
+
+          by_name = allowed_names[name]
+          by_regx = allowed_REs.find { |re| r.failure.message =~ re || name =~ re }
+
+          # this will add the name as bad unless hit by regexp as well
+          # if hit by regex, then we want to report it as "good" so
+          # the name gets removed from the allow list:
+
+          hit[name] = true if by_name && !by_regx
+
+          by_name || by_regx
+        }
       end
 
+      # 5. remove string and regexps that matched any of the above from allow
+      self.allow -= hit.keys
+
+      errored, failed = allow_results
+        .flatten
+        .partition { |t| Minitest::UnexpectedError === t.failure }
+
+      failed = failed.map(&:full_name)
+      errors = Hash.new { |h,k| h[k] = [] }
+
+      errored.each do |t|
+        msg = t.failure.message.lines.first.chomp.gsub(/0x\h+/, "0xHHHH")
+
+        errors[Regexp.new(Regexp.escape(msg))] << t.full_name
+      end
+
+      extra_bad = failed.uniq
+      extra_bad << errors.transform_values(&:uniq) unless errors.empty?
+
+      # 6. report new failures including regular expressions for errors
+
       unless extra_bad.empty? then
         io.puts
         io.puts "Bad tests that are NOT allowed:"
+        Psych.dump extra_bad, io, line_width:-1
         io.puts
-        io.puts extra_bad.to_yaml
       end
     end
 
@@ -85,10 +148,8 @@ module Minitest
       unless good.empty? then
         io.puts
         io.puts "Excluded tests that now pass:"
+        Psych.dump good, io, line_width:-1
         io.puts
-        good.each do |name|
-          io.puts "  :allow_good: %p" % [name]
-        end
       end
     end
 

data/test/minitest/test_allow_plugin.rb

@@ -5,6 +5,13 @@ module TestMinitest; end
 
 class TestMinitest::TestAllow < Minitest::Test
   def test_sanity
-    flunk "write tests or I will kneecap you"
+    flunk "nah"
+  end
+
+  3.times do |n|
+    name = "test_regexp_%02d" % [n]
+    define_method name do
+      raise "This is bad %02d" % [n]
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: minitest-allow
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Ryan Davis
@@ -10,9 +10,9 @@ bindir: bin
 cert_chain:
 - |
   -----BEGIN CERTIFICATE-----
-  MIIDPjCCAiagAwIBAgIBBTANBgkqhkiG9w0BAQsFADBFMRMwEQYDVQQDDApyeWFu
+  MIIDPjCCAiagAwIBAgIBBjANBgkqhkiG9w0BAQsFADBFMRMwEQYDVQQDDApyeWFu
   ZC1ydWJ5MRkwFwYKCZImiZPyLGQBGRYJemVuc3BpZGVyMRMwEQYKCZImiZPyLGQB
-  GRYDY29tMB4XDTIwMTIyMjIwMzgzMFoXDTIxMTIyMjIwMzgzMFowRTETMBEGA1UE
+  GRYDY29tMB4XDTIxMTIyMzIzMTkwNFoXDTIyMTIyMzIzMTkwNFowRTETMBEGA1UE
   AwwKcnlhbmQtcnVieTEZMBcGCgmSJomT8ixkARkWCXplbnNwaWRlcjETMBEGCgmS
   JomT8ixkARkWA2NvbTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBALda
   b9DCgK+627gPJkB6XfjZ1itoOQvpqH1EXScSaba9/S2VF22VYQbXU1xQXL/WzCkx
@@ -22,14 +22,14 @@ cert_chain:
   qhtV7HJxNKuPj/JFH0D2cswvzznE/a5FOYO68g+YCuFi5L8wZuuM8zzdwjrWHqSV
   gBEfoTEGr7Zii72cx+sCAwEAAaM5MDcwCQYDVR0TBAIwADALBgNVHQ8EBAMCBLAw
   HQYDVR0OBBYEFEfFe9md/r/tj/Wmwpy+MI8d9k/hMA0GCSqGSIb3DQEBCwUAA4IB
-  AQAE3XRm1YZcCVjAJy5yMZvTOFrS7B2SYErc+0QwmKYbHztTTDY2m5Bii+jhpuxh
-  H+ETcU1z8TUKLpsBUP4kUpIRowkVN1p/jKapV8T3Rbwq+VuYFe+GMKsf8wGZSecG
-  oMQ8DzzauZfbvhe2kDg7G9BBPU0wLQlY25rDcCy9bLnD7R0UK3ONqpwvsI5I7x5X
-  ZIMXR0a9/DG+55mawwdGzCQobDKiSNLK89KK7OcNTALKU0DfgdTkktdgKchzKHqZ
-  d/AHw/kcnU6iuMUoJEcGiJd4gVCTn1l3cDcIvxakGslCA88Jubw0Sqatan0TnC9g
-  KToW560QIey7SPfHWduzFJnV
+  AQCKB5jfsuSnKb+t/Wrh3UpdkmX7TrEsjVmERC0pPqzQ5GQJgmEXDD7oMgaKXaAq
+  x2m+KSZDrqk7c8uho5OX6YMqg4KdxehfSLqqTZGoeV78qwf/jpPQZKTf+W9gUSJh
+  zsWpo4K50MP+QtdSbKXZwjAafpQ8hK0MnnZ/aeCsW9ov5vdXpYbf3dpg6ADXRGE7
+  lQY2y1tJ5/chqu6h7dQmnm2ABUqx9O+JcN9hbCYoA5i/EeubUEtFIh2w3SpO6YfB
+  JFmxn4h9YO/pVdB962BdBNNDia0kgIjI3ENnkLq0dKpYU3+F3KhEuTksLO0L6X/V
+  YsuyUzsMz6GQA4khyaMgKNSD
   -----END CERTIFICATE-----
-date: 2021-04-19 00:00:00.000000000 Z
+date: 2022-05-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
@@ -71,14 +71,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.22'
+        version: '3.23'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.22'
+        version: '3.23'
 description: |-
   Allows you to provide an exclusion list of allowed failures/errors.
   Failures and errors on this list still get run and reported as usual,
@@ -123,7 +123,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.16
+rubygems_version: 3.3.12
 signing_key:
 specification_version: 4
 summary: Allows you to provide an exclusion list of allowed failures/errors