approvals 0.0.22 → 0.0.26

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 6f444a0aa1c3b1b8adeade0729913ffd0bc77af0
-  data.tar.gz: fcb304998044a0f34fdcf339d3bac514e6175529
+SHA256:
+  metadata.gz: cc889240c80c9fb3649ef9b9edfd7b7fa54d2921d678dfe4303c3821ee2a23c8
+  data.tar.gz: 30c4924a8e599a89c9c386892719ebfa9bdc3c6cc0664be606de4dc83c23c217
 SHA512:
-  metadata.gz: 0425fc7fab8349a9d762e8f134feebeba3d49c632f2bc3e4dc09df1b35041526acf86a6eda2f98b44b6443c968c2e32778741d984e73632124195f98c666f1ee
-  data.tar.gz: 503eef74aa5ed03cf9591ca43dc771bfc5fde4c5fb2d57bd1823ea7dfab4cbcbd95bebeba4cd1cb448b8fea32bf8bc4909b206fa1934b7eab2c8906170729dfd
+  metadata.gz: 8cb1489bae6337b513eac1e1cd78a789612357bd43d35bd874b324ee03c4ae8314e5fbf5377853cbe60effea3b2e87b85a1028bb3c62910dab13e1aae912f3a9
+  data.tar.gz: 99756cc25f1c2f3492de6291280b3483d861fe4977d11c19a5eee9c0372541457b05a7704c9fe6eabc909c0bcb6817c89ddf13bc89d0b6b009a0b27607abd2f9

data/.gitignore

