active_nomad 0.2.1 → 0.2.2

data/CHANGELOG

@@ -1,3 +1,8 @@
+== 0.2.2 2010-10-07
+
+ * Support custom destruction strategies.
+ * #save noops if no persistence strategy defined.
+
 == 0.2.1 2010-10-06
 
  * from_json noops on invalid JSON strings.

data/lib/active_nomad/version.rb

@@ -1,5 +1,5 @@
 module ActiveNomad
-  VERSION = [0, 2, 1]
+  VERSION = [0, 2, 2]
 
   class << VERSION
     include Comparable

data/lib/active_nomad.rb

@@ -2,8 +2,6 @@ require 'active_record'
 require 'cgi'
 
 module ActiveNomad
-  NoPersistenceStrategy = Class.new(RuntimeError)
-
   class Base < ActiveRecord::Base
     #
     # Tell this record how to save itself.
@@ -97,30 +95,33 @@ module ActiveNomad
       from_serialized_attributes(serialized_attributes)
     end
 
-    protected
-
     #
-    # Persist the object.
+    # Destroy the object.
     #
     # The default is to call the block registered with
-    # #to_save. Override if you don't want to use #to_save.
+    # #to_destroy. Override if you don't want to use #to_destroy.
     #
-    def persist
-      @save_proc or
-        raise NoPersistenceStrategy, "no persistence strategy - use #to_save to define one"
-      @save_proc.call(self)
+    def destroy
+      if @destroy_proc
+        @destroy_proc.call(self)
+      end
+      self
     end
 
+    protected
+
     #
-    # Destroy the object.
+    # Persist the object.
     #
     # The default is to call the block registered with
-    # #to_destroy. Override if you don't want to use #to_destroy.
+    # #to_save. Override if you don't want to use #to_save.
     #
-    def destroy
-      @destroy_proc or
-        raise NoDestructionStrategy, "no destruction strategy - use #to_destroy to define one"
-      @destroy_proc.call(self)
+    def persist
+      if @save_proc
+        @save_proc.call(self)
+      else
+        true
+      end
     end
 
     private

data/spec/unit/active_nomad_spec.rb

@@ -84,9 +84,9 @@ describe ActiveNomad::Base do
 
   describe "#save" do
     describe "when no save strategy has been defined" do
-      it "should raise a NoPersistenceStrategy error" do
+      it "should return true" do
         instance = ActiveNomad::Base.new
-        lambda{instance.save}.should raise_error(ActiveNomad::NoPersistenceStrategy)
+        instance.save.should be_true
       end
     end
 
@@ -123,6 +123,30 @@ describe ActiveNomad::Base do
     end
   end
 
+  describe "#destroy" do
+    describe "no destruction strategy has been defined" do
+      it "should return the object" do
+        instance = ActiveNomad::Base.new
+        instance.destroy.should equal(instance)
+      end
+
+      describe "when a destruction strategy has been defined" do
+        before do
+          destroys = @destroys = []
+          @instance = ActiveNomad::Base.new
+          @instance.to_destroy do |*args|
+            destroys << args
+          end
+        end
+
+        it "should call the destroy_proc with the record as an argument" do
+          @instance.destroy
+          @destroys.should == [[@instance]]
+        end
+      end
+    end
+  end
+
   describe "#to_serialized_attributes" do
     it "should return a list of attribute names with serialized attributes, sorted by name" do
       klass = Class.new(ActiveNomad::Base) do

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: active_nomad
 version: !ruby/object:Gem::Version 
-  hash: 21
+  hash: 19
   prerelease: false
   segments: 
   - 0
   - 2
-  - 1
-  version: 0.2.1
+  - 2
+  version: 0.2.2
 platform: ruby
 authors: 
 - George Ogata
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-10-06 00:00:00 -04:00
+date: 2010-10-07 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency