Merge pull request #120 from brianp/master

NoMethodError fix

lib/recurly/resource.rb

@@ -765,8 +765,8 @@ def save!
     #   account.save   # => true
     #   account.valid? # => true
     def valid?
-      return true if persisted? && changed_attributes.empty?
-      return if errors.empty? && changed_attributes?
+      return true if persisted? && !changed?
+      return if errors.empty? && changed?
       errors.empty?
     end
 

spec/recurly/resource_spec.rb

@@ -453,5 +453,33 @@ def record.to_param() 1 end
         proc { record.destroy }.must_raise Resource::NotFound
       end
     end
+
+    describe "#valid?" do
+      it "must return true if persisted without changes" do
+        record.persist!
+        record.valid?.must_equal true
+      end
+
+      it "must return true if not persisted without changes and no errors" do
+        record.valid?.must_equal true
+      end
+
+      it "must return nil if persisted with changes" do
+        record.persist!
+        record[:uuid] = 'changed'
+        record.valid?.must_equal nil
+      end
+
+      it "must return nil if not persisted with changes and no errors" do
+        record[:uuid] = 'changed'
+        record.valid?.must_equal nil
+      end
+
+      it "must return false if it has errors" do
+        record.errors[:name] = 'an error'
+        record.valid?.must_equal false
+      end
+    end
+
   end
 end