[MODEL] Added the :validate option to the save method for models

    article.save validate: false

Author: karmi

elasticsearch-persistence/lib/elasticsearch/persistence/model/store.rb

@@ -44,7 +44,10 @@ module InstanceMethods
           # @return [Hash,FalseClass] The Elasticsearch response as a Hash or `false`
           #
           def save(options={})
-            return false unless valid?
+            unless options.delete(:validate) == false
+              return false unless valid?
+            end
+
             run_callbacks :save do
               options.update id: self.id
               options.update index: self._index if self._index

elasticsearch-persistence/test/integration/model/model_basic_test.rb

@@ -51,7 +51,7 @@ class ::Person
 
         should "save and find the object" do
           person = Person.new name: 'John Smith', birthday: Date.parse('1970-01-01')
-          person.save
+          assert person.save
 
           assert_not_nil person.id
           document = Person.find(person.id)
@@ -63,6 +63,20 @@ class ::Person
           assert_not_nil Elasticsearch::Persistence.client.get index: 'people', type: 'person', id: person.id
         end
 
+        should "not save an invalid object" do
+          person = Person.new name: nil
+          assert ! person.save
+        end
+
+        should "save an invalid object with the :validate option" do
+          person = Person.new name: nil, salary: 100
+          assert person.save validate: false
+
+          assert_not_nil person.id
+          document = Person.find(person.id)
+          assert_equal 100, document.salary
+        end
+
         should "delete the object" do
           person = Person.create name: 'John Smith', birthday: Date.parse('1970-01-01')
 

elasticsearch-persistence/test/unit/model_store_test.rb

@@ -80,10 +80,7 @@ class DummyStoreModel
           end
           .returns({'_id' => 'abc123'})
 
-        assert ! subject.persisted?
-
         assert subject.save
-        assert subject.persisted?
       end
 
       should "save the model and set the ID" do
@@ -108,6 +105,37 @@ class DummyStoreModel
         assert_equal Time.parse('2014-01-01T00:00:00Z'), subject.updated_at
       end
 
+      should "not save an invalid model" do
+        @gateway
+          .expects(:save)
+          .never
+
+        subject.instance_eval do
+          def valid?; false; end;
+        end
+
+        assert ! subject.save
+        assert ! subject.persisted?
+      end
+
+      should "skip the validation with the :validate option" do
+        @gateway
+          .expects(:save)
+          .with do |object, options|
+            assert_equal subject, object
+            assert_equal nil, options[:id]
+            true
+          end
+          .returns({'_id' => 'abc123'})
+
+        subject.instance_eval do
+          def valid?; false; end;
+        end
+
+        assert subject.save validate: false
+        assert subject.persisted?
+      end
+
       should "pass the options to gateway" do
         @gateway
           .expects(:save)