shoulda-matchers 4.4.1 → 4.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4f12034724c2954cf58fc7e89723ce42119e3a6f39676899c0510545a61f5fee
-  data.tar.gz: 0c5cd62168c0dd00fa3246c9afe2ee4499db8c37312ff6f0f8ee10afb16f0f42
+  metadata.gz: '0268da0cff119fb6c9128b70f1826451b582bdc2495acf382f4e512a6636d0c8'
+  data.tar.gz: bf0698ed7605d8a8ef6b7e78bdc198bfa9502e1f2df134e3c5f524d1809b82da
 SHA512:
-  metadata.gz: 3e33ed5061a6913d3b448f2f251e79a7618c9226275a308bf5a0547a81dddb4fbaf17a8834ddf3a1e3662cc712afaf6994bb9ea44a727adcdffbd41653ced13c
-  data.tar.gz: e7ca6969c599eeb2fa8a084a4486085b6156760d0a77a1466d3989c0a9f15b7b092d7cdb7f4e2bbdf1befd5437afbd69121316e2ba6d9b271c809e3abfad82d1
+  metadata.gz: 407cc79fc367dd4f02b346a199a8d9a9c4ecc5f2d814b98b25989489d96fd2d02308459e2d3a086aeacd67cbd5a9e91e7e237f3fe04efb64c196307dd53abe71
+  data.tar.gz: a4eb04356408c22c7606ef403e1d4a042461e5323a9459875ad6a67f9f5cbc2944dee295d6086e6f1bea5b1d4fd7cace0fc5b701a4ba0199c75f7f002939f47a

data/LICENSE

@@ -0,0 +1,22 @@
+Copyright (c) 2006-2021 Tammer Saleh and thoughtbot, inc.
+
+Permission is hereby granted, free of charge, to any person
+obtaining a copy of this software and associated documentation
+files (the "Software"), to deal in the Software without
+restriction, including without limitation the rights to use,
+copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the
+Software is furnished to do so, subject to the following
+conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+OTHER DEALINGS IN THE SOFTWARE.

data/README.md

@@ -1,4 +1,4 @@
-# Shoulda Matchers [![Gem Version][version-badge]][rubygems] [![Build Status][travis-badge]][travis] [![Total Downloads][downloads-total]][rubygems] [![Downloads][downloads-badge]][rubygems] [![Hound][hound-badge]][hound]
+# Shoulda Matchers [![Gem Version][version-badge]][rubygems] [![Build Status][travis-badge]][travis] [![Total Downloads][downloads-total]][rubygems] [![Downloads][downloads-badge]][rubygems]
 
 [version-badge]: https://img.shields.io/gem/v/shoulda-matchers.svg
 [rubygems]: https://rubygems.org/gems/shoulda-matchers
@@ -7,8 +7,6 @@
 [downloads-total]: https://img.shields.io/gem/dt/shoulda-matchers.svg
 [downloads-badge]: https://img.shields.io/gem/dtv/shoulda-matchers.svg
 [downloads-badge]: https://img.shields.io/gem/dtv/shoulda-matchers.svg
-[hound-badge]: https://img.shields.io/badge/Reviewed_by-Hound-8E64B0.svg
-[hound]: https://houndci.com
 
 [![shoulda-matchers][logo]][website]
 
@@ -24,7 +22,7 @@ complex, and error-prone.
 📖 **[Read the documentation for the latest version][rubydocs].**  
 📢 **[See what's changed in recent versions][changelog].**
 
-[rubydocs]: http://matchers.shoulda.io/docs
+[rubydocs]: https://matchers.shoulda.io/docs
 [changelog]: CHANGELOG.md
 
 ## Table of contents
@@ -385,7 +383,7 @@ about any of them, make sure to [consult the documentation][rubydocs]!
   tests your `belongs_to` associations.
 * **[define_enum_for](lib/shoulda/matchers/active_record/define_enum_for_matcher.rb)**
   tests usage of the `enum` macro.
-* **[have_and_belong_to_many](lib/shoulda/matchers/active_record/association_matcher.rb)**
+* **[have_and_belong_to_many](lib/shoulda/matchers/active_record/association_matcher.rb#L827)**
   tests your `has_and_belongs_to_many` associations.
 * **[have_db_column](lib/shoulda/matchers/active_record/have_db_column_matcher.rb)**
   tests that the table that backs your model has a specific column.
@@ -393,11 +391,11 @@ about any of them, make sure to [consult the documentation][rubydocs]!
   tests that the table that backs your model has an index on a specific column.
 * **[have_implicit_order_column](lib/shoulda/matchers/active_record/have_implicit_order_column.rb)**
   tests usage of `implicit_order_column`.
-* **[have_many](lib/shoulda/matchers/active_record/association_matcher.rb)**
+* **[have_many](lib/shoulda/matchers/active_record/association_matcher.rb#L328)**
   tests your `has_many` associations.
 * **[have_many_attached](lib/shoulda/matchers/active_record/have_attached_matcher.rb)**
   tests your `has_many_attached` associations.
-* **[have_one](lib/shoulda/matchers/active_record/association_matcher.rb)**
+* **[have_one](lib/shoulda/matchers/active_record/association_matcher.rb#L598)**
   tests your `has_one` associations.
 * **[have_one_attached](lib/shoulda/matchers/active_record/have_attached_matcher.rb)**
   tests your `has_one_attached` associations.
@@ -480,7 +478,7 @@ For Ruby < 2.4 and Rails < 4.1 compatibility, please use [v3.1.3][v3.1.3].
 ## Versioning
 
 Shoulda Matchers follows Semantic Versioning 2.0 as defined at
-<http://semver.org>.
+<https://semver.org>.
 
 ## Team
 
@@ -492,7 +490,7 @@ Albuk][guialbuk].
 
 ## Copyright/License
 
-Shoulda Matchers is copyright © 2006-2020 Tammer Saleh and [thoughtbot,
+Shoulda Matchers is copyright © 2006-2021 Tammer Saleh and [thoughtbot,
 inc][thoughtbot-website]. It is free and opensource software and may be
 redistributed under the terms specified in the [LICENSE](LICENSE) file.
 

data/lib/shoulda/matchers/action_controller/callback_matcher.rb

@@ -179,11 +179,13 @@ module Shoulda
         end
 
         def failure_message
-          "Expected that #{controller_class.name} would have :#{method_name} as a #{kind}_#{callback_type}"
+          "Expected that #{controller_class.name} would have :#{method_name}"\
+            " as a #{kind}_#{callback_type}"
         end
 
         def failure_message_when_negated
-          "Expected that #{controller_class.name} would not have :#{method_name} as a #{kind}_#{callback_type}"
+          "Expected that #{controller_class.name} would not have"\
+            " :#{method_name} as a #{kind}_#{callback_type}"
         end
 
         def description

data/lib/shoulda/matchers/action_controller/filter_param_matcher.rb

@@ -31,12 +31,13 @@ module Shoulda
           @key = key
         end
 
-        def matches?(controller)
+        def matches?(_controller)
           filters_key?
         end
 
         def failure_message
-          "Expected #{@key} to be filtered; filtered keys: #{filtered_keys.join(', ')}"
+          "Expected #{@key} to be filtered; filtered keys:"\
+            " #{filtered_keys.join(', ')}"
         end
 
         def failure_message_when_negated

data/lib/shoulda/matchers/action_controller/permit_matcher.rb

@@ -209,7 +209,8 @@ module Shoulda
         attr_writer :stubbed_params
 
         def initialize(expected_permitted_parameter_names)
-          @expected_permitted_parameter_names = expected_permitted_parameter_names
+          @expected_permitted_parameter_names =
+            expected_permitted_parameter_names
           @action = nil
           @verb = nil
           @request_params = {}
@@ -262,18 +263,20 @@ module Shoulda
         end
 
         def failure_message
-          "Expected #{verb.upcase} ##{action} to #{expectation},\nbut #{reality}."
+          "Expected #{verb.upcase} ##{action} to #{expectation},"\
+          "\nbut #{reality}."
         end
 
         def failure_message_when_negated
-          "Expected #{verb.upcase} ##{action} not to #{expectation},\nbut it did."
+          "Expected #{verb.upcase} ##{action} not to #{expectation},"\
+          "\nbut it did."
         end
 
         protected
 
-        attr_reader :controller, :double_collections_by_parameter_name, :action, :verb,
-          :request_params, :expected_permitted_parameter_names, :context, :subparameter_name,
-          :parameters_double_registry
+        attr_reader :controller, :double_collections_by_parameter_name, :action,
+          :verb, :request_params, :expected_permitted_parameter_names,
+          :context, :subparameter_name, :parameters_double_registry
 
         def expectation
           message = 'restrict parameters '
@@ -282,7 +285,8 @@ module Shoulda
             message << "on #{subparameter_name.inspect} "
           end
 
-          message << 'to ' + format_parameter_names(expected_permitted_parameter_names)
+          message << 'to '\
+            "#{format_parameter_names(expected_permitted_parameter_names)}"
 
           message
         end
@@ -291,9 +295,9 @@ module Shoulda
           if actual_permitted_parameter_names.empty?
             'it did not restrict any parameters'
           else
-            'the restricted parameters were ' +
-              format_parameter_names(actual_permitted_parameter_names) +
-              ' instead'
+            'the restricted parameters were '\
+            "#{format_parameter_names(actual_permitted_parameter_names)}"\
+            ' instead'
           end
         end
 
@@ -303,12 +307,12 @@ module Shoulda
 
         def actual_permitted_parameter_names
           @_actual_permitted_parameter_names ||= begin
-            if subparameter_name
-              options = { for: subparameter_name }
-            else
-              options = {}
-            end
-
+            options =
+              if subparameter_name
+                { for: subparameter_name }
+              else
+                {}
+              end
             parameters_double_registry.permitted_parameter_names(options)
           end
         end
@@ -329,8 +333,8 @@ module Shoulda
 
         def default_verb
           case action
-            when :create then :post
-            when :update then RailsShim.verb_for_update
+          when :create then :post
+          when :update then RailsShim.verb_for_update
           end
         end
 
@@ -346,7 +350,7 @@ module Shoulda
 
           def register
             double_collection = Doublespeak.double_collection_for(
-              ::ActionController::Parameters.singleton_class
+              ::ActionController::Parameters.singleton_class,
             )
             double_collection.register_proxy(:new).to_return do |call|
               params = call.return_value
@@ -389,7 +393,7 @@ module Shoulda
 
             if double_collections_by_parameter_name.key?(subparameter_name)
               self.class.permitted_parameter_names_within(
-                double_collections_by_parameter_name[subparameter_name]
+                double_collections_by_parameter_name[subparameter_name],
               )
             else
               []
@@ -436,7 +440,8 @@ module Shoulda
         # @private
         class VerbNotDefinedError < StandardError
           def message
-            'You must specify an HTTP verb when using a non-RESTful action. For example: for(:authorize, verb: :post)'
+            'You must specify an HTTP verb when using a non-RESTful action.'\
+            ' For example: for(:authorize, verb: :post)'
           end
         end
       end

data/lib/shoulda/matchers/action_controller/redirect_to_matcher.rb

@@ -75,14 +75,12 @@ module Shoulda
         private
 
         def redirects_to_url?
-          begin
-            @context.__send__(:assert_redirected_to, url)
-            @failure_message_when_negated = "Didn't expect to redirect to #{url}"
-            true
-          rescue Shoulda::Matchers.assertion_exception_class => error
-            @failure_message = error.message
-            false
-          end
+          @context.__send__(:assert_redirected_to, url)
+          @failure_message_when_negated = "Didn't expect to redirect to #{url}"
+          true
+        rescue Shoulda::Matchers.assertion_exception_class => e
+          @failure_message = e.message
+          false
         end
 
         def url

data/lib/shoulda/matchers/action_controller/render_template_matcher.rb

@@ -71,14 +71,12 @@ module Shoulda
         private
 
         def renders_template?
-          begin
-            @context.__send__(:assert_template, @options, @message)
-            @failure_message_when_negated = "Didn't expect to render #{@template}"
-            true
-          rescue Shoulda::Matchers.assertion_exception_class => error
-            @failure_message = error.message
-            false
-          end
+          @context.__send__(:assert_template, @options, @message)
+          @failure_message_when_negated = "Didn't expect to render #{@template}"
+          true
+        rescue Shoulda::Matchers.assertion_exception_class => e
+          @failure_message = e.message
+          false
         end
       end
     end

data/lib/shoulda/matchers/action_controller/render_with_layout_matcher.rb

@@ -64,12 +64,12 @@ module Shoulda
       # @private
       class RenderWithLayoutMatcher
         def initialize(expected_layout)
-          if expected_layout
-            @expected_layout = expected_layout.to_s
-          else
-            @expected_layout = nil
-          end
-
+          @expected_layout =
+            if expected_layout
+              expected_layout.to_s
+            else
+              nil
+            end
           @controller = nil
         end
 
@@ -95,11 +95,12 @@ module Shoulda
 
         def description
           description = 'render with '
-          if @expected_layout.nil?
-            description << 'a layout'
-          else
-            description << "the #{@expected_layout.inspect} layout"
-          end
+          description <<
+            if @expected_layout.nil?
+              'a layout'
+            else
+              "the #{@expected_layout.inspect} layout"
+            end
           description
         end
 
@@ -118,7 +119,9 @@ module Shoulda
         end
 
         def rendered_layouts
-          recorded_layouts.keys.compact.map { |layout| layout.sub(%r{^layouts/}, '') }
+          recorded_layouts.keys.compact.map { |layout|
+            layout.sub(%r{^layouts/}, '')
+          }
         end
 
         def recorded_layouts
@@ -135,7 +138,7 @@ module Shoulda
 
         def result
           if rendered_with_layout?
-            'rendered with ' + rendered_layouts.map(&:inspect).join(', ')
+            "rendered with #{rendered_layouts.map(&:inspect).join(', ')}"
           else
             'rendered without a layout'
           end

data/lib/shoulda/matchers/action_controller/rescue_from_matcher.rb

@@ -79,7 +79,8 @@ module Shoulda
           end
 
           unless handler_exists?
-            expectation << " but #{controller} does not respond to #{expected_method}"
+            expectation << " but #{controller} does not respond to"\
+              " #{expected_method}"
           end
           expectation
         end

data/lib/shoulda/matchers/action_controller/route_matcher.rb

@@ -177,13 +177,12 @@ module Shoulda
 
         def add_port_to_path(path, port)
           if port
-            "http://example.com:#{port}" + path
+            "https://example.com:#{port}" + path
           else
             path
           end
         end
 
-
         def guess_controller_if_necessary(controller)
           params[:controller] ||= controller.controller_path
         end
@@ -195,11 +194,11 @@ module Shoulda
             params,
           )
           true
-        rescue ::ActionController::RoutingError => error
-          @failure_message = error.message
+        rescue ::ActionController::RoutingError => e
+          @failure_message = e.message
           false
-        rescue Shoulda::Matchers.assertion_exception_class => error
-          @failure_message = error.message
+        rescue Shoulda::Matchers.assertion_exception_class => e
+          @failure_message = e.message
           false
         end
       end

data/lib/shoulda/matchers/action_controller/route_params.rb

@@ -3,7 +3,7 @@ module Shoulda
     module ActionController
       # @private
       class RouteParams
-        PARAMS_TO_SYMBOLIZE = %i{ format }
+        PARAMS_TO_SYMBOLIZE = %i{format}.freeze
 
         def initialize(args)
           @args = args

data/lib/shoulda/matchers/action_controller/set_session_or_flash_matcher.rb

@@ -20,7 +20,9 @@ module Shoulda
         def to(expected_value = nil, &block)
           if block
             unless context_set?
-              message = "When specifying a value as a block, a context must be specified beforehand, e.g., #{store.name}.in_context(context).to { ... }"
+              message = 'When specifying a value as a block, a context must be'\
+                ' specified beforehand,'\
+                " e.g., #{store.name}.in_context(context).to { ... }"
               raise ArgumentError, message
             end
 
@@ -42,12 +44,14 @@ module Shoulda
         end
 
         def failure_message
-          "Expected #{controller.class} to #{expectation_description}, but it did not"
+          "Expected #{controller.class} to #{expectation_description},"\
+          ' but it did not'
         end
         alias_method :failure_message_for_should, :failure_message
 
         def failure_message_when_negated
-          "Expected #{controller.class} not to #{expectation_description}, but it did"
+          "Expected #{controller.class} not to #{expectation_description},"\
+          ' but it did'
         end
         alias_method :failure_message_for_should_not,
           :failure_message_when_negated
@@ -81,18 +85,20 @@ module Shoulda
         def expectation_description
           string = 'set'
 
-          if key_set?
-            string << " #{store.name}[#{key.inspect}]"
-          else
-            string << " any key in #{store.name}"
-          end
-
-          if expected_value_set?
-            if expected_value.is_a?(Regexp)
-              string << " to a value matching #{expected_value.inspect}"
+          string <<
+            if key_set?
+              " #{store.name}[#{key.inspect}]"
             else
-              string << " to #{expected_value.inspect}"
+              " any key in #{store.name}"
             end
+
+          if expected_value_set?
+            string <<
+              if expected_value.is_a?(Regexp)
+                " to a value matching #{expected_value.inspect}"
+              else
+                " to #{expected_value.inspect}"
+              end
           end
 
           string