dupe 0.4.8 → 0.4.9

data/lib/dupe/attribute_template.rb

@@ -58,12 +58,12 @@ class Dupe
         #     ===> :title, 'test'
         def generate(value=NilValue)
           if value == NilValue
-            v = @default.respond_to?(:call) ? @default.call : @default
+            v = @default.respond_to?(:call) ? @default.call : (@default.dup rescue @default)
           else
             v = (@transformer ? @transformer.call(value) : value)
           end
           
-          return @name, (v && v.respond_to?(:dup) ? v.dup : v)
+          return @name, v
         end
       end
     end

data/spec/lib_specs/dupe_spec.rb

@@ -433,4 +433,32 @@ describe Dupe do
     end
     
   end
+  
+  describe "creating resources should not change definitions (regression test)" do
+    before do
+      Dupe.define :book do |book|
+        book.authors []
+        book.genre do
+          Dupe.find_or_create :genre, :name => 'Sci-fi'
+        end
+      end
+    end
+    
+    it "should not add a record to the definition when the default value is an array" do
+      b = Dupe.create :book
+      a = Dupe.create :author
+      b.authors << a
+      b.authors.include?(a).should be_true
+      b2 = Dupe.create :book
+      b2.authors.should be_empty
+    end
+    
+    it "should not dupe the value returned by a lazy attribute" do
+      b = Dupe.create :book
+      b2 = Dupe.create :book
+      b.genre.should == b2.genre
+      b.genre.name = 'Science Fiction'
+      b2.genre.name.should == 'Science Fiction'
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: dupe
 version: !ruby/object:Gem::Version 
-  version: 0.4.8
+  version: 0.4.9
 platform: ruby
 authors: 
 - Matt Parker
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-02-11 00:00:00 -05:00
+date: 2010-02-12 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency