Allow save & save! without performing validations

Fixes #17

Change-Id: I2dad010c8739bf7b0e81b02f449877edaa26a16b
Reviewed-on: http://review.couchbase.org/41272
Reviewed-by: Sergey Avseyev <sergey.avseyev@gmail.com>
Tested-by: Sergey Avseyev <sergey.avseyev@gmail.com>

lib/couchbase/model.rb

@@ -520,9 +520,8 @@ def initialize(attrs = {})
     #   p.create
     def create(options = {})
       @id ||= Couchbase::Model::UUID.generator.next(1, model.thread_storage[:uuid_algorithm])
-      if respond_to?(:valid?) && !valid?
-        return false
-      end
+      return false if failed_validations?(options)
+
       options = model.defaults.merge(options)
       value = (options[:format] == :plain) ?  @raw : attributes_with_values
       unless @meta
@@ -567,9 +566,8 @@ def create!(options = {})
     #
     def save(options = {})
       return create(options) unless @meta
-      if respond_to?(:valid?) && !valid?
-        return false
-      end
+      return false if failed_validations?(options)
+
       options = model.defaults.merge(options)
       value = (options[:format] == :plain) ?  @raw : attributes_with_values
       @meta['cas'] = model.bucket.replace(@id, value, options)
@@ -842,6 +840,14 @@ def attributes_with_values
 
     private :attributes_with_values
 
+    # @private Returns if validations can be and should be performed,
+    #          and they fail
+    #
+    # @since 0.5.5
+    def failed_validations?(options)
+      options[:validate] != false && respond_to?(:valid?) && !valid?
+    end
+
     # Redefine (if exists) #to_key to use #key if #id is missing
     def to_key
       keys = [id || key]

test/test_active_model_integration.rb

@@ -143,4 +143,17 @@ def test_dirty_tracking_previous_values
     assert_equal tester.previous_changes[:email], ['joe@example.com', 'bob@example.com']
   end
 
+  def test_save_without_validations
+    tester = ActiveUser.new(:email => 'joe@example.com', :role => nil)
+    assert tester.save(:validate => false), 'Validations not skipped'
+  end
+
+  def test_save_bang_without_validations
+    tester = ActiveUser.new(:email => 'joe@example.com', :role => nil)
+    begin
+      tester.save!(:validate => false)
+    rescue
+      assert false, 'Validations not skipped'
+    end
+  end
 end