shoulda-matchers 1.5.5 → 1.5.6

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    shoulda-matchers (1.5.5)
+    shoulda-matchers (1.5.6)
       activesupport (>= 3.0.0)
       bourne (~> 1.3)
 

data/NEWS.md

@@ -1,5 +1,9 @@
 # HEAD
 
+# v 1.5.6
+* Revert previous change in `AllowValueMatcher` that added a check for a
+properly-set attribute.
+
 # v 1.5.5
 * `AllowValueMatcher` checks that the right value is used for attempts at
 setting the attribute with it

data/gemfiles/3.0.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/melissaxie/thoughtbot/shoulda-matchers
   specs:
-    shoulda-matchers (1.5.5)
+    shoulda-matchers (1.5.6)
       activesupport (>= 3.0.0)
       bourne (~> 1.3)
 

data/gemfiles/3.1.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/melissaxie/thoughtbot/shoulda-matchers
   specs:
-    shoulda-matchers (1.5.5)
+    shoulda-matchers (1.5.6)
       activesupport (>= 3.0.0)
       bourne (~> 1.3)
 

data/gemfiles/3.2.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/melissaxie/thoughtbot/shoulda-matchers
   specs:
-    shoulda-matchers (1.5.5)
+    shoulda-matchers (1.5.6)
       activesupport (>= 3.0.0)
       bourne (~> 1.3)
 

data/lib/shoulda/matchers/active_model/allow_value_matcher.rb

@@ -56,9 +56,10 @@ module Shoulda # :nodoc:
         def matches?(instance)
           self.instance = instance
 
-          values_to_match.all? do |current_value|
-            set_attribute_on_instance(current_value)
-            matches_attribute_value?(current_value) && errors_do_not_match?
+          values_to_match.none? do |value|
+            self.value = value
+            instance.send("#{attribute}=", value)
+            errors_match?
           end
         end
 
@@ -79,21 +80,12 @@ module Shoulda # :nodoc:
         attr_accessor :values_to_match, :message_finder_factory,
           :instance, :attribute, :value, :matched_error
 
-        def set_attribute_on_instance(current_value)
-          self.value = current_value
-          instance.send("#{attribute}=", current_value)
+        def errors_match?
+          has_messages? && errors_for_attribute_match?
         end
 
-        def matches_attribute_value?(current_value)
-          instance.send(attribute.to_sym) == current_value
-        end
-
-        def errors_do_not_match?
-          has_no_messages? || !errors_for_attribute_match?
-        end
-
-        def has_no_messages?
-          !message_finder.has_messages?
+        def has_messages?
+          message_finder.has_messages?
         end
 
         def errors_for_attribute_match?

data/lib/shoulda/matchers/version.rb

@@ -1,5 +1,5 @@
 module Shoulda
   module Matchers
-    VERSION = '1.5.5'.freeze
+    VERSION = '1.5.6'.freeze
   end
 end

data/spec/shoulda/matchers/active_model/ensure_inclusion_of_matcher_spec.rb

@@ -79,13 +79,6 @@ describe Shoulda::Matchers::ActiveModel::EnsureInclusionOfMatcher do
   end
 
   context 'an attribute which must be included in an array' do
-    context 'given an attribute that does not accept strings' do
-      it 'allows an attribute to be set as an integer' do
-        validating_inclusion(:in => [0,1,2], :column_type => :integer).
-          should ensure_inclusion_of(:attr).in_array([0,1,2])
-      end
-    end
-
     it 'accepts with correct array' do
       validating_inclusion(:in => %w(one two)).
         should ensure_inclusion_of(:attr).in_array(%w(one two))
@@ -172,9 +165,7 @@ describe Shoulda::Matchers::ActiveModel::EnsureInclusionOfMatcher do
 end
 
 def validating_inclusion(options)
-  options[:column_type] ||= :string
-
-  define_model(:example, :attr => options[:column_type]) do
+  define_model(:example, :attr => :string) do
     validates_inclusion_of :attr, options
   end.new
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: shoulda-matchers
 version: !ruby/object:Gem::Version
-  version: 1.5.5
+  version: 1.5.6
   prerelease: 
 platform: ruby
 authors:
@@ -14,7 +14,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-28 00:00:00.000000000 Z
+date: 2013-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -309,7 +309,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -727795483328027229
+      hash: 1811541480195488740
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -318,7 +318,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -727795483328027229
+      hash: 1811541480195488740
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.25