Merge pull request #96 from chiperific/95_undiscard_return_false

#95 Fix `undiscard` returns false unless discarded?

lib/discard/model.rb

@@ -138,7 +138,7 @@ def discard!
     #
     # @return [Boolean] true if successful, otherwise false
     def undiscard
-      return unless discarded?
+      return false unless discarded?
       run_callbacks(:undiscard) do
         update_attribute(self.class.discard_column, nil)
       end

spec/discard/model_spec.rb

@@ -85,6 +85,10 @@
             post.undiscard
           }.not_to change { post.reload.discarded_at }
         end
+
+        it "returns false" do
+          expect(post.undiscard).to be false
+        end
       end
 
       describe '#undiscard!' do
@@ -139,6 +143,10 @@
             post.discard
           }.not_to change { post.reload.discarded_at }
         end
+
+        it "returns false" do
+          expect(post.discard).to be false
+        end
       end
 
       describe '#discard!' do