simple_approvals-rspec 1.1.1 → 3.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 8c20a53cf7f893c8d8f0f9a58e2a64e6a5a682bb
-  data.tar.gz: 0ed1fd3bb0196f4eda2706f46bcb15872b1c0e33
+SHA256:
+  metadata.gz: 58062f9d2760edfd82719892ff5cb7b859ddee062d60fe169269721f21fb12d8
+  data.tar.gz: 0e0ed4b8ccd440ffb9f829e1d90b9fbe3ef52fbca000b5b55eb5a152c354fa05
 SHA512:
-  metadata.gz: dd650c025baabbf738ba6387850104387666db9488d8e0a57c784daebfcd1436a9e50647698cbacf4498bfa9d346e8e78dd9584d50344c7e4e28d39dfc26d007
-  data.tar.gz: 49c001062ba5562c699b57e021b25516166b25004b2a09c7752568df45421880bbb5f019d8b7dd6bc5d324fa4d4258d227a224e3630161b48b9a63140e03d66c
+  metadata.gz: 8b1cd64285868af7604dbe14e5bbfc777987c2212617cbaa4329df897b46d77f716e4b8f0a3d77ecdcd540a4f72b29f5b6b2157f39afea1311fe620bb2c90ea7
+  data.tar.gz: 4f7547e83ae43ff9b8af4cad2494481358d807b96eb7a7812fd9698da983f69bffac43687a51e845889d7e041f396bbd3fe60295eeb71ce375abbf4ab78f32ed

data/lib/simple_approvals/rspec/approvals.rb

@@ -5,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)
@@ -43,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
@@ -52,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
@@ -61,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")
+        ::RSpec::Expectations.fail_with message, expected_content, received_content
+      else
+        # 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
+      end
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: simple_approvals-rspec
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 3.1.3
 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: []