common_repository_model 0.0.8 → 0.0.9

data/common_repository_model.gemspec

@@ -24,7 +24,6 @@ Gem::Specification.new do |gem|
   gem.add_dependency "activesupport", "~>3.2"
   gem.add_dependency "activemodel", "~>3.2"
   gem.add_dependency "builder", "~>3.0"
-  gem.add_dependency "morphine"
   gem.add_dependency "active_model_serializers"
   gem.add_runtime_dependency "minitest"
   gem.add_runtime_dependency "minitest-matchers"

data/lib/common_repository_model/collection.rb

@@ -60,9 +60,26 @@ module CommonRepositoryModel
       has_and_belongs_to_many(method_name, options)
     end
 
-    before_save :register_parent_and_collections
+    before_validation :assign_area!, on: :create
+    def assign_area!
+      self.__area =
+      if area
+        CommonRepositoryModel::Area.find(area.pid)
+      else
+        CommonRepositoryModel::Area.find_by_name!(name_of_area_to_assign)
+      end
+    rescue CommonRepositoryModel::ObjectNotFoundError
+      true
+    end
+    protected :assign_area!
+
+    def name_of_area_to_assign
+      'NDLIB'
+    end
+
+    before_validation :register_parent_and_collections, on: :create
+    before_validation :register_parent_and_collections, on: :update
     def register_parent_and_collections
-      self.__area = CommonRepositoryModel::Area.find(area.pid) if area
 
       collected_has_members = membership_registry.has_members.
       collect do |association_name|
@@ -74,7 +91,6 @@ module CommonRepositoryModel
       collect do |association_name|
         public_send(association_name)
       end.flatten
-
       self.parent_collections = collected_is_member_of
 
       true
@@ -89,31 +105,21 @@ module CommonRepositoryModel
       parent_collections.collect(&:__area).uniq
     end
 
-    def area=(an_area)
-      if is_root?
-        self.__area = an_area
-      else
-        self.__area = parent_areas.first
-      end
-    end
-
     def area
       if is_root?
         __area
       else
         parent_areas.first
-      end
+      end ||
+      CommonRepositoryModel::Area.find_by_name!(name_of_area_to_assign)
+    rescue CommonRepositoryModel::ObjectNotFoundError
+      nil
     end
 
     def area_name
       area.name
     end
 
-    include Morphine
-    register :named_area_finder do
-      CommonRepositoryModel::Area.method(:find_by_name!)
-    end
-
     belongs_to(
       :__area,
       class_name:'CommonRepositoryModel::Area',

data/lib/common_repository_model/test_support.rb

@@ -21,11 +21,17 @@ module CommonRepositoryModel
       end
     end
 
-    def with_persisted_area(name = nil)
+    def with_persisted_area(name = nil, &block)
       options = {}
       options[:name] = name if name
-      area = FactoryGirl.create(:common_repository_model_area, options)
-      yield(area)
+      area = nil
+      area = CommonRepositoryModel::Area.find_by_name(name) if name
+      area ||= FactoryGirl.create(:common_repository_model_area, options)
+      if block.arity == 1
+        block.call(area)
+      else
+        block.call
+      end
     ensure
       area.delete if area
     end

data/lib/common_repository_model/version.rb

@@ -1,3 +1,3 @@
 module CommonRepositoryModel
-  VERSION = "0.0.8"
+  VERSION = "0.0.9"
 end

data/lib/generators/common_repository_model/collection/templates/collection.rb.erb

@@ -2,6 +2,9 @@
 #  `rails generate common_repository_model::collection <%= class_name %>`
 require 'common_repository_model/collection'
 class <%= class_name %> < CommonRepositoryModel::Collection
+  # Write the logic for assigning the area
+  def name_of_area_to_assign
+  end
 
   # ***************************************************************************
   # ***************************************************************************
@@ -29,7 +32,7 @@ class <%= class_name %> < CommonRepositoryModel::Collection
   # is_member_of :containers, type: "Container", property: "is_contained_in"
   #
   # For Examples Specs:
-  # https://github.com/ndlib/common_respository_model/blob/master/spec/common_respository_model/integration_spec.rb
+  # https://github.com/ndlib/common_repository_model/blob/master/spec/common_repository_model/integration_spec.rb
   #
 
 end

data/lib/generators/common_repository_model/collection/templates/collection_factory.rb.erb

@@ -5,7 +5,7 @@
 
 require 'common_repository_model/test_support'
 FactoryGirl.define do
-  factory :<%= singular_table_name %>, class: <%= class_name %> do
+  factory :<%= singular_table_name %>, class: <%= class_name %>, parent: :common_repository_model_collection do
 <% for attribute in attributes -%>
     <%= attribute.name %> <%= attribute.default.inspect %>
 <% end -%>

data/spec/common_respository_model/area_spec.rb

@@ -30,9 +30,9 @@ describe CommonRepositoryModel::Area do
   end
 
   describe 'integration (with persistence)' do
-    let(:collection) { FactoryGirl.build(:common_repository_model_collection, area: nil) }
+    let(:collection) { FactoryGirl.build(:common_repository_model_collection) }
     it 'should .find_by_name and .find_by_name!' do
-      with_persisted_area do |area|
+      with_persisted_area(collection.name_of_area_to_assign) do |area|
         CommonRepositoryModel::Area.find_by_name(area.name).must_equal area
         CommonRepositoryModel::Area.
           find_by_name("#{area.name}-tmp").must_equal nil
@@ -44,7 +44,7 @@ describe CommonRepositoryModel::Area do
       end
     end
     it 'should save' do
-      with_persisted_area do |area|
+      with_persisted_area(collection.name_of_area_to_assign) do |area|
         # Before we can add a collection, the containing object
         # must be saved
         area.collections << collection

data/spec/common_respository_model/collection_serializer_spec.rb

@@ -9,7 +9,9 @@ describe CommonRepositoryModel::CollectionSerializer do
   let(:json) { JSON.parse(subject.to_json) }
   let(:root) { json.fetch('collection') }
   it 'should be JSON' do
-    root.fetch('pid')
-    root.fetch('area').must_equal collection.area.pid
+    with_persisted_area(collection.name_of_area_to_assign) do
+      root.fetch('pid')
+      root.fetch('area').must_equal collection.area.pid
+    end
   end
 end

data/spec/common_respository_model/collection_spec.rb

@@ -8,22 +8,18 @@ describe CommonRepositoryModel::Collection do
     subject.respond_to?(:area_name)
   end
 
-  it 'should have #named_area_finder' do
-    lambda {
-      subject.named_area_finder.call('NEVER-GONNA_EXIST')
-    }.must_raise CommonRepositoryModel::ObjectNotFoundError
+  it 'should have an #name_of_area_to_assign' do
+    subject.respond_to?(:name_of_area_to_assign)
   end
 
   it 'should require an area' do
-    with_persisted_area do |area|
-      subject.named_area_finder.call(area.name)
-
-      subject.area = area
-      subject.area.must_be_kind_of(CommonRepositoryModel::Area)
+    subject.valid?.must_equal false
+    subject.errors[:area].size.wont_equal 0
+    with_persisted_area(subject.name_of_area_to_assign) do |area|
+      subject.area.must_equal area
       subject.valid?.must_equal true
-      subject.area = nil
-      subject.valid?.must_equal false
     end
+
   end
 
   describe '#find_or_build_data_for_given_slot_names' do
@@ -61,34 +57,19 @@ describe CommonRepositoryModel::Collection do
 
   describe 'integration' do
     let(:child_collection) {
-      FactoryGirl.build(:common_repository_model_collection, area: nil)
+      FactoryGirl.build(:common_repository_model_collection)
     }
     it 'should keep a child collection in the parent collection' do
-      with_persisted_area do |area_1|
-        with_persisted_area do |area_2|
-          area_1.wont_equal area_2
-
-          subject.area = area_1
-          subject.save!
-
-          child_collection.area = area_2
-
-          child_collection.parent_collections += [subject]
-
-          child_collection.area.must_equal subject.area
-
-          child_collection.save!
-
-          child_collection.area.must_equal subject.area
-        end
+      with_persisted_area(subject.name_of_area_to_assign) do
+        subject.save!
+        child_collection.parent_collections += [subject]
+        child_collection.area.must_equal subject.area
       end
     end
     it 'should handle parent/child collection' do
       # Before we can add a collection, the containing object
       # must be saved
-      with_persisted_area do |area|
-        subject.area = area
-        child_collection.area = area
+      with_persisted_area(subject.name_of_area_to_assign) do
         subject.save!
         subject.child_collections << child_collection
         subject.save!

data/spec/common_respository_model/data_spec.rb

@@ -32,8 +32,8 @@ describe CommonRepositoryModel::Data do
     end
 
     it 'should save' do
-      with_persisted_area do |area|
-        collection.area = area
+      with_persisted_area(collection.name_of_area_to_assign) do |area|
+
         # Before we can add a collection, the containing object
         # must be saved
         collection.save!

data/spec/common_respository_model/integration_spec.rb

@@ -1,258 +1,223 @@
 require_relative '../spec_helper'
-require 'common_repository_model/collection'
-
-
-class Clothing < CommonRepositoryModel::Data
-end
-
-class Job < CommonRepositoryModel::Collection
-  has_and_belongs_to_many(
-    :people,
-    class_name: 'Person',
-    property: :has_workers,
-    inverse_of: :is_working_as
-  )
-end
-
-class Family < CommonRepositoryModel::Collection
-  has_members(
-    :family_members,
-    class_name: 'Person',
-    property: :is_family_member_of
-  )
-end
-
-class Person < CommonRepositoryModel::Collection
-
-  is_member_of(
-    :parents,
-    class_name: 'Person',
-    property: :is_child_of
-  )
-
-  has_members(
-    :children,
-    class_name: 'Person',
-    property: :is_child_of
-  )
-
-  # People can be part of multiple families
-  is_member_of(
-    :families,
-    class_name: 'Family',
-    property: :is_family_member_of
-  )
-
-  has_and_belongs_to_many(
-    :jobs,
-    class_name: 'Job',
-    property: :is_working_as,
-    inverse_of: :has_workers
-  )
-
-end
+require_relative '../support/spec_models'
 
 describe CommonRepositoryModel::Collection do
-  after(:each) do
-    area.delete if area.persisted?
-  end
-  let(:area) { FactoryGirl.create(:common_repository_model_area) }
-  let(:family) { Family.new(area: area) }
-  let(:lawyer) { Job.new(area: area) }
-  let(:doctor) { Job.new(area: area) }
-  let(:heathcliff) { Person.new(area: area) }
-  let(:claire) { Person.new(area: area) }
-  let(:theo) { Person.new(area: area) }
-  let(:vanessa) { Person.new(area: area) }
-  let(:rudy) { Person.new(area: area) }
-  let(:dress) { Clothing.new(slot_name: 'outerwear') }
-
-  before(:all) do
-    family.save
-
-    lawyer.save
-    doctor.save
-
-    heathcliff.save
-    claire.save
-    theo.save
-    vanessa.save
-    rudy.save
-
-    dress.save
-
-    claire.children = [theo,vanessa,rudy]
-    claire.jobs << lawyer
-    claire.families << family
-    claire.data << dress
-    claire.save
-
-    heathcliff.children = [theo,vanessa,rudy]
-    heathcliff.jobs << doctor
-    heathcliff.families << family
-    heathcliff.save
-
-    theo.families << family
-    theo.save
-    vanessa.families << family
-    vanessa.save
-    rudy.families << family
-    rudy.save
-
-    family.family_members = [claire,heathcliff,theo,vanessa,rudy]
-    family.save.must_equal true
-
-    @dress = dress.class.find(dress.pid)
-    @theo = theo.class.find(theo.pid)
-    @family = family.class.find(family.pid)
-    @claire = claire.class.find(claire.pid)
-    @rudy = rudy.class.find(rudy.pid)
-  end
-
-  it 'verifies complicated relationships' do
-    verify_initial_relations_for_family
-    verify_initial_relations_for_theo
-    verify_initial_relations_for_claire
-    verify_initial_relations_for_dress
-
-    verify_claire_adding_a_child
-
-    verify_claire_losing_a_child
-
-    verify_rudy_losing_a_parent
-  end
-
-  protected
-  def verify_initial_relations_for_family
-    # We are not storing the RELS-EXT entry on the "container" collection
-    assert_rels_ext(
-      @family,
-      :has_family_members,
-      []
-    )
-    assert_rels_ext(
-      @family,
-      :has_members,
-      []
-    )
-
-    # However, we do have access to Family#family_members
-    assert_active_fedora_has_many(
-      @family,
-      :family_members,
-      [@theo,@claire, heathcliff,rudy, vanessa]
-    )
-
-    assert_active_fedora_has_many(
-      @family,
-      :child_collections,
-      [@theo,@claire, heathcliff,rudy, vanessa]
-    )
-  end
-
-  def verify_initial_relations_for_theo
-    assert_rels_ext @theo, :is_child_of, [@claire, heathcliff]
-    assert_rels_ext @theo, :is_member_of, [@claire, family, heathcliff]
-
-    assert_active_fedora_has_many(@theo, :jobs, [])
-    assert_active_fedora_has_many(@theo, :families, [family])
-    assert_active_fedora_has_many(@theo, :parents, [heathcliff,claire])
-    assert_active_fedora_has_many(@theo, :child_collections, [])
-    assert_active_fedora_has_many(
-      @theo, :parent_collections, [heathcliff,claire,family]
-    )
+  describe 'with journal / volume' do
+    it 'should have volumes' do
+      journal = Journal.new
+      with_persisted_area(journal.name_of_area_to_assign) do |area|
+        journal.save!
+        volume = JournalVolume.new
+        volume.save!
+        volume.journals << journal
+        volume.save!
+
+        @journal = journal.class.find(journal.pid)
+        @volume = volume.class.find(volume.pid)
+
+        assert_rels_ext(@volume, :is_volume_of, [@journal])
+        assert_rels_ext(@volume, :is_member_of, [@journal])
+        assert_active_fedora_has_many(@volume, :journals, [@journal])
+
+        assert_rels_ext(@journal, :is_volume_of, [])
+        assert_rels_ext(@journal, :is_member_of, [])
+        assert_active_fedora_has_many(@journal, :volumes, [@volume])
+
+      end
+    end
   end
-
-  def verify_initial_relations_for_dress
-    assert_rels_ext(@dress, :is_part_of, [@claire])
-    assert_active_fedora_belongs_to(@dress, :collection, @claire)
-  end
-
-  def verify_initial_relations_for_claire
-    assert_rels_ext @claire, :is_parent_of, []
-    assert_rels_ext @claire, :is_child_of, []
-    assert_rels_ext @claire, :is_family_member_of, [family]
-    assert_rels_ext @claire, :is_member_of, [family]
-
-    assert_active_fedora_has_many(@claire,:data,[@dress])
-    assert_active_fedora_has_many(@claire,:jobs,[lawyer])
-    assert_active_fedora_has_many(@claire,:families,[family])
-    assert_active_fedora_has_many(@claire,:children,[theo, rudy, vanessa])
-    assert_active_fedora_has_many(
-      @claire,:child_collections,[theo, rudy, vanessa]
-    )
-    assert_active_fedora_has_many(@claire,:parent_collections,[family])
-  end
-
-  def verify_claire_adding_a_child
-    sandra = Person.new(area: area)
-    sandra.save
-    @claire.children << sandra
-    @claire.save
-    @claire = @claire.class.find(@claire.pid)
-
-    assert_rels_ext @claire, :is_parent_of, []
-    assert_rels_ext @claire, :is_child_of, []
-
-    assert_active_fedora_has_many(@claire,:parents,[])
-    assert_active_fedora_has_many(@claire,:children,[theo, rudy, vanessa,sandra])
-    assert_active_fedora_has_many(
-      @claire,:child_collections,[theo, rudy, vanessa, sandra]
-    )
-    assert_active_fedora_has_many(
-      @claire,:parent_collections,[family]
-    )
-  end
-
-  def verify_claire_losing_a_child
-    @claire.children = [theo,rudy]
-    @claire.save
-
-    assert_active_fedora_has_many(@claire,:children,[theo, rudy])
-    assert_active_fedora_has_many(
-      @claire,:child_collections,[theo, rudy]
-    )
-
-    reloaded_vanessa = vanessa.class.find(vanessa.pid)
-
-    # Note, just because we said Claire was not Vanessa's parent, Vanessa did
-    # not update
-    assert_rels_ext reloaded_vanessa, :is_child_of, [claire, heathcliff]
-    assert_rels_ext(
-      reloaded_vanessa, :is_member_of, [claire, heathcliff, family]
-    )
-
-    assert_active_fedora_has_many(
-      reloaded_vanessa, :parents, [claire, heathcliff]
-    )
-    assert_active_fedora_has_many(
-      reloaded_vanessa, :parent_collections, [claire,heathcliff, family]
-    )
-
-  end
-
-  def verify_rudy_losing_a_parent
-    assert_rels_ext @rudy, :is_child_of, [heathcliff,claire]
-    assert_rels_ext @rudy, :is_member_of, [heathcliff,claire,family]
-    assert_active_fedora_has_many @rudy, :parents, [heathcliff,claire]
-    assert_active_fedora_has_many(
-      @rudy, :parent_collections, [heathcliff,claire,family]
-    )
-
-    @rudy.break_relation_with_parents(heathcliff)
-    @rudy.save.must_equal true
-
-    assert_active_fedora_has_many(@rudy, :parents, [claire])
-    assert_active_fedora_has_many(@rudy, :parent_collections, [claire, family])
-    assert_rels_ext @rudy, :is_child_of, [claire]
-    assert_rels_ext @rudy, :is_member_of, [claire, family]
-
-    @rudy = @rudy.class.find(@rudy.pid)
-
-    assert_rels_ext @rudy, :is_child_of, [claire]
-    assert_rels_ext @rudy, :is_member_of, [claire, family]
-
-    assert_active_fedora_has_many(@rudy, :parents, [claire])
-    assert_active_fedora_has_many(@rudy, :parent_collections, [claire, family])
+  describe 'with family' do
+    let(:family) { Family.new }
+    let(:lawyer) { Job.new }
+    let(:doctor) { Job.new }
+    let(:heathcliff) { Person.new }
+    let(:claire) { Person.new }
+    let(:theo) { Person.new }
+    let(:vanessa) { Person.new }
+    let(:rudy) { Person.new }
+    let(:dress) { Clothing.new(slot_name: 'outerwear') }
+
+    it 'verifies complicated relationships' do
+      with_persisted_area(Family.new.name_of_area_to_assign) do |area|
+        setup_variables
+        verify_initial_relations_for_family
+        verify_initial_relations_for_theo
+        verify_initial_relations_for_claire
+        verify_initial_relations_for_dress
+        verify_claire_adding_a_child
+        verify_claire_losing_a_child
+        verify_rudy_losing_a_parent
+      end
+    end
+
+    protected
+    def setup_variables
+      family.save!
+
+      lawyer.save!
+      doctor.save!
+
+      heathcliff.save!
+      claire.save!
+      theo.save!
+      vanessa.save!
+      rudy.save!
+
+      dress.save!
+
+      claire.children = [theo,vanessa,rudy]
+      claire.jobs << lawyer
+      claire.families << family
+      claire.data << dress
+      claire.save!
+
+      heathcliff.children = [theo,vanessa,rudy]
+      heathcliff.jobs << doctor
+      heathcliff.families << family
+      heathcliff.save!
+
+      theo.families << family
+      theo.save!
+      vanessa.families << family
+      vanessa.save!
+      rudy.families << family
+      rudy.save!
+
+      family.family_members = [claire,heathcliff,theo,vanessa,rudy]
+      family.save!
+
+      @dress = dress.class.find(dress.pid)
+      @theo = theo.class.find(theo.pid)
+      @family = family.class.find(family.pid)
+      @claire = claire.class.find(claire.pid)
+      @rudy = rudy.class.find(rudy.pid)
+      @vanessa = vanessa.class.find(vanessa.pid)
+    end
+    def verify_initial_relations_for_family
+      # We are not storing the RELS-EXT entry on the "container" collection
+      assert_rels_ext(@family,:has_family_members,[])
+      assert_rels_ext(@family,:has_members,[])
+
+      # However, we do have access to Family#family_members
+      assert_active_fedora_has_many(
+        @family,
+        :family_members,
+        [@theo,@claire, heathcliff,rudy, @vanessa]
+      )
+
+      assert_active_fedora_has_many(
+        @family,
+        :child_collections,
+        [@theo,@claire, heathcliff,rudy, @vanessa]
+      )
+    end
+
+    def verify_initial_relations_for_theo
+      assert_rels_ext @theo, :is_child_of, [@claire, heathcliff]
+      assert_rels_ext @theo, :is_member_of, [@claire, family, heathcliff]
+
+      assert_active_fedora_has_many(@theo, :jobs, [])
+      assert_active_fedora_has_many(@theo, :families, [family])
+      assert_active_fedora_has_many(@theo, :parents, [heathcliff,claire])
+      assert_active_fedora_has_many(@theo, :child_collections, [])
+      assert_active_fedora_has_many(
+        @theo, :parent_collections, [heathcliff,claire,family]
+      )
+    end
+
+    def verify_initial_relations_for_dress
+      assert_rels_ext(@dress, :is_part_of, [@claire])
+      assert_active_fedora_belongs_to(@dress, :collection, @claire)
+    end
+
+    def verify_initial_relations_for_claire
+      assert_rels_ext @claire, :is_parent_of, []
+      assert_rels_ext @claire, :is_child_of, []
+      assert_rels_ext @claire, :is_family_member_of, [family]
+      assert_rels_ext @claire, :is_member_of, [family]
+
+      assert_active_fedora_has_many(@claire,:data,[@dress])
+      assert_active_fedora_has_many(@claire,:jobs,[lawyer])
+      assert_active_fedora_has_many(@claire,:families,[family])
+      assert_active_fedora_has_many(@claire,:children,[theo, rudy, @vanessa])
+      assert_active_fedora_has_many(
+        @claire,:child_collections,[theo, rudy, @vanessa]
+      )
+      assert_active_fedora_has_many(@claire,:parent_collections,[family])
+    end
+
+    def verify_claire_adding_a_child
+      @sandra = Person.new
+      @sandra.save!
+      @claire.children << @sandra
+      @claire.save!
+      @claire = @claire.class.find(@claire.pid)
+      @sandra = @sandra.class.find(@sandra.pid)
+
+      assert_rels_ext @claire, :is_parent_of, []
+      assert_rels_ext @claire, :is_child_of, []
+
+      assert_active_fedora_has_many(@claire,:parents,[])
+      assert_active_fedora_has_many(@claire,:children,[theo, rudy, @vanessa,@sandra])
+      assert_active_fedora_has_many(
+        @claire,:child_collections,[theo, rudy, @vanessa, @sandra]
+      )
+      assert_active_fedora_has_many(
+        @claire,:parent_collections,[family]
+      )
+    end
+
+    def verify_claire_losing_a_child
+      @vanessa.break_relation_with_parents(@claire)
+      @vanessa.save!
+      @sandra.break_relation_with_parents(@claire)
+      @sandra.save!
+
+      @claire.save!
+      @claire = @claire.class.find(@claire.pid)
+
+      assert_active_fedora_has_many(@claire,:children,[theo, rudy])
+      assert_active_fedora_has_many(
+        @claire,:child_collections,[theo, rudy]
+      )
+
+      @vanessa = @vanessa.class.find(@vanessa.pid)
+
+      # Note, just because we said Claire was not Vanessa's parent, Vanessa did
+      # not update
+      assert_rels_ext(@vanessa, :is_child_of, [heathcliff])
+      assert_rels_ext(@vanessa, :is_member_of, [heathcliff, family])
+
+      assert_active_fedora_has_many(@vanessa, :parents, [heathcliff])
+      assert_active_fedora_has_many(
+        @vanessa, :parent_collections, [heathcliff, family]
+      )
+
+    end
+
+    def verify_rudy_losing_a_parent
+      assert_rels_ext @rudy, :is_child_of, [heathcliff,@claire]
+      assert_rels_ext @rudy, :is_member_of, [heathcliff,@claire,family]
+      assert_active_fedora_has_many @rudy, :parents, [heathcliff,@claire]
+      assert_active_fedora_has_many(
+        @rudy, :parent_collections, [heathcliff,@claire,family]
+      )
+
+      @rudy.break_relation_with_parents(heathcliff)
+      @rudy.save!
+
+      assert_active_fedora_has_many(@rudy, :parents, [@claire])
+      assert_active_fedora_has_many(@rudy, :parent_collections, [@claire, family])
+      assert_rels_ext @rudy, :is_child_of, [@claire]
+      assert_rels_ext @rudy, :is_member_of, [@claire, family]
+
+      @rudy = @rudy.class.find(@rudy.pid)
+
+      assert_rels_ext @rudy, :is_child_of, [@claire]
+      assert_rels_ext @rudy, :is_member_of, [@claire, family]
+
+      assert_active_fedora_has_many(@rudy, :parents, [@claire])
+      assert_active_fedora_has_many(@rudy, :parent_collections, [@claire, family])
+    end
   end
 end

data/spec/config/predicate_mappings.yml

@@ -23,6 +23,7 @@
     :has_family_members: hasFamilyMembers
     :is_working_as: isWorkingAs
     :has_workers: hasWorkers
+    :is_volume_of: isVolumeOf
   info:fedora/fedora-system:def/relations-external#:
     :conforms_to: conformsTo
     :has_annotation: hasAnnotation

data/spec/factories/common_repository_model/collection_factory.rb

@@ -2,6 +2,10 @@ require_relative '../../../lib/common_repository_model/data'
 
 FactoryGirl.define do
   factory :common_repository_model_collection, class: CommonRepositoryModel::Collection do
-    area { CommonRepositoryModel::Area.all.first }
+    before(:create) { |collection|
+      area_name = collection.name_of_area_to_assign
+      CommonRepositoryModel::Area.find_by_name(area_name) ||
+      FactoryGirl.create(:common_repository_model_area, name: area_name)
+    }
   end
 end

data/spec/support/spec_models.rb

@@ -0,0 +1,68 @@
+require 'common_repository_model/collection'
+
+class JournalVolume < CommonRepositoryModel::Collection
+  def name_of_area_to_assign
+    Journal::AREA_NAME.freeze
+  end
+
+  is_member_of :journals, class_name: "Journal", property: :is_volume_of
+end
+
+class Journal < CommonRepositoryModel::Collection
+  AREA_NAME ='NDLIB-LAW'.freeze
+  def name_of_area_to_assign
+    AREA_NAME
+  end
+
+  has_members :volumes, class_name: "JournalVolume", property: :is_volume_of
+end
+
+class Clothing < CommonRepositoryModel::Data
+end
+
+class Job < CommonRepositoryModel::Collection
+  has_and_belongs_to_many(
+    :people,
+    class_name: 'Person',
+    property: :has_workers,
+    inverse_of: :is_working_as
+  )
+end
+
+class Family < CommonRepositoryModel::Collection
+  has_members(
+    :family_members,
+    class_name: 'Person',
+    property: :is_family_member_of
+  )
+end
+
+class Person < CommonRepositoryModel::Collection
+
+  is_member_of(
+    :parents,
+    class_name: 'Person',
+    property: :is_child_of
+  )
+
+  has_members(
+    :children,
+    class_name: 'Person',
+    property: :is_child_of
+  )
+
+  # People can be part of multiple families
+  is_member_of(
+    :families,
+    class_name: 'Family',
+    property: :is_family_member_of
+  )
+
+  has_and_belongs_to_many(
+    :jobs,
+    class_name: 'Job',
+    property: :is_working_as,
+    inverse_of: :has_workers
+  )
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: common_repository_model
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.9
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-28 00:00:00.000000000 Z
+date: 2012-11-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -59,22 +59,6 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: '3.0'
-- !ruby/object:Gem::Dependency
-  name: morphine
-  requirement: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: active_model_serializers
   requirement: !ruby/object:Gem::Requirement
@@ -210,6 +194,7 @@ files:
 - spec/factories/common_repository_model/collection_factory.rb
 - spec/factories/common_repository_model/data_factory.rb
 - spec/spec_helper.rb
+- spec/support/spec_models.rb
 homepage: https://github.com/ndlib/common_repository_model
 licenses: []
 post_install_message: 
@@ -253,3 +238,4 @@ test_files:
 - spec/factories/common_repository_model/collection_factory.rb
 - spec/factories/common_repository_model/data_factory.rb
 - spec/spec_helper.rb
+- spec/support/spec_models.rb