dm-accepts_nested_attributes_for 1.2.0

data/spec/comb/n-m_surrogate_spec.rb

@@ -0,0 +1,154 @@
+require 'spec_helper'
+
+describe "N:M (surrogate PK)" do
+  before(:all) do
+    class ::Person
+      include DataMapper::Resource
+      extend ConstraintSupport
+
+      property :id,       Serial,  :key => true
+      property :audit_id, Integer, :key => true, :min => 0
+      property :name,     String,  :required => true
+
+      has n, :memberships, constraint(:destroy)
+      has n, :projects, :through => :memberships
+    end
+
+    class ::Project
+      include DataMapper::Resource
+      extend ConstraintSupport
+
+      property :id,       Serial,  :key => true
+      property :audit_id, Integer, :key => true, :min => 0
+      property :name,     String,  :required => true
+
+      has n, :memberships, constraint(:destroy)
+      has n, :people, :through => :memberships
+    end
+
+    class ::Membership
+      include DataMapper::Resource
+
+      # keep properties unordered
+      property :person_audit_id,  Integer, :required => true, :min => 0
+      property :project_audit_id, Integer, :required => true, :min => 0
+      property :id,               Serial,  :key => true
+      property :project_id,       Integer, :required => true, :min => 0
+      property :person_id,        Integer, :required => true, :min => 0
+      property :role,             String
+
+      belongs_to :person
+      belongs_to :project
+    end
+
+    DataMapper.auto_migrate!
+  end
+
+  before(:each) do
+    Membership.all.destroy!
+    Project.all.destroy!
+    Person.all.destroy!
+  end
+
+  describe "Person.accepts_nested_attributes_for(:projects)" do
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects
+    end
+
+    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
+        person1  = Person.create(:id => 1, :audit_id => 10, :name => 'Martin')
+        project1 = Project.create(:id => 100, :audit_id => 1000, :name => 'foo')
+        project2 = Project.create(:id => 200, :audit_id => 2000, :name => 'bar')
+        Membership.create(:id => 10000, :person => person1, :project => project1, :role => 'foo-maintainer')
+        Membership.create(:id => 20000, :person => person1, :project => project2, :role => 'bar-contributor')
+        person1.reload
+  
+        person2  = Person.create(:id => 2, :audit_id => 20, :name => 'John')
+        project3 = Project.create(:id => 300, :audit_id => 3000, :name => 'qux')
+        project4 = Project.create(:id => 400, :audit_id => 4000, :name => 'baz')
+        Membership.create(:id => 30000, :person => person2, :project => project3, :role => 'qux-maintainer')
+        Membership.create(:id => 40000, :person => person2, :project => project4, :role => 'baz-contributor')
+        person2.reload
+  
+        Person.all.size.should     == 2
+        Project.all.size.should    == 4
+        Membership.all.size.should == 4
+  
+        person1.projects_attributes = [{ :id => 100, :audit_id => 1000, :name => 'still foo' }]
+        person1.save.should be_true
+  
+        Person.all.map { |p| [p.id, p.audit_id, p.name] }.should == [
+          [1, 10, 'Martin'],
+          [2, 20, 'John'],
+        ]
+        Project.all.map { |p| [p.id, p.audit_id, p.name] }.should == [
+          [100, 1000, 'still foo'],
+          [200, 2000, 'bar'],
+          [300, 3000, 'qux'],
+          [400, 4000, 'baz'],
+        ]
+        Membership.all.map { |m| [m.person_id, m.person_audit_id, m.project_id, m.project_audit_id, m.role] }.should == [
+          [1, 10, 100, 1000, 'foo-maintainer'],
+          [1, 10, 200, 2000, 'bar-contributor'],
+          [2, 20, 300, 3000, 'qux-maintainer'],
+          [2, 20, 400, 4000, 'baz-contributor'],
+        ]
+      end
+    end
+  
+    it "should allow to create a new project via Person#projects_attributes" do
+      pending_if "#{DataMapper::Spec.adapter_name} doesn't support M2M", !HAS_M2M_SUPPORT do
+        person1  = Person.create(:id => 1, :audit_id => 10, :name => 'Martin')
+        project1 = Project.create(:id => 100, :audit_id => 1000, :name => 'foo')
+        project2 = Project.create(:id => 200, :audit_id => 2000, :name => 'bar')
+        Membership.create(:id => 10000, :person => person1, :project => project1, :role => 'foo-maintainer')
+        Membership.create(:id => 20000, :person => person1, :project => project2, :role => 'bar-contributor')
+        person1.reload
+  
+        person2  = Person.create(:id => 2, :audit_id => 20, :name => 'John')
+        project3 = Project.create(:id => 300, :audit_id => 3000, :name => 'qux')
+        project4 = Project.create(:id => 400, :audit_id => 4000, :name => 'baz')
+        Membership.create(:id => 30000, :person => person2, :project => project3, :role => 'qux-maintainer')
+        Membership.create(:id => 40000, :person => person2, :project => project4, :role => 'baz-contributor')
+        person2.reload
+  
+        Person.all.size.should     == 2
+        Project.all.size.should    == 4
+        Membership.all.size.should == 4
+  
+        person1.projects_attributes = [{ :id => 500, :audit_id => 5000, :name => 'fibble' }]
+        person1.save.should be_true
+  
+        Person.all.map { |p| [p.id, p.audit_id, p.name] }.should == [
+          [1, 10, 'Martin'],
+          [2, 20, 'John'],
+        ]
+        Project.all.map { |p| [p.id, p.audit_id, p.name] }.should == [
+          [100, 1000, 'foo'],
+          [200, 2000, 'bar'],
+          [300, 3000, 'qux'],
+          [400, 4000, 'baz'],
+          [500, 5000, 'fibble'],
+        ]
+
+        ids = [10000, 20000, 30000, 40000]
+        Membership.all(:id => ids).map { |m|
+          [m.id, m.person_id, m.person_audit_id, m.project_id, m.project_audit_id, m.role]
+        }.should == [
+          [10000, 1, 10, 100, 1000, 'foo-maintainer'],
+          [20000, 1, 10, 200, 2000, 'bar-contributor'],
+          [30000, 2, 20, 300, 3000, 'qux-maintainer'],
+          [40000, 2, 20, 400, 4000, 'baz-contributor'],
+        ]
+
+        # exclude the serial property
+        Membership.all(:id.not => ids).map { |m|
+          [m.person_id, m.person_audit_id, m.project_id, m.project_audit_id, m.role]
+        }.should == [
+          [1, 10, 500, 5000, nil],
+        ]
+      end
+    end
+  end
+end

data/spec/many_to_many_composite_spec.rb

@@ -0,0 +1,120 @@
+require 'spec_helper'
+
+describe "Person.has(n, :projects, :through => :memberships) with CPK;" do
+
+  before(:all) do
+    class ::Person
+
+      include DataMapper::Resource
+      extend ConstraintSupport
+
+      property :id,       Serial,  :key => true
+      property :audit_id, Integer, :key => true, :min => 0
+      property :name,     String,  :required => true
+
+      has n, :memberships, constraint(:destroy)
+      has n, :projects, :through => :memberships
+
+    end
+
+    class ::Project
+
+      include DataMapper::Resource
+      extend ConstraintSupport
+
+      property :id,       Serial,  :key => true
+      property :audit_id, Integer, :key => true, :min => 0
+      property :name,     String,  :required => true
+
+      has n, :memberships, constraint(:destroy)
+      has n, :people, :through => :memberships
+
+    end
+
+    class ::Membership
+
+      include DataMapper::Resource
+
+      property :person_id,        Integer, :key => true, :min => 0
+      property :person_audit_id,  Integer, :key => true, :min => 0
+      property :project_id,       Integer, :key => true, :min => 0
+      property :project_audit_id, Integer, :key => true, :min => 0
+      property :role,             String,  :required => false
+
+      belongs_to :person, :key => true
+      belongs_to :project, :key => true
+
+    end
+
+    DataMapper.auto_migrate!
+
+  end
+
+  before(:each) do
+    Membership.all.destroy!
+    Project.all.destroy!
+    Person.all.destroy!
+  end
+
+
+  describe "Person.accepts_nested_attributes_for(:projects)" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects
+    end
+
+    it_should_behave_like "every accessible many_to_many composite association"
+    it_should_behave_like "every accessible many_to_many composite association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_many composite association with :allow_destroy => false"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:projects, :allow_destroy => false)" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects, :allow_destroy => false
+    end
+
+    it_should_behave_like "every accessible many_to_many composite association"
+    it_should_behave_like "every accessible many_to_many composite association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_many composite association with :allow_destroy => false"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:projects, :allow_destroy = true)" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects, :allow_destroy => true
+    end
+
+    it_should_behave_like "every accessible many_to_many composite association"
+    it_should_behave_like "every accessible many_to_many composite association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_many composite association with :allow_destroy => true"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:projects, :reject_if => lambda { |attrs| true })" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects, :reject_if => lambda { |attrs| true }
+    end
+
+    it_should_behave_like "every accessible many_to_many composite association"
+    it_should_behave_like "every accessible many_to_many composite association with a valid reject_if proc"
+    it_should_behave_like "every accessible many_to_many composite association with :allow_destroy => false"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:projects, :reject_if => lambda { |attrs| false })" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects, :reject_if => lambda { |attrs| false }
+    end
+
+    it_should_behave_like "every accessible many_to_many composite association"
+    it_should_behave_like "every accessible many_to_many composite association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_many composite association with :allow_destroy => false"
+
+  end
+
+end

data/spec/many_to_many_spec.rb

@@ -0,0 +1,129 @@
+require 'spec_helper'
+
+describe "Person.has(n, :projects, :through => :project_memberships);" do
+
+  before(:all) do
+
+    class ::Person
+
+      include DataMapper::Resource
+      extend ConstraintSupport
+
+      property :id,   Serial
+      property :name, String, :required => true
+
+      has n, :project_memberships, constraint(:destroy)
+      has n, :projects, :through => :project_memberships
+      has n, :tasks, :through => :projects
+
+    end
+
+    class ::Project
+
+      include DataMapper::Resource
+      extend ConstraintSupport
+
+      property :id,   Serial
+      property :name, String, :required => true
+
+      has n, :project_memberships, constraint(:destroy)
+      has n, :people, :through => :project_memberships
+      has n, :tasks, constraint(:destroy)
+
+    end
+
+    class ::ProjectMembership
+
+      include DataMapper::Resource
+
+      property :id,         Serial
+
+      belongs_to :person
+      belongs_to :project
+
+    end
+
+    class ::Task
+
+      include DataMapper::Resource
+
+      property :id,   Serial
+      property :name, String,  :required => true
+
+      belongs_to :project
+
+    end
+
+    DataMapper.auto_migrate!
+
+  end
+
+  before(:each) do
+    ProjectMembership.all.destroy!
+    Task.all.destroy!
+    Project.all.destroy!
+    Person.all.destroy!
+  end
+
+
+  describe "Person.accepts_nested_attributes_for(:projects)" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects
+    end
+
+    it_should_behave_like "every accessible many_to_many association"
+    it_should_behave_like "every accessible many_to_many association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_many association with :allow_destroy => false"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:projects, :allow_destroy => false)" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects, :allow_destroy => false
+    end
+
+    it_should_behave_like "every accessible many_to_many association"
+    it_should_behave_like "every accessible many_to_many association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_many association with :allow_destroy => false"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:projects, :allow_destroy = true)" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects, :allow_destroy => true
+    end
+
+    it_should_behave_like "every accessible many_to_many association"
+    it_should_behave_like "every accessible many_to_many association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_many association with :allow_destroy => true"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:projects, :reject_if => lambda { |attrs| true })" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects, :reject_if => lambda { |attrs| true }
+    end
+
+    it_should_behave_like "every accessible many_to_many association"
+    it_should_behave_like "every accessible many_to_many association with a valid reject_if proc"
+    it_should_behave_like "every accessible many_to_many association with :allow_destroy => false"
+
+  end
+
+  describe "Person.accepts_nested_attributes_for(:projects, :reject_if => lambda { |attrs| false })" do
+
+    before(:all) do
+      Person.accepts_nested_attributes_for :projects, :reject_if => lambda { |attrs| false }
+    end
+
+    it_should_behave_like "every accessible many_to_many association"
+    it_should_behave_like "every accessible many_to_many association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_many association with :allow_destroy => false"
+
+  end
+
+end

data/spec/many_to_one_composite_spec.rb

@@ -0,0 +1,120 @@
+require 'spec_helper'
+
+describe "ProjectMembership.belongs_to(:person) with CPK;" do
+
+  before(:all) do
+    class ::Person
+
+      include DataMapper::Resource
+      extend ConstraintSupport
+
+      property :id,       Serial,  :key => true
+      property :audit_id, Integer, :key => true, :min => 0
+      property :name,     String,  :required => true
+
+      has n, :memberships, constraint(:destroy)
+      has n, :projects, :through => :memberships
+
+    end
+
+    class ::Project
+
+      include DataMapper::Resource
+      extend ConstraintSupport
+
+      property :id,       Serial,  :key => true
+      property :audit_id, Integer, :key => true, :min => 0
+      property :name,     String,  :required => true
+
+      has n, :memberships, constraint(:destroy)
+      has n, :people, :through => :memberships
+
+    end
+
+    class ::Membership
+
+      include DataMapper::Resource
+
+      property :person_id,        Integer, :key => true, :min => 0
+      property :person_audit_id,  Integer, :key => true, :min => 0
+      property :project_id,       Integer, :key => true, :min => 0
+      property :project_audit_id, Integer, :key => true, :min => 0
+      property :role,             String,  :required => false
+
+      belongs_to :person, :key => true
+      belongs_to :project, :key => true
+
+    end
+
+    DataMapper.auto_migrate!
+
+  end
+
+  before(:each) do
+    Membership.all.destroy!
+    Project.all.destroy!
+    Person.all.destroy!
+  end
+
+
+  describe "Membership.accepts_nested_attributes_for(:person)" do
+
+    before(:all) do
+      Membership.accepts_nested_attributes_for :person
+    end
+
+    it_should_behave_like "every accessible many_to_one composite association"
+    it_should_behave_like "every accessible many_to_one composite association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_one composite association with :allow_destroy => false"
+
+  end
+
+  describe "Membership.accepts_nested_attributes_for(:person, :allow_destroy => false)" do
+
+    before(:all) do
+      Membership.accepts_nested_attributes_for :person, :allow_destroy => false
+    end
+
+    it_should_behave_like "every accessible many_to_one composite association"
+    it_should_behave_like "every accessible many_to_one composite association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_one composite association with :allow_destroy => false"
+
+  end
+
+  describe "Membership.accepts_nested_attributes_for(:person, :allow_destroy = true)" do
+
+    before(:all) do
+      Membership.accepts_nested_attributes_for :person, :allow_destroy => true
+    end
+
+    it_should_behave_like "every accessible many_to_one composite association"
+    it_should_behave_like "every accessible many_to_one composite association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_one composite association with :allow_destroy => true"
+
+  end
+
+  describe "Membership.accepts_nested_attributes_for(:person, :reject_if => lambda { |attrs| true })" do
+
+    before(:all) do
+      Membership.accepts_nested_attributes_for :person, :reject_if => lambda { |attrs| true }
+    end
+
+    it_should_behave_like "every accessible many_to_one composite association"
+    it_should_behave_like "every accessible many_to_one composite association with a valid reject_if proc"
+    it_should_behave_like "every accessible many_to_one composite association with :allow_destroy => false"
+
+  end
+
+  describe "Membership.accepts_nested_attributes_for(:person, :reject_if => lambda { |attrs| false })" do
+
+    before(:all) do
+      Membership.accepts_nested_attributes_for :person, :reject_if => lambda { |attrs| false }
+    end
+
+    it_should_behave_like "every accessible many_to_one composite association"
+    it_should_behave_like "every accessible many_to_one composite association with no reject_if proc"
+    it_should_behave_like "every accessible many_to_one composite association with :allow_destroy => false"
+
+  end
+
+end