simple_approvals-rspec 1.1.0 → 3.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: c55a27cedbf93964a2c84516d1f8d1ba26db78c2
-  data.tar.gz: 6f52c988b080e2734acefc9b36584cc12770960a
+SHA256:
+  metadata.gz: f71df43a87b07fe15b71e6cc0c24c3482c2aff226ff36714be8a988d919edfd2
+  data.tar.gz: 3cfe824a91b39e426f1e101616a558c787f4104e24e8117b1707112e5f1a923c
 SHA512:
-  metadata.gz: 7816895a44b23c16fd49ef9930ea78c706afa24f7053c0daa93728fc87e42ddb4450e853b9a449b4ca76614cb29f0f4a6666a0019fab875ea8ad3c1bb020604a
-  data.tar.gz: a847e0f5072b1988d63770737b8b1fb09d8bc607f8f836051b2c040e573c04763295dee4dd83eaa2b1b1a79b40e054a2a98f59b4f539b8f50a65379145e590d9
+  metadata.gz: 25605c30b71b93bab1ededdc8ade9f8fbfefaa64c513eee4d0972792687fb399fe68cbb3287f76602bbe6b8146ac84af08be0fc11f2cd02828d87ac18a069359
+  data.tar.gz: 817c7d0811dfeea3e111ed9d99dd45149bb41ba72305666dff71c26d05d31c3f0a7e52f8d3eaa5cede99154b670b470ea44d12670bc36daca903e1678b24689d

data/lib/simple_approvals/rspec/approvals.rb

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 # Top-level class to provide approval test support (inspired by http://approvaltests.com/)
 class Approvals
   class << self
@@ -7,33 +5,33 @@ class Approvals
       scrubbed_content = apply_optional_scrubber(rendered_content, options)
       pre_approve_if_asked(approved_path, scrubbed_content, options)
 
-      expected_content = load_expected_content(approved_path, **options)
+      expected_content = load_expected_content(approved_path, options)
       received_path = received_path_for approved_path
 
       if scrubbed_content.strip.empty?
-        raise_verify_error_with_diff(approved_path, received_path, scrubbed_content, **options) unless options[:allow_empty]
+        raise_verify_error_with_diff(approved_path, received_path, scrubbed_content, options) unless options[:allow_empty]
       elsif scrubbed_content.strip == expected_content.strip
-        handle_match(received_path, scrubbed_content, **options)
+        handle_match(received_path, scrubbed_content, options)
       else
-        raise_verify_error_with_diff(approved_path, received_path, scrubbed_content, **options)
+        raise_verify_error_with_diff(approved_path, received_path, scrubbed_content, options)
       end
     end
 
     private
 
-    def apply_optional_scrubber(content, **options)
-      if options.key?(:scrubber) && options[:scrubber]
+    def apply_optional_scrubber(content, options)
+      if options.has_key?(:scrubber) && options[:scrubber]
         options[:scrubber].call(content)
       else
         content
       end
     end
 
-    def pre_approve_if_asked(approved_path, scrubbed_content, **options)
+    def pre_approve_if_asked(approved_path, scrubbed_content, options)
       File.write(approved_path, scrubbed_content) if options[:approve_all] || options[:accept_all] || ENV['APPROVE_ALL']
     end
 
-    def handle_match(received_path, received_content, **options)
+    def handle_match(received_path, received_content, options)
       if options[:keep_received_file] == true
         File.write(received_path, received_content)
       elsif File.file?(received_path)
@@ -45,7 +43,7 @@ class Approvals
       "#{approved_path}.received"
     end
 
-    def load_expected_content(approved_path, **options)
+    def load_expected_content(approved_path, options)
       if File.file? approved_path
         File.read(approved_path)
       else
@@ -54,7 +52,7 @@ class Approvals
       end
     end
 
-    def raise_verify_error_with_diff(approved_path, received_path, received_content, **options)
+    def raise_verify_error_with_diff(approved_path, received_path, received_content, options)
       File.write(received_path, received_content)
 
       # and generate a diff for the console
@@ -63,8 +61,18 @@ class Approvals
       message << %(- writing received content to: "#{received_path}")
       message << "\n"
 
-      expected_content = load_expected_content(approved_path)
-      ::RSpec::Expectations.fail_with message, expected_content, received_content
+      expected_content = load_expected_content(approved_path, options)
+
+      if !expected_content.include?("\n") || !received_content.include?("\n")
+        # work around a bug in rspec-support 3.12.0's differ
+        # where it shows nothing when both strings are single line
+        # https://github.com/rspec/rspec-support/blob/v3.12.0/lib/rspec/support/differ.rb#L16-L18
+        message << "expected: #{expected_content.inspect}\n"
+        message << "received: #{received_content.inspect}\n"
+        ::RSpec::Expectations.fail_with message
+      else
+        ::RSpec::Expectations.fail_with message, expected_content, received_content
+      end
     end
   end
 end

data/lib/simple_approvals/rspec/matchers.rb

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 require 'rspec/expectations'
 
 RSpec::Matchers.define :match_hash do |expected|

data/lib/simple_approvals/rspec.rb

@@ -1,4 +1,2 @@
-# frozen_string_literal: true
-
 require 'simple_approvals/rspec/approvals'
 require 'simple_approvals/rspec/matchers'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: simple_approvals-rspec
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 3.1.2
 platform: ruby
 authors:
 - David Alpert
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-08-22 00:00:00.000000000 Z
+date: 2023-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -122,24 +122,23 @@ files:
 homepage: https://github.com/davidalpert/simple_approvals-rspec/
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - "~>"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 3.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.11
-signing_key: 
+rubygems_version: 3.3.7
+signing_key:
 specification_version: 4
 summary: ''
 test_files: []