dm-accepts_nested_attributes_for 1.2.0

data/spec/one_to_one_spec.rb

@@ -0,0 +1,115 @@
+require 'spec_helper'
+
+describe "Person.has(1, :profile);" do
+
+  before(:all) do
+
+    class ::Person
+
+      include DataMapper::Resource
+      extend ConstraintSupport
+
+      property :id,   Serial
+      property :name, String, :required => true
+
+      has 1, :profile, constraint(:destroy)
+      has 1, :address, :through => :profile
+
+    end
+
+    class ::Profile
+
+      include DataMapper::Resource
+
+      property :id,        Serial
+      property :nick,      String,  :required => true
+
+      belongs_to :person
+      has 1, :address
+
+    end
+
+    class ::Address
+
+      include DataMapper::Resource
+
+      property :id,         Serial
+      property :profile_id, Integer, :required => true, :unique => true, :unique_index => true, :min => 0
+      property :body,       String,  :required => true
+
+      belongs_to :profile
+
+    end
+
+    DataMapper.auto_migrate!
+
+  end
+
+  before(:each) do
+    Address.all.destroy!
+    Profile.all.destroy!
+    Person.all.destroy!
+  end
+
+
+  describe "Person.accepts_nested_attributes_for(:profile)" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :profile
+    end
+
+    it_should_behave_like "every accessible one_to_one association"
+    it_should_behave_like "every accessible one_to_one association with no reject_if proc"
+    it_should_behave_like "every accessible one_to_one association with :allow_destroy => false"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:profile, :allow_destroy => false)" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :profile, :allow_destroy => false
+    end
+
+    it_should_behave_like "every accessible one_to_one association"
+    it_should_behave_like "every accessible one_to_one association with no reject_if proc"
+    it_should_behave_like "every accessible one_to_one association with :allow_destroy => false"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:profile, :allow_destroy => true)" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :profile, :allow_destroy => true
+    end
+
+    it_should_behave_like "every accessible one_to_one association"
+    it_should_behave_like "every accessible one_to_one association with no reject_if proc"
+    it_should_behave_like "every accessible one_to_one association with :allow_destroy => true"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:profile, :reject_if => lambda { |attrs| true })" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :profile, :reject_if => lambda { |attrs| true }
+    end
+
+    it_should_behave_like "every accessible one_to_one association"
+    it_should_behave_like "every accessible one_to_one association with a valid reject_if proc"
+    it_should_behave_like "every accessible one_to_one association with :allow_destroy => false"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:profile, :reject_if => lambda { |attrs| false })" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :profile, :reject_if => lambda { |attrs| false }
+    end
+
+    it_should_behave_like "every accessible one_to_one association"
+    it_should_behave_like "every accessible one_to_one association with no reject_if proc"
+    it_should_behave_like "every accessible one_to_one association with :allow_destroy => false"
+
+  end
+
+end

data/spec/rcov.opts

@@ -0,0 +1,6 @@
+--exclude "spec"
+--sort coverage
+--callsites
+--xrefs
+--profile
+--text-summary

data/spec/resource_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+
+describe DataMapper::NestedAttributes::Resource do
+
+  before(:all) do
+    class ::Project
+      include DataMapper::Resource
+      extend ConstraintSupport
+    
+      property :id, Integer, :key => true, :min => 0
+      property :name, String, :required => true
+
+      has n, :memberships, constraint(:destroy)
+      has n, :people, :through => :memberships
+    end
+
+    class ::Person
+      include DataMapper::Resource
+      extend ConstraintSupport
+
+      property :id, Integer, :key => true, :min => 0
+      property :name, String, :required => true
+
+      has n, :memberships, constraint(:destroy)
+      has n, :projects, :through => :memberships
+      accepts_nested_attributes_for :memberships, :allow_destroy => true
+    end
+
+    class ::Membership
+      include DataMapper::Resource
+
+      property :person_id, Integer, :key => true, :min => 0
+      property :project_id, Integer, :key => true, :min => 0
+      property :role, String, :required => true
+
+      belongs_to :person
+      belongs_to :project
+
+      accepts_nested_attributes_for :person
+      accepts_nested_attributes_for :project
+    end
+
+    DataMapper.auto_migrate!
+  end
+
+  before(:each) do
+    Membership.all.destroy!
+    Person.all.destroy!
+    Project.all.destroy!
+  end
+
+  describe "#uncreatable_keys" do
+    it "includes delete marker" do
+      Person.new.send(:uncreatable_keys).should == [:_delete]
+      Membership.new.send(:uncreatable_keys).should == [:_delete]
+    end
+  end
+
+  describe "#unupdateable_keys" do
+    it "includes primary keys and delete marker" do
+      Person.new.send(:unupdatable_keys).should == [:id, :_delete]
+      Membership.new.send(:unupdatable_keys).should == [:person_id, :project_id, :_delete]
+    end
+  end
+end

data/spec/shared/many_to_many_composite_spec.rb

@@ -0,0 +1,149 @@
+describe "every accessible many_to_many composite association", :shared => true do
+
+  it "should allow to update an existing project via Person#projects_attributes" do
+    pending_if "#{DataMapper::Spec.adapter_name} doesn't support M2M", !HAS_M2M_SUPPORT do
+      Person.all.size.should     == 0
+      Project.all.size.should    == 0
+      Membership.all.size.should == 0
+
+      person = Person.create(:audit_id => 1, :name => 'snusnu')
+      project = Project.create(:audit_id => 2, :name => 'dm-accepts_nested_attributes')
+      membership = Membership.create(:person => person, :project => project, :role => 'maintainer')
+
+      Person.all.size.should     == 1
+      Project.all.size.should    == 1
+      Membership.all.size.should == 1
+
+      person.projects_attributes = [{
+        :id => project.id,
+        :audit_id => project.audit_id,
+        :name => 'still dm-accepts_nested_attributes'
+      }]
+      person.save
+
+      Person.all.size.should     == 1
+      Project.all.size.should    == 1
+      Membership.all.size.should == 1
+
+      Project.first.name.should == 'still dm-accepts_nested_attributes'
+    end
+  end
+
+  it "should return the attributes written to Person#projects_attributes from the Person#projects_attributes reader" do
+    person = Person.new(:audit_id => 1, :name => 'snusnu')
+    person.projects_attributes.should be_nil
+    person.projects_attributes = [{ :name => 'write specs' }]
+    person.projects_attributes.should == [{ :name => 'write specs' }]
+  end
+
+end
+
+describe "every accessible many_to_many composite association with a valid reject_if proc", :shared => true do
+
+  it "should not allow to create a new project via Person#projects_attributes" do
+    Person.all.size.should     == 0
+    Project.all.size.should    == 0
+    Membership.all.size.should == 0
+
+    person = Person.create(:audit_id => 1, :name => 'snusnu')
+
+    Person.all.size.should     == 1
+    Project.all.size.should    == 0
+    Membership.all.size.should == 0
+
+    person.projects_attributes = [{ :name => 'dm-accepts_nested_attributes' }]
+    person.save
+
+    Person.all.size.should     == 1
+    Project.all.size.should    == 0
+    Membership.all.size.should == 0
+  end
+
+end
+
+describe "every accessible many_to_many composite association with no reject_if proc", :shared => true do
+
+  it "should allow to create a new project via Person#projects_attributes" do
+    Person.all.size.should     == 0
+    Project.all.size.should    == 0
+    Membership.all.size.should == 0
+
+    person = Person.create(:audit_id => 1, :name => 'snusnu')
+
+    Person.all.size.should     == 1
+    Project.all.size.should    == 0
+    Membership.all.size.should == 0
+
+    person.projects_attributes = [{ :audit_id => 2, :name => 'dm-accepts_nested_attributes' }]
+
+    Person.all.size.should     == 1
+    Project.all.size.should    == 0
+    Membership.all.size.should == 0
+
+    person.save
+
+    Person.all.size.should     == 1
+    Project.all.size.should    == 1
+    Membership.all.size.should == 1
+
+    Project.first.name.should == 'dm-accepts_nested_attributes'
+  end
+
+end
+
+describe "every accessible many_to_many composite association with :allow_destroy => false", :shared => true do
+
+  it "should not allow to delete an existing project via Person#projects_attributes" do
+    person = Person.create(:audit_id => 1, :name => 'snusnu')
+    project = Project.create(:audit_id => 2, :name => 'dm-accepts_nested_attributes')
+    membership = Membership.create(:person => person, :project => project)
+
+    Person.all.size.should     == 1
+    Project.all.size.should    == 1
+    Membership.all.size.should == 1
+
+    person.projects_attributes = [{ :id => project.id, :audit_id => project.audit_id, :_delete => true }]
+
+    Person.all.size.should     == 1
+    Project.all.size.should    == 1
+    Membership.all.size.should == 1
+
+    person.save
+
+    Person.all.size.should     == 1
+    Project.all.size.should    == 1
+    Membership.all.size.should == 1
+  end
+
+end
+
+describe "every accessible many_to_many composite association with :allow_destroy => true", :shared => true do
+
+  it "should allow to delete an existing project via Person#projects_attributes" do
+    pending_if "#{DataMapper::Spec.adapter_name} doesn't support M2M", !HAS_M2M_SUPPORT do
+      person = Person.create(:audit_id => 1, :name => 'snusnu')
+
+      project_1 = Project.create(:audit_id => 2, :name => 'dm-accepts_nested_attributes')
+      project_2 = Project.create(:audit_id => 3, :name => 'dm-is-localizable')
+      membership_1 = Membership.create(:person => person, :project => project_1)
+      membership_2 = Membership.create(:person => person, :project => project_2)
+
+      Person.all.size.should     == 1
+      Project.all.size.should    == 2
+      Membership.all.size.should == 2
+
+      person.projects_attributes = [{ :id => project_1.id, :audit_id => project_1.audit_id, :_delete => true }]
+
+      Person.all.size.should     == 1
+      Project.all.size.should    == 2
+      Membership.all.size.should == 2
+
+      person.save
+
+      Person.all.size.should     == 1
+      Project.all.size.should    == 1
+      Membership.all.size.should == 1
+    end
+  end
+
+end

data/spec/shared/many_to_many_spec.rb

@@ -0,0 +1,146 @@
+describe "every accessible many_to_many association", :shared => true do
+
+  it "should allow to update an existing project via Person#projects_attributes" do
+    pending_if "#{DataMapper::Spec.adapter_name} doesn't support M2M", !HAS_M2M_SUPPORT do
+      Person.all.size.should            == 0
+      Project.all.size.should           == 0
+      ProjectMembership.all.size.should == 0
+
+      person = Person.create :name => 'snusnu'
+      project = Project.create(:name => 'dm-accepts_nested_attributes')
+      project_membership = ProjectMembership.create(:person => person, :project => project)
+
+      Person.all.size.should            == 1
+      Project.all.size.should           == 1
+      ProjectMembership.all.size.should == 1
+
+      person.projects_attributes = [{ :id => project.id, :name => 'still dm-accepts_nested_attributes' }]
+      person.save
+
+      Person.all.size.should            == 1
+      ProjectMembership.all.size.should == 1
+      Project.all.size.should           == 1
+
+      Project.first.name.should == 'still dm-accepts_nested_attributes'
+    end
+  end
+
+  it "should return the attributes written to Person#projects_attributes from the Person#projects_attributes reader" do
+    person = Person.new :name => 'snusnu'
+    person.projects_attributes.should be_nil
+    person.projects_attributes = [{ :name => 'write specs' }]
+    person.projects_attributes.should == [{ :name => 'write specs' }]
+  end
+
+end
+
+describe "every accessible many_to_many association with a valid reject_if proc", :shared => true do
+
+  it "should not allow to create a new project via Person#projects_attributes" do
+    Person.all.size.should            == 0
+    Project.all.size.should           == 0
+    ProjectMembership.all.size.should == 0
+
+    person = Person.create :name => 'snusnu'
+
+    Person.all.size.should            == 1
+    ProjectMembership.all.size.should == 0
+    Project.all.size.should           == 0
+
+    person.projects_attributes = [{ :name => 'dm-accepts_nested_attributes' }]
+    person.save
+
+    Person.all.size.should            == 1
+    ProjectMembership.all.size.should == 0
+    Project.all.size.should           == 0
+  end
+
+end
+
+describe "every accessible many_to_many association with no reject_if proc", :shared => true do
+
+  it "should allow to create a new project via Person#projects_attributes" do
+    Person.all.size.should            == 0
+    Project.all.size.should           == 0
+    ProjectMembership.all.size.should == 0
+
+    person = Person.create :name => 'snusnu'
+
+    Person.all.size.should            == 1
+    ProjectMembership.all.size.should == 0
+    Project.all.size.should           == 0
+
+    person.projects_attributes = [{ :name => 'dm-accepts_nested_attributes' }]
+
+    Person.all.size.should            == 1
+    ProjectMembership.all.size.should == 0
+    Project.all.size.should           == 0
+
+    person.save
+
+    Person.all.size.should            == 1
+    ProjectMembership.all.size.should == 1
+    Project.all.size.should           == 1
+
+    Project.first.name.should == 'dm-accepts_nested_attributes'
+  end
+
+end
+
+describe "every accessible many_to_many association with :allow_destroy => false", :shared => true do
+
+  it "should not allow to delete an existing project via Person#projects_attributes" do
+    person = Person.create :name => 'snusnu'
+
+    project = Project.create(:name => 'dm-accepts_nested_attributes')
+    project_membership = ProjectMembership.create(:person => person, :project => project)
+
+    Person.all.size.should            == 1
+    ProjectMembership.all.size.should == 1
+    Project.all.size.should           == 1
+
+    person.projects_attributes = [{ :id => project.id, :_delete => true }]
+
+    Person.all.size.should            == 1
+    ProjectMembership.all.size.should == 1
+    Project.all.size.should           == 1
+
+    person.save
+
+    Person.all.size.should            == 1
+    ProjectMembership.all.size.should == 1
+    Project.all.size.should           == 1
+  end
+
+end
+
+describe "every accessible many_to_many association with :allow_destroy => true", :shared => true do
+
+  it "should allow to delete an existing project via Person#projects_attributes" do
+    pending_if "#{DataMapper::Spec.adapter_name} doesn't support M2M", !HAS_M2M_SUPPORT do
+      person = Person.create :name => 'snusnu'
+
+      project_1 = Project.create(:name => 'dm-accepts_nested_attributes')
+      project_2 = Project.create(:name => 'dm-is-localizable')
+      project_membership_1 = ProjectMembership.create(:person => person, :project => project_1)
+      project_membership_2 = ProjectMembership.create(:person => person, :project => project_2)
+
+      Person.all.size.should            == 1
+      ProjectMembership.all.size.should == 2
+      Project.all.size.should           == 2
+
+      person.projects_attributes = [{ :id => project_1.id, :_delete => true }]
+
+      Person.all.size.should            == 1
+      ProjectMembership.all.size.should == 2
+      Project.all.size.should           == 2
+
+      person.save
+
+      Person.all.size.should            == 1
+      ProjectMembership.all.size.should == 1
+      Project.all.size.should           == 1
+    end
+  end
+
+end