check_please 0.5.2 → 0.5.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c39ef90c83a488ddfa309002bfad243cbfe2415481aa0d82e3efd7119be7e420
-  data.tar.gz: 54deb528cb61709484f86b11b4601e54770a6ce9a2a4694905eb7dcd3cceebd7
+  metadata.gz: fc034c2aab137c78fde2c231817f571a6c27fc25ba7380473d4ffe0f2e912356
+  data.tar.gz: 43f5364030a8d15b6c91c4b856bacebc5c83e769321f42d55c797f883ae36fb1
 SHA512:
-  metadata.gz: 2ab7404049194c7b70bf86b2ac4a3edc765e50b1ae63a2725e8877e67b1e49735a607f29f486414892a77d4fe604d8361db0b3bd6669b6c067e7ba90838c00bc
-  data.tar.gz: 97fd7e608cac462e278601372d79a3e9fdee8013f3a13912078aad3d86cf875ce4724f68902282433df9a8fcee314c206414e473b9026c30b06e77c271b34393
+  metadata.gz: c8f7142ae0758c3fed66bf3fb3af2e2beb8e5b320f7d8ed944cff3bc624390dcb93dbf3b7dc822c22cad91ebb634f6d9ce23dec98c912ba985812b0545c7e07f
+  data.tar.gz: dc05fc6736a4f1f1d83b3bd5557e09cbc9895bb95e0932fb78dc65ddfad3a3047ae969588db6531eaa452b3f8291b4c7bdc605fae6c366a6f0ee5a7fb9afdc5d

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    check_please (0.5.2)
+    check_please (0.5.7)
       table_print
 
 GEM

data/README.md