@@ -5,3 +5,7 @@ pkg/*
 *.received.txt
 .approvals
 tmp/
+vendor/
+
+ext/Rakefile
+.ruby-version

data/.travis.yml

@@ -1,10 +1,27 @@
 language: ruby
 rvm:
-  - 1.9.2
-  - 1.9.3
   - 2.0.0
-  - 2.1.0
+  - 2.1.10
+  - 2.2.10
+  - 2.3.8
+  - 2.4.8
+  - 2.5.7
+  - 2.6.5
+  - 2.7.0
   - ruby-head
-  - rbx-2
+  - rbx-3
+matrix:
+  allow_failures:
+    - rvm: rbx-3
+before_install:
+  - |
+    r_eng="$(ruby -e 'STDOUT.write RUBY_ENGINE')";
+    rv="$(ruby -e 'STDOUT.write RUBY_VERSION')";
+    if [ "$r_eng" == "ruby" ]; then
+      if   [ "$rv" \< "2.3" ]; then gem update --system 2.7.10 --no-document
+      elif [ "$rv" \< "2.6" ]; then gem update --system --no-document --conservative
+      fi
+    fi
+  - gem update bundler
 script: bundle exec rspec spec
 cache: bundler

data/CHANGELOG.md

@@ -4,6 +4,8 @@
 
 * Your contribution here
 
+* Updated gemspec to make 'json' an explicit dependency
+
 ### v0.0.22 (2015-10-22)
 
 * Fix bug in non-binary comparisons.

data/README.md

@@ -23,8 +23,8 @@ which Llewellyn Falco is interviewed about approvals.
 ## Configuration
 
 ```ruby
-Approvals.configure do |c|
-  c.approvals_path = 'output/goes/here/'
+Approvals.configure do |config|
+  config.approvals_path = 'output/goes/here/'
 end
 ```
 
@@ -42,18 +42,43 @@ Approvals.verify(your_subject, :format => :json)
 
 This will raise an `ApprovalError` in the case of a failure.
 
-The default writer uses the `:to_s` method on the subject will be used to generate the output for
-the `received` file. For custom complex objects you will need to override
-`:to_s` to get helpful output, rather than the default:
-
-    #<Object:0x0000010105ea40> # or whatever the object id is
-
 The first time the approval is run, a file will be created with the contents of the subject of your approval:
 
     the_name_of_the_approval.received.txt # or .json, .html, .xml as appropriate
 
 Since you have not yet approved anything, the `*.approved` file does not exist, and the comparison will fail.
 
+### Customizing formatted output
+
+The default writer uses the `:to_s` method on the subject to generate the output for the received file.
+For custom complex objects you will need to provide a custom writer to get helpful output, rather than the default:
+
+    #<Object:0x0000010105ea40> # or whatever the object id is
+
+Create a custom writer class somewhere accessible to your test:
+
+```
+class MyCustomWriter < Approvals::Writers::TextWriter
+  def format(data)
+    # Custom data formatting here
+  end
+
+  def filter(data)
+    # Custom data filtering here
+  end
+end
+```
+
+In your test, use a string to reference your custom class:
+
+```
+it "verifies a complex object" do
+  Approvals.verify hello, :format => "MyCustomWriter"
+end
+```
+
+Define and use different custom writers as needed!
+
 ## CLI
 
 The gem comes with a command-line tool that makes it easier to manage the
@@ -115,17 +140,17 @@ spec/fixtures/approvals/
 You can override this:
 
 ```ruby
-RSpec.configure do |c|
-  c.approvals_path = 'some/other/path'
+RSpec.configure do |config|
+  config.approvals_path = 'some/other/path'
 end
 ```
 
 The basic format of the approval is modeled after RSpec's `it`:
 
 ```ruby
-it "works" do
+it 'works' do
   verify do
-    "this is the the thing you want to verify"
+    'this is the the thing you want to verify'
   end
 end
 ```
@@ -135,21 +160,21 @@ end
 When using RSpec, the namer is set for you, using the example's `full_description`.
 
 ```ruby
-Approvals.verify(thing, :name => "the name of your test")
+Approvals.verify(thing, :name => 'the name of your test')
 ```
 
 ### Formatting
 
 You can pass a format for your output before it gets written to the file.
-At the moment, only text, xml, html, and json are supported.
+At the moment, only text, xml, html, and json are supported, while text is the default.
 
-Simply add a `:format => :text`, `:format => :xml`, `:format => :html`, or `:format => :json` option to the example:
+Simply add a `:format => :txt`, `:format => :xml`, `:format => :html`, or `:format => :json` option to the example:
 
 ```ruby
-page = "<html><head></head><body><h1>ZOMG</h1></body></html>"
+page = '<html><head></head><body><h1>ZOMG</h1></body></html>'
 Approvals.verify page, :format => :html
 
-data = "{\"beverage\":\"coffee\"}"
+data = '{\'beverage\':\'coffee\'}'
 Approvals.verify data, :format => :json
 ```
 
@@ -157,19 +182,27 @@ In RSpec, it looks like this:
 
 ```ruby
 verify :format => :html do
-  "<html><head></head><body><h1>ZOMG</h1></body></html>"
+  '<html><head></head><body><h1>ZOMG</h1></body></html>'
 end
 
 verify :format => :json do
-  "{\"beverage\":\"coffee\"}"
+  '{\'beverage\':\'coffee\'}'
+end
+```
+
+If you like you could also change the default format globally with:
+
+```ruby
+RSpec.configure do |config|
+  config.approvals_default_format = :json # or :xml, :html
 end
 ```
 
 ### Exclude dynamically changed values from json
 
 ```ruby
-Approvals.configure do |c|
-  c.excluded_json_keys = {
+Approvals.configure do |config|
+  config.excluded_json_keys = {
     :id =>/(\A|_)id$/,
     :date => /_at$/
   }

data/approvals.gemspec

@@ -9,9 +9,9 @@ Gem::Specification.new do |s|
   s.licenses    = ['MIT']
   s.authors     = ["Katrina Owen"]
   s.email       = ["katrina.owen@gmail.com"]
-  s.homepage    = ""
+  s.homepage    = "https://github.com/kytrinyx/approvals"
   s.summary     = %q{Approval Tests for Ruby}
-  s.description = %q{Approval Tests for Ruby}
+  s.description = %q{A library to make it easier to do golden-master style testing in Ruby}
 
   s.rubyforge_project = "approvals"
 
@@ -20,8 +20,17 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
+  s.add_dependency 'thor', '~> 1.0'
+  s.add_dependency 'json', '~> 2.0'
+
+  if RUBY_VERSION < "2.1"
+    s.add_dependency 'nokogiri', '~> 1.6.8'
+  else
+    s.add_dependency 'nokogiri', '~> 1.8'
+  end
+  # We also depend on the json gem, but the version we need is
+  # Ruby-version-specific. See `ext/mkrf_conf.rb`.
+
+  s.add_development_dependency 'rake', '~> 13.0'
   s.add_development_dependency 'rspec', '~> 3.1'
-  s.add_development_dependency 'json', '~> 1.8'
-  s.add_dependency 'thor', '~> 0.18'
-  s.add_dependency 'nokogiri', '~> 1.6'
 end

data/lib/approvals/approval.rb

@@ -19,9 +19,9 @@ module Approvals
     # Add a Proc that tests if subject is a kind of format
     IDENTITIES = {
       hash: Proc.new(){|subject|subject.respond_to? :each_pair},
-      array: Proc.new(){|subject|subject.respond_to? :each_with_index},      
+      array: Proc.new(){|subject|subject.respond_to? :each_with_index},
     }
-    
+
     def identify_format
       IDENTITIES.each_pair do |format, id_test|
         return format if id_test.call(subject)
@@ -34,6 +34,10 @@ module Approvals
       @writer ||= Writer.for(@format)
     end
 
+    def verifier
+      @verifier ||= Verifier.for(@format)
+    end
+
     def verify
       unless File.exist?(namer.output_dir)
         FileUtils.mkdir_p(namer.output_dir)
@@ -63,6 +67,10 @@ module Approvals
     BINARY_FORMATS = [:binary]
 
     def received_matches?
+      return verifier
+        .new(received_path, approved_path)
+        .verify if verifier
+
       if BINARY_FORMATS.include?(@format) # Read without ERB
         IO.read(received_path).chomp == IO.read(approved_path).chomp
       else

data/lib/approvals/filter.rb

@@ -16,18 +16,18 @@ module Approvals
     end
 
     def censored value, key=nil
-      case value
-      when Array
-        value.map { |item| censored(item) }
-      when Hash
-        Hash[value.map { |inner_key, inner_value| [inner_key, censored(inner_value, inner_key)] }]
+      if value.nil?
+        nil
+      elsif key && placeholder_for(key)
+        "<#{placeholder_for(key)}>"
       else
-        if value.nil?
-          nil
-        elsif key && placeholder_for(key)
-          "<#{placeholder_for(key)}>"
-        else
-          value
+        case value
+          when Array
+            value.map { |item| censored(item) }
+          when Hash
+            Hash[value.map { |inner_key, inner_value| [inner_key, censored(inner_value, inner_key)] }]
+          else
+            value
         end
       end
     end

data/lib/approvals/verifier.rb

@@ -0,0 +1,15 @@
+require 'approvals/verifiers/json_verifier'
+
+module Approvals
+  module Verifier
+    REGISTRY = {
+      json: Verifiers::JsonVerifier,
+    }
+
+    class << self
+      def for(format)
+        REGISTRY[format]
+      end
+    end
+  end
+end

data/lib/approvals/verifiers/json_verifier.rb

@@ -0,0 +1,26 @@
+module Approvals
+  module Verifiers
+    class JsonVerifier
+      def initialize(received_path, approved_path)
+        self.received_path = received_path
+        self.approved_path = approved_path
+      end
+
+      def verify
+        approved == received
+      end
+
+      private
+
+      attr_accessor :approved_path, :received_path
+
+      def approved
+        JSON.parse(File.read(approved_path))
+      end
+
+      def received
+        JSON.parse(File.read(received_path))
+      end
+    end
+  end
+end

data/lib/approvals/version.rb

@@ -1,3 +1,3 @@
 module Approvals
-  VERSION = '0.0.22'
+  VERSION = '0.0.26'
 end

data/lib/approvals/writer.rb

@@ -16,15 +16,21 @@ module Approvals
       html: Writers::HtmlWriter.new,
       hash: Writers::HashWriter.new,
       array: Writers::ArrayWriter.new,
+      txt: Writers::TextWriter.new,
     }
 
 
     class << self
       def for(format)
-        if REGISTRY.include?(format)
-          REGISTRY[format]
-        else
-          TextWriter.new
+        begin
+          REGISTRY[format] || Object.const_get(format).new
+        rescue NameError => e
+          error = ApprovalError.new(
+            "Approval Error: #{ e }. Please define a custom writer as outlined"\
+            " in README section 'Customizing formatted output': "\
+            "https://github.com/kytrinyx/approvals#customizing-formatted-output"
+          )
+          raise error
         end
       end
     end

data/lib/approvals.rb

@@ -13,6 +13,7 @@ require 'approvals/executable'
 require 'approvals/reporters'
 require 'approvals/filter'
 require 'approvals/writer'
+require 'approvals/verifier'
 require 'approvals/namers/default_namer'
 
 module Approvals

data/spec/approvals_spec.rb

@@ -51,6 +51,32 @@ describe Approvals do
     Approvals.verify hello, :namer => namer
   end
 
+  context "custom writer" do
+    let(:hello) { Object.new }
+
+    class MyCustomWriter < Approvals::Writers::TextWriter
+      def format(data)
+        filter(data)
+      end
+
+      def filter(data)
+        data.to_s.chars.reject {|c| c =~ /[a-zA-Z0-9]/}
+      end
+    end
+
+    it "verifies a complex object" do
+      Approvals.verify hello, :namer => namer, :format => "MyCustomWriter"
+    end
+
+    it "raises an error with an uninitialized custom writer class" do
+      expect{
+        Approvals.verify hello, :namer => namer, :format => "UninitializedWriter"
+      }.to raise_error.with_message(
+        /Please define a custom writer as outlined in README section 'Customizing formatted output':/
+      )
+    end
+ end
+
   it "verifies html" do
     html = <<-HTML
     <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "DTD/xhtml1-strict.dtd"><html><head><title>Approval</title></head><body><h1>An Approval</h1><p>It has a paragraph</p></body></html>
@@ -59,6 +85,7 @@ describe Approvals do
   end
 
   it "verifies a malformed html fragment" do
+    pending
     html = <<-HTML
 <!DOCTYPE html>
 <html>
@@ -79,6 +106,11 @@ describe Approvals do
     Approvals.verify json, :format => :json, :namer => namer
   end
 
+  it "ignores whitespace differences in json" do
+    hash = { foo: {} }
+
+    Approvals.verify hash, :format => :json, :namer => namer
+  end
 
   it "verifies json and is newline agnostic" do
     json = '{"pet":{"species":"turtle","color":"green","name":"Anthony"}}'

data/spec/filter_spec.rb

@@ -74,6 +74,40 @@ describe Approvals::Filter do
     })
   end
 
+  it "filters array keys" do
+    filter = Approvals::Filter.new({foolist: /^foolist$/})
+    input = {
+      foo: 'bar124',
+      foolist: [{foo: 'bar 145', bar: 'foo'}, 'foobar'],
+      nonfoo: 'bar',
+    }
+
+    output = filter.apply(input)
+
+    expect(output).to eq({
+      foo: 'bar124',
+      foolist: '<foolist>',
+      nonfoo: 'bar',
+    })
+  end
+
+  it "filters hash keys" do
+    filter = Approvals::Filter.new({foohash: /^foohash$/})
+    input = {
+      foo: 'bar124',
+      foohash: {foo: 'bar 145', barlist: ['foo', 'bar']},
+      nonfoo: 'bar',
+    }
+
+    output = filter.apply(input)
+
+    expect(output).to eq({
+      foo: 'bar124',
+      foohash: '<foohash>',
+      nonfoo: 'bar',
+    })
+  end
+
   it "takes the last applicable filter" do
     filter = Approvals::Filter.new({foo: /^foo/, bar: /bar$/})
     input = {

data/spec/fixtures/approvals/approvals_custom_writer_verifies_a_complex_object.approved.txt

@@ -0,0 +1 @@
+["#", "<", ":", ">"]

data/spec/fixtures/approvals/approvals_ignores_whitespace_differences_in_json.approved.json

@@ -0,0 +1,6 @@
+{
+  "foo": {
+
+
+  }
+}

data/spec/fixtures/approvals/approvals_verifies_json_with_a_time_object.approved.json

@@ -0,0 +1,3 @@
+{
+  "foo": "2017-01-01 00:00:00 +0100"
+}

data/spec/fixtures/json_approval_with_different_whitespace/approved.json

@@ -0,0 +1,7 @@
+{
+  "foo": {
+  
+  
+  
+  }
+}

data/spec/fixtures/json_approval_with_different_whitespace/received.json

@@ -0,0 +1,3 @@
+{
+  "foo": {}
+}

data/spec/fixtures/json_approval_with_different_whitespace/received_different_content.json

@@ -0,0 +1,3 @@
+{
+  "bar": 1
+}

data/spec/verifiers/json_verifier_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe Approvals::Verifiers::JsonVerifier do
+  subject(:instance) do
+    described_class.new(received_path, approved_path)
+  end
+
+  context "with same json content but different formatting" do
+    let(:received_path) do
+      "./spec/fixtures/json_approval_with_different_whitespace/received.json"
+    end
+    let(:approved_path) do
+      "./spec/fixtures/json_approval_with_different_whitespace/approved.json"
+    end
+
+    it "passes verification" do
+      expect(instance.verify).to be_truthy
+    end
+  end
+
+  context "with different json content" do
+    let(:received_path) do
+      "./spec/fixtures/json_approval_with_different_whitespace/received_different_content.json"
+    end
+    let(:approved_path) do
+      "./spec/fixtures/json_approval_with_different_whitespace/approved.json"
+    end
+
+    it "does not passe verification" do
+      expect(instance.verify).to be_falsy
+    end
+  end
+end

metadata

@@ -1,37 +1,51 @@
 --- !ruby/object:Gem::Specification
 name: approvals
 version: !ruby/object:Gem::Version
-  version: 0.0.22
+  version: 0.0.26
 platform: ruby
 authors:
 - Katrina Owen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-22 00:00:00.000000000 Z
+date: 2022-06-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: thor
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.1'
-  type: :development
+        version: '1.0'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: json
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.8'
-  type: :development
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -39,34 +53,34 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.8'
 - !ruby/object:Gem::Dependency
-  name: thor
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
-  type: :runtime
+        version: '13.0'
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
-  name: nokogiri
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.6'
-  type: :runtime
+        version: '3.1'
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.6'
-description: Approval Tests for Ruby
+        version: '3.1'
+description: A library to make it easier to do golden-master style testing in Ruby
 email:
 - katrina.owen@gmail.com
 executables:
@@ -112,6 +126,8 @@ files:
 - lib/approvals/rspec.rb
 - lib/approvals/scrubber.rb
 - lib/approvals/system_command.rb
+- lib/approvals/verifier.rb
+- lib/approvals/verifiers/json_verifier.rb
 - lib/approvals/version.rb
 - lib/approvals/writer.rb
 - lib/approvals/writers/array_writer.rb
@@ -127,6 +143,8 @@ files:
 - spec/executable_spec.rb
 - spec/extensions/rspec_approvals_spec.rb
 - spec/filter_spec.rb
+- spec/fixtures/approvals/approvals_custom_writer_verifies_a_complex_object.approved.txt
+- spec/fixtures/approvals/approvals_ignores_whitespace_differences_in_json.approved.json
 - spec/fixtures/approvals/approvals_passes_approved_files_through_erb.approved.txt
 - spec/fixtures/approvals/approvals_passes_the_received_files_through_erb.approved.txt
 - spec/fixtures/approvals/approvals_supports_excluded_keys_option_also_supports_an_array_of_hashes.approved.json
@@ -143,6 +161,7 @@ files:
 - spec/fixtures/approvals/approvals_verifies_html.approved.html
 - spec/fixtures/approvals/approvals_verifies_json.approved.json
 - spec/fixtures/approvals/approvals_verifies_json_and_is_newline_agnostic.approved.json
+- spec/fixtures/approvals/approvals_verifies_json_with_a_time_object.approved.json
 - spec/fixtures/approvals/approvals_verifies_xml.approved.xml
 - spec/fixtures/approvals/verifications_a_string.approved.txt
 - spec/fixtures/approvals/verifies_a_complex_object.approved.txt
@@ -163,6 +182,9 @@ files:
 - spec/fixtures/approvals/verifies_html.approved.html
 - spec/fixtures/approvals/verifies_json.approved.json
 - spec/fixtures/approvals/verifies_xml.approved.xml
+- spec/fixtures/json_approval_with_different_whitespace/approved.json
+- spec/fixtures/json_approval_with_different_whitespace/received.json
+- spec/fixtures/json_approval_with_different_whitespace/received_different_content.json
 - spec/fixtures/one.png
 - spec/fixtures/one.txt
 - spec/fixtures/two.png
@@ -180,7 +202,8 @@ files:
 - spec/scrubber_spec.rb
 - spec/spec_helper.rb
 - spec/system_command_spec.rb
-homepage: ''
+- spec/verifiers/json_verifier_spec.rb
+homepage: https://github.com/kytrinyx/approvals
 licenses:
 - MIT
 metadata: {}
@@ -199,10 +222,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: approvals
-rubygems_version: 2.2.2
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Approval Tests for Ruby
 test_files: []
-has_rdoc: