check_please 0.5.4 → 0.5.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2bd44bf6bcc771bf5d0179fa562f5ddff80f0ed4bb2420ede197d60007d8057d
-  data.tar.gz: 7c0725d4b8cf3d03d7c8ffd2e8df51a8e5a1616e86c4cad02695dca73921340f
+  metadata.gz: 9c80defc851cfda5e74a5590da7be40388643084c1e06818639685bfe05e8a46
+  data.tar.gz: fe1a9a3b8be29b8e148917ed19052ae2d0195218624b0bf195421cc29c83c20b
 SHA512:
-  metadata.gz: 0add2fc010b651ae30a1309c3df9b26c1280e40fc7c45b5214ad22922fb3103c533f0f0ebbf26d46c3dbfb97b09d36ded3c9a419818039437aed2d198ced4669
-  data.tar.gz: 88611e7acca8a803f69e48ddefddfbc208faeae359fc3417b0f0faa0aae11e853a45891919290abd062f503558f0caedc3b05fa037397f3b4142b11a3905245c
+  metadata.gz: 0b70a2c80630b51d789327c36e9b288aa2484d2b80cc888f5e8a94acf6bed1d7eef67c659f7ac17a9e7ed6c14aacc8cad9c18414881b2cfe07bf4e96aaad785e
+  data.tar.gz: 2cde0b422e2ae297517680f2408201b3f7c88e2e1f81da667d2c875dfddd31b544bef57864dff3ac8739be01b09676e5fd22ff1cbb54677e64f791ff606d3a05

data/Gemfile.lock

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

data/lib/check_please.rb

@@ -18,7 +18,6 @@ module CheckPlease
   autoload :PathSegment,        "check_please/path_segment"
   autoload :PathSegmentMatcher, "check_please/path_segment_matcher"
   autoload :Printers,           "check_please/printers"
-  autoload :Refinements,        "check_please/refinements"
 end
 
 

data/lib/check_please/comparison.rb

@@ -1,5 +1,4 @@
 module CheckPlease
-  using Refinements
 
   class Comparison
     def self.perform(reference, candidate, flags = {})
@@ -7,7 +6,7 @@ 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

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|
@@ -56,6 +55,18 @@ module CheckPlease
       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/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/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.4" # about to release? rerun `bundle lock` to update Gemfile.lock
+  VERSION = "0.5.5" # about to release? rerun `bundle lock` to update Gemfile.lock
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: check_please
 version: !ruby/object:Gem::Version
-  version: 0.5.4
+  version: 0.5.5
 platform: ruby
 authors:
 - Sam Livingston-Gray
@@ -91,8 +91,8 @@ files:
 - 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