@@ -22,6 +22,7 @@ structures parsed from either of those.
          * [Repeatable Flags](#repeatable-flags)
          * [Expanded Documentation for Specific Flags](#expanded-documentation-for-specific-flags)
             * [Flag: match_by_key](#flag-match_by_key)
+            * [Flag: normalize_values](#flag-normalize_values)
    * [TODO (maybe)](#todo-maybe)
    * [Development](#development)
    * [Contributing](#contributing)
@@ -420,6 +421,37 @@ the following path expression: `/authors/id=1/books/isbn=12345`
 **This syntax is intended to be readable by humans first.**  If you need to
 build tooling that consumes it... well, I'm open to suggestions.  :)
 
+#### Flag: `normalize_values`
+
+NOTE: This flag is only accessible via the Ruby API.
+(I have no idea how to reasonably express it in a CLI flag.)
+
+Before comparing values at specified paths, normalize both values using
+the provided message or Proc.
+
+To use an example from the tests, the following reference/candidate pair would
+normally create three "mismatch" diffs:
+
+```ruby
+  ref = { list: [ "foo", "bar", "yak" ] }
+  can = { list: [ :foo,  :bar,  :yak ] }
+```
+
+However, the values can be converted to String before comparison via any of the following:
+
+```ruby
+  CheckPlease.diff(ref, can, normalize_values: { "/list/*" => ->(v) { v.to_s } })
+  CheckPlease.diff(ref, can, normalize_values: { "/list/*" => :to_s })
+  CheckPlease.diff(ref, can, normalize_values: { "/list/*" => "to_s" })
+```
+
+Note that the value of the flag is a Hash.
+* Its keys must be strings representing path expressions.
+* If the value associated with a given path is a lambda/proc, it will be
+  called with both the reference value and the candidate value.
+* If the value is a String or Symbol, it will be sent as a message to
+  both the reference and candidate values using Object#send.
+
 -----
 
 # TODO (maybe)

data/lib/check_please.rb

@@ -7,17 +7,17 @@ require "check_please/error"
 require "check_please/version"
 
 module CheckPlease
-  autoload :Reification, "check_please/reification"
-  autoload :CLI,         "check_please/cli"
-  autoload :Comparison,  "check_please/comparison"
-  autoload :Diff,        "check_please/diff"
-  autoload :Diffs,       "check_please/diffs"
-  autoload :Flag,        "check_please/flag"
-  autoload :Flags,       "check_please/flags"
-  autoload :Path,        "check_please/path"
-  autoload :PathSegment, "check_please/path_segment"
-  autoload :Printers,    "check_please/printers"
-  autoload :Refinements, "check_please/refinements"
+  autoload :Reification,        "check_please/reification"
+  autoload :CLI,                "check_please/cli"
+  autoload :Comparison,         "check_please/comparison"
+  autoload :Diff,               "check_please/diff"
+  autoload :Diffs,              "check_please/diffs"
+  autoload :Flag,               "check_please/flag"
+  autoload :Flags,              "check_please/flags"
+  autoload :Path,               "check_please/path"
+  autoload :PathSegment,        "check_please/path_segment"
+  autoload :PathSegmentMatcher, "check_please/path_segment_matcher"
+  autoload :Printers,           "check_please/printers"
 end
 
 
@@ -143,6 +143,20 @@ module CheckPlease
     EOF
   end
 
+  Flags.define :match_by_value do |flag|
+    flag.repeatable
+    flag.coerce { |value| CheckPlease::Path.reify(value) }
+
+    flag.cli_long = "--match-by-value FOO"
+    flag.description = <<~EOF
+      When comparing two arrays that match a specified path, the candidate
+      array will be scanned for each element in the reference array.
+        May be repeated; values will be treated as an 'OR' list.
+        NOTE: explodes if either array at a given path contains other collections.
+        NOTE: paths of 'extra' diffs use the index in the candidate array.
+    EOF
+  end
+
   Flags.define :indifferent_keys do |flag|
     flag.default = false
     flag.coerce { |value| !!value }
@@ -163,4 +177,9 @@ module CheckPlease
     EOF
   end
 
+  Flags.define :normalize_values do |flag|
+    # NOTE: This flag is only accessible via the Ruby API.
+    #       See the README for documentation.
+  end
+
 end

data/lib/check_please/comparison.rb

@@ -1,5 +1,4 @@
 module CheckPlease
-  using Refinements
 
   class Comparison
     def self.perform(reference, candidate, flags = {})
@@ -7,13 +6,13 @@ module CheckPlease
     end
 
     def perform(reference, candidate, flags = {})
-      @flags = Flags(flags) # whoa, it's almost like Java in here
+      @flags = Flags.reify(flags)
       @diffs = Diffs.new(flags: @flags)
 
       catch(:max_diffs_reached) do
         compare reference, candidate, CheckPlease::Path.root
       end
-      diffs
+      diffs.filter_by_flags(@flags)
     end
 
     private
@@ -42,8 +41,11 @@ module CheckPlease
       end
 
       def compare_arrays(ref_array, can_array, path)
-        if ( key = path.key_for_compare(flags) )
+        case
+        when ( key = path.key_to_match_by(flags) )
           compare_arrays_by_key ref_array, can_array, path, key
+        when path.match_by_value?(flags)
+          compare_arrays_by_value ref_array, can_array, path
         else
           compare_arrays_by_index ref_array, can_array, path
         end
@@ -107,6 +109,45 @@ module CheckPlease
             elements_by_key
           end
 
+        # FIXME: this can generate duplicate paths.
+        # Time to introduce lft_path, rgt_path ?
+        def compare_arrays_by_value(ref_array, can_array, path)
+          assert_can_match_by_value! ref_array
+          assert_can_match_by_value! can_array
+
+          matches = can_array.map { false }
+
+          # Look for missing values
+          ref_array.each.with_index do |ref, i|
+            new_path = path + (i+1) # count in human pls
+
+            # Weird, but necessary to handle duplicates properly
+            j = can_array.index.with_index { |can, j|
+              matches[j] == false && can == ref
+            }
+
+            if j
+              matches[j] = true
+            else
+              record_diff ref, nil, new_path, :missing
+            end
+          end
+
+          # Look for extra values
+          can_array.zip(matches).each.with_index do |(can, match), i|
+            next if match
+            new_path = path + (i+1) # count in human pls
+            record_diff nil, can, new_path, :extra
+          end
+        end
+
+          def assert_can_match_by_value!(array)
+            if array.any? { |e| Array === e || Hash === e }
+              raise CheckPlease::BehaviorUndefined,
+                "match_by_value behavior is not defined for collections!"
+            end
+          end
+
         def compare_arrays_by_index(ref_array, can_array, path)
           max_len = [ ref_array, can_array ].map(&:length).max
           (0...max_len).each do |i|
@@ -169,14 +210,36 @@ module CheckPlease
         end
 
       def compare_others(ref, can, path)
-        if flags.indifferent_values
-          ref = stringify_symbol(ref)
-          can = stringify_symbol(can)
-        end
-        return if ref == can
+        ref_prime = normalize_value(path, ref)
+        can_prime = normalize_value(path, can)
+        return if ref_prime == can_prime
+
         record_diff ref, can, path, :mismatch
       end
 
+        def normalize_value(path, value)
+          if flags.indifferent_values
+            value = stringify_symbol(value)
+          end
+
+          if flags.normalize_values
+            # We assume that normalize_values is a hash of path expression strings to a proc, string, or symbol that will be used to normalize the value
+            _, normalizer = flags.normalize_values.detect { |path_string, a_proc|
+              path.match?(path_string)
+            }
+
+            case normalizer
+            when nil            ; value
+            when Proc           ; normalizer.call(value)
+            when String, Symbol ; value.send(normalizer)
+            else                ; raise ArgumentError, "Not sure how to use #{normalizer.inspect} to normalize #{value.inspect}"
+            end
+          else
+            return value
+          end
+
+        end
+
     def record_diff(ref, can, path, type)
       diff = Diff.new(type, path, ref, can)
       diffs << diff

data/lib/check_please/diffs.rb

@@ -1,14 +1,13 @@
 require 'forwardable'
 
 module CheckPlease
-  using Refinements
 
   # Custom collection class for Diff instances.
   # Can retrieve members using indexes or paths.
   class Diffs
     attr_reader :flags
     def initialize(diff_list = nil, flags: {})
-      @flags = Flags(flags)
+      @flags = Flags.reify(flags)
       @list = []
       @hash = {}
       Array(diff_list).each do |diff|
@@ -47,10 +46,27 @@ module CheckPlease
       @list.map(&:attributes)
     end
 
+    def filter_by_flags(flags)
+      new_list = @list.reject { |diff| Path.new(diff.path).excluded?(flags) }
+      self.class.new(new_list, flags: flags)
+    end
+
     def to_s(flags = {})
       CheckPlease::Printers.render(self, flags)
     end
 
+    def method_missing(meth, *args, &blk)
+      if formats.include?(meth.to_sym)
+        CheckPlease::Printers.render(self, format: meth)
+      else
+        super
+      end
+    end
+
+    def formats
+      CheckPlease::Printers::FORMATS
+    end
+
     extend Forwardable
     def_delegators :@list, *%i[
       each

data/lib/check_please/error.rb

@@ -6,6 +6,10 @@ module CheckPlease
     # instead....
   end
 
+  class BehaviorUndefined < ::StandardError
+    include CheckPlease::Error
+  end
+
   class DuplicateKeyError < ::IndexError
     include CheckPlease::Error
   end

data/lib/check_please/flags.rb

@@ -3,6 +3,9 @@ module CheckPlease
   # NOTE: this gets all of its attributes defined (via .define) in ../check_please.rb
 
   class Flags
+    include CheckPlease::Reification
+    can_reify Hash
+
     BY_NAME = {} ; private_constant :BY_NAME
 
     def self.[](name)

data/lib/check_please/path.rb

@@ -17,7 +17,12 @@ module CheckPlease
     def initialize(name_or_segments = [])
       case name_or_segments
       when String, Symbol, Numeric, nil
-        names = name_or_segments.to_s.split(SEPARATOR)
+        string = name_or_segments.to_s
+        if string =~ %r(//)
+          raise InvalidPath, "paths cannot have empty segments"
+        end
+
+        names = string.split(SEPARATOR)
         names.shift until names.empty? || names.first =~ /\S/
         segments = PathSegment.reify(names)
       when Array
@@ -26,10 +31,6 @@ module CheckPlease
         raise InvalidPath, "not sure what to do with #{name_or_segments.inspect}"
       end
 
-      if segments.any?(&:empty?)
-        raise InvalidPath, "#{self.class.name} cannot contain empty segments"
-      end
-
       @segments = Array(segments)
 
       @to_s = SEPARATOR + @segments.join(SEPARATOR)
@@ -81,13 +82,10 @@ module CheckPlease
       "<#{self.class.name} '#{to_s}'>"
     end
 
-		# TODO: Naming Things
-    def key_for_compare(flags)
-      mbk_exprs = unpack_mbk_exprs(flags)
-      matches = mbk_exprs.select { |mbk_expr|
-        # NOTE: matching on parent because MBK '/foo/:id' should return 'id' for path '/foo'
-        mbk_expr.parent.match?(self)
-      }
+    def key_to_match_by(flags)
+      key_exprs = unpack_key_exprs(flags.match_by_key)
+      # NOTE: match on parent because if self.to_s == '/foo', MBK '/foo/:id' should return 'id'
+      matches = key_exprs.select { |e| e.parent.match?(self) }
 
       case matches.length
       when 0 ; nil
@@ -96,6 +94,10 @@ module CheckPlease
       end
     end
 
+    def match_by_value?(flags)
+      flags.match_by_value.any? { |e| e.match?(self) }
+    end
+
     def match?(path_or_string)
       # If the strings are literally equal, we're good..
       return true if self == path_or_string
@@ -123,7 +125,7 @@ module CheckPlease
     # (as of this writing, this should never actually happen, but I'm being thorough)
     def ancestor_on_list?(paths)
       paths.any? { |path|
-        ancestors.any? { |ancestor| ancestor == path }
+        ancestors.any? { |ancestor| ancestor.match?(path) }
       }
     end
 
@@ -152,7 +154,7 @@ module CheckPlease
 
     # O(n) check to see if the path itself is on a list
     def self_on_list?(paths)
-      paths.any? { |path| self == path }
+      paths.any? { |path| self.match?(path) }
     end
 
     def too_deep?(flags)
@@ -160,8 +162,8 @@ module CheckPlease
       depth > flags.max_depth
     end
 
-    def unpack_mbk_exprs(flags)
-      flags.match_by_key
+    def unpack_key_exprs(path_list)
+      path_list
         .map { |path| path.send(:key_exprs) }
         .flatten
         .uniq { |e| e.to_s } # use the block form so we don't have to implement #hash and #eql? in horrible ways

data/lib/check_please/path_segment.rb

@@ -30,19 +30,16 @@ module CheckPlease
 
     def initialize(name = nil)
       @name = name.to_s.strip
-      if @name =~ %r(\s) # has any whitespace
-        raise InvalidPathSegment, <<~EOF
-          #{name.inspect} is not a valid #{self.class} name
-        EOF
+
+      case @name
+      when "", /\s/ # blank or has any whitespace
+        raise InvalidPathSegment, "#{name.inspect} is not a valid #{self.class} name"
       end
+
       parse_key_and_value
       freeze
     end
 
-    def empty?
-      name.empty?
-    end
-
     def key_expr?
       name.match?(KEY_EXPR)
     end
@@ -52,23 +49,12 @@ module CheckPlease
     end
 
     def match?(other_segment_or_string)
-      other = self.class.reify(other_segment_or_string)
-
-      match_types = [ self.match_type, other.match_type ]
-      case match_types
-      when [ :plain,     :plain     ] ; self.name == other.name
-      when [ :key,       :key_value ] ; self.key  == other.key
-      when [ :key_value, :key       ] ; self.key  == other.key
-      else                            ; false
-      end
+      other = reify(other_segment_or_string)
+      PathSegmentMatcher.call(self, other)
     end
 
-    protected
-
-    def match_type
-      return :key       if key_expr?
-      return :key_value if key_val_expr?
-      :plain
+    def splat?
+      name == '*'
     end
 
     private

data/lib/check_please/path_segment_matcher.rb

@@ -0,0 +1,44 @@
+module CheckPlease
+
+  class PathSegmentMatcher
+    def self.call(a,b)
+      new(a,b).call
+    end
+
+    attr_reader :a, :b, :types
+    def initialize(a, b)
+      @a, @b = a, b
+      @types = [ _type(a), _type(b) ].sort
+    end
+
+    def call
+      return true             if either?(:splat)
+      return a.name == b.name if both?(:plain)
+      return a.key  == b.key  if key_and_key_value?
+
+      false
+    end
+
+    private
+
+    def _type(x)
+      return :splat     if x.splat?
+      return :key       if x.key_expr?
+      return :key_value if x.key_val_expr?
+      :plain
+    end
+
+    def both?(type)
+      types.uniq == [type]
+    end
+
+    def either?(type)
+      types.include?(type)
+    end
+
+    def key_and_key_value?
+      types == [ :key, :key_value ]
+    end
+  end
+
+end

data/lib/check_please/printers.rb

@@ -1,20 +1,21 @@
 module CheckPlease
-  using Refinements
 
   module Printers
     autoload :Base,       "check_please/printers/base"
     autoload :JSON,       "check_please/printers/json"
+    autoload :Long,       "check_please/printers/long"
     autoload :TablePrint, "check_please/printers/table_print"
 
     PRINTERS_BY_FORMAT = {
       table: Printers::TablePrint,
       json:  Printers::JSON,
+      long:  Printers::Long,
     }
     FORMATS = PRINTERS_BY_FORMAT.keys.sort
     DEFAULT_FORMAT = :table
 
     def self.render(diffs, flags = {})
-      flags = Flags(flags)
+      flags = Flags.reify(flags)
       printer = PRINTERS_BY_FORMAT[flags.format]
       printer.render(diffs)
     end

data/lib/check_please/printers/long.rb

@@ -0,0 +1,31 @@
+module CheckPlease
+module Printers
+
+  class Long < Base
+    def to_s
+      return "" if diffs.empty?
+
+      out = build_string do |io|
+        diffs.each do |diff|
+          t = diff.type.to_sym
+          ref, can = *[ diff.reference, diff.candidate ].map(&:inspect)
+          diff_string = <<~EOF.strip
+            #{diff.path} [#{diff.type}]
+              reference: #{( t == :extra   ) ? "[no value]" : ref}
+              candidate: #{( t == :missing ) ? "[no value]" : can}
+          EOF
+
+          io.puts diff_string
+          io.puts
+        end
+      end
+
+      out.strip
+    end
+
+    private
+  end
+
+end
+end
+

data/lib/check_please/version.rb

@@ -1,5 +1,5 @@
 module CheckPlease
   # NOTE: 'check_please_rspec_matcher' depends on this,
   #       so try to keep them roughly in sync
-  VERSION = "0.5.2" # about to release? rerun `bundle check` to update Gemfile.lock also
+  VERSION = "0.5.7" # about to release? rerun `bundle lock` to update Gemfile.lock
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: check_please
 version: !ruby/object:Gem::Version
-  version: 0.5.2
+  version: 0.5.7
 platform: ruby
 authors:
 - Sam Livingston-Gray
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-01-29 00:00:00.000000000 Z
+date: 2021-03-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: table_print
@@ -87,11 +87,12 @@ files:
 - lib/check_please/flags.rb
 - lib/check_please/path.rb
 - lib/check_please/path_segment.rb
+- lib/check_please/path_segment_matcher.rb
 - lib/check_please/printers.rb
 - lib/check_please/printers/base.rb
 - lib/check_please/printers/json.rb
+- lib/check_please/printers/long.rb
 - lib/check_please/printers/table_print.rb
-- lib/check_please/refinements.rb
 - lib/check_please/reification.rb
 - lib/check_please/version.rb
 - usage_examples.rb

data/lib/check_please/refinements.rb

@@ -1,16 +0,0 @@
-module CheckPlease
-
-  module Refinements
-    refine Kernel do
-      def Flags(flags_or_hash)
-        case flags_or_hash
-        when Flags ; return flags_or_hash
-        when Hash  ; return Flags.new(flags_or_hash)
-        else
-          raise ArgumentError, "Expected either a CheckPlease::Flags or a Hash; got #{flags_or_hash.inspect}"
-        end
-      end
-    end
-  end
-
-end