active-fedora 6.8.0 → 7.0.0.pre1

data/spec/integration/delete_all_spec.rb

@@ -1,56 +1,63 @@
 require 'spec_helper'
 
-module SpecModelD
-  class Basic < ActiveFedora::Base
-    class_attribute :callback_counter
-    before_destroy :inc_counter
-
-    def inc_counter
-      self.class.callback_counter += 1
+describe ActiveFedora::Base do
+  
+  before(:all) do
+    module SpecModel
+      class Basic < ActiveFedora::Base
+        class_attribute :callback_counter
+        
+        before_destroy :inc_counter
+
+        def inc_counter
+          self.class.callback_counter += 1
+        end
+      end
     end
   end
-end
-
-describe ActiveFedora::Base do
+  
+  after(:all) do
+    Object.send(:remove_const, :SpecModel)
+  end
 
-  let!(:model1) { SpecModelD::Basic.create! }
-  let!(:model2) { SpecModelD::Basic.create! }
+  let!(:model1) { SpecModel::Basic.create! }
+  let!(:model2) { SpecModel::Basic.create! }
 
-  before :each do
-    SpecModelD::Basic.callback_counter = 0
+  before do
+    SpecModel::Basic.callback_counter = 0
   end
 
-  describe '.destroy_all' do
-    it 'should remove both and run callbacks' do
-      model1
-      model2
-      expect(SpecModelD::Basic.count).to eq(2)
-      expect(SpecModelD::Basic.callback_counter).to eq(0)
-      SpecModelD::Basic.destroy_all
-      expect(SpecModelD::Basic.count).to eq(0)
-      expect(SpecModelD::Basic.callback_counter).to eq(2)
+
+  describe ".destroy_all" do
+    it "should remove both and run callbacks" do 
+      SpecModel::Basic.destroy_all
+      SpecModel::Basic.count.should == 0 
+      SpecModel::Basic.callback_counter.should == 2
     end
 
-    describe 'when a model is missing' do
-      let(:model3) { SpecModelD::Basic.create! }
-      after { model3.delete }
-      it 'should be able to skip a missing model' do
-        expect(model1).to receive(:destroy).and_call_original
-        expect(model2).to receive(:destroy).and_call_original
-        expect(model3).to receive(:destroy).and_raise(ActiveFedora::ObjectNotFoundError)
-        expect_any_instance_of(ActiveFedora::Relation).to receive(:to_a).and_return([model1, model3, model2])
-        expect(ActiveFedora::Relation.logger).to receive(:error).with("When trying to destroy #{model3.pid}, encountered an ObjectNotFoundError. Solr may be out of sync with Fedora")
-        SpecModelD::Basic.destroy_all
-        expect(SpecModelD::Basic.count).to eq(1)
+    describe "when a model is missing" do
+      let(:model3) { SpecModel::Basic.create! }
+      let!(:pid) { model3.pid }
+      before { model3.inner_object.delete }
+      after do
+        ActiveFedora::SolrService.instance.conn.tap do |conn|
+          conn.delete_by_query "id:\"#{pid}\""
+          conn.commit
+        end
+      end
+      it "should be able to skip a missing model" do 
+        ActiveFedora::Relation.logger.should_receive(:error).with("Although #{pid} was found in Solr, it doesn't seem to exist in Fedora. The index is out of synch.")
+        SpecModel::Basic.destroy_all
+        SpecModel::Basic.count.should == 1 
       end
     end
   end
 
-  describe '.delete_all' do
-    it 'should remove both and not run callbacks' do
-      SpecModelD::Basic.delete_all
-      expect(SpecModelD::Basic.count).to eq(0)
-      expect(SpecModelD::Basic.callback_counter).to eq(0)
+  describe ".delete_all" do
+    it "should remove both and not run callbacks" do 
+      SpecModel::Basic.delete_all
+      SpecModel::Basic.count.should == 0
+      SpecModel::Basic.callback_counter.should == 0
     end
   end
 end

data/spec/integration/fedora_solr_sync_spec.rb

@@ -1,14 +1,14 @@
 require 'spec_helper'
 require 'timeout'
 
-describe 'fedora_solr_sync_issues' do
+describe "fedora_solr_sync_issues" do
   before :all do
     class ParentThing < ActiveFedora::Base
-      has_many :things, :class_name => 'ChildThing', :property => :is_part_of
+      has_many :things, :class_name=>'ChildThing', :property=>:is_part_of
     end
 
     class ChildThing < ActiveFedora::Base
-      belongs_to :parent, :class_name => 'ParentThing', :property => :is_part_of
+      belongs_to :parent, :class_name=>'ParentThing', :property=>:is_part_of
     end
   end
 
@@ -20,9 +20,9 @@ describe 'fedora_solr_sync_issues' do
   let(:parent) { ParentThing.create }
   subject { ChildThing.create :parent => parent }
 
-  it 'should not go into an infinite loop' do
+  it "should not go into an infinite loop" do
     subject.inner_object.delete
     parent.reload
-    expect(parent.things).to eq([])
+    parent.things.should == []
   end
 end

data/spec/integration/field_to_solr_name_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe "An object with RDF backed attributes" do
+
+  before do
+    class TestOne < ActiveFedora::Base
+      class MyMetadata < ActiveFedora::NtriplesRDFDatastream
+        map_predicates do |map|
+          map.title(in: RDF::DC) do |index|
+            index.as :stored_searchable
+          end
+          map.date_uploaded(to: "dateSubmitted", in: RDF::DC) do |index|
+            index.type :date
+            index.as :stored_searchable, :sortable
+          end
+        end
+      end
+      has_metadata 'descMetadata', type: MyMetadata
+      has_attributes :title, :date_uploaded, datastream: 'descMetadata'
+    end
+  end
+
+  after do 
+    Object.send(:remove_const, :TestOne)
+  end
+
+  it "should be able to grab the solr name" do
+    expect(TestOne.defined_attributes[:title].primary_solr_name).to eq 'desc_metadata__title_tesim'
+  end
+
+  it "should be able to grab the solr name for a date" do
+    expect(TestOne.defined_attributes[:date_uploaded].primary_solr_name).to eq 'desc_metadata__date_uploaded_dtsim'
+  end
+end

data/spec/integration/full_featured_model_spec.rb

@@ -9,24 +9,24 @@ describe ActiveFedora::Base do
     class OralHistory < ActiveFedora::Base
         # These are all the properties that don't quite fit into Qualified DC
         # Put them on the object itself (in the properties datastream) for now.
-        has_metadata :name => 'properties', :type => ActiveFedora::SimpleDatastream do |m|
-          m.field 'narrator',  :string
-          m.field 'interviewer', :string
-          m.field 'transcript_editor', :text
-          m.field 'bio', :string
-          m.field 'notes', :text
-          m.field 'hard_copy_availability', :text
-          m.field 'hard_copy_location', :text
-          m.field 'other_contributor', :string
-          m.field 'restrictions', :text
-          m.field 'series', :string
-          m.field 'location', :string
+        has_metadata :name => "properties", :type => ActiveFedora::SimpleDatastream do |m|
+          m.field "narrator",  :string
+          m.field "interviewer", :string
+          m.field "transcript_editor", :text
+          m.field "bio", :string
+          m.field "notes", :text
+          m.field "hard_copy_availability", :text
+          m.field "hard_copy_location", :text
+          m.field "other_contributor", :string
+          m.field "restrictions", :text
+          m.field "series", :string
+          m.field "location", :string
         end
-
-
-        has_metadata :name => 'mods_article', :type => Hydra::ModsArticleDatastream
-
-        has_metadata :name => 'dublin_core', :type => ActiveFedora::QualifiedDublinCoreDatastream do |m|
+        
+        
+        has_metadata :name=>"mods_article", :type=> Hydra::ModsArticleDatastream 
+        
+        has_metadata :name => "dublin_core", :type => ActiveFedora::QualifiedDublinCoreDatastream do |m|
           # Default :multiple => true
           #
           # on retrieval, these will be pluralized and returned as arrays
@@ -34,30 +34,30 @@ describe ActiveFedora::Base do
           #
           # aimint to use method-missing to support calling methods like
           # my_oral_history.subjects  OR   my_oral_history.titles  OR EVEN my_oral_history.title whenever possible
-
+          
           # Setting new Types for dates and text content
           #m.field "creation_date", :date, :xml_node => "date"
           #m.field "abstract", :text, :xml_node => "abstract"
           #m.field "rights", :text, :xml_node => "rights"
-
+          
           # Setting up special named fields
-          #m.field "subject_heading", :string, :xml_node => "subject", :encoding => "LCSH"
+          #m.field "subject_heading", :string, :xml_node => "subject", :encoding => "LCSH" 
           #m.field "spatial_coverage", :string, :xml_node => "spatial", :encoding => "TGN"
           #m.field "temporal_coverage", :string, :xml_node => "temporal", :encoding => "Period"
           #m.field "type", :string, :xml_node => "type", :encoding => "DCMITYPE"
           #m.field "alt_title", :string, :xml_node => "alternative"
         end
-
-        has_metadata :name => 'significant_passages', :type => ActiveFedora::SimpleDatastream do |m|
-          m.field 'significant_passage', :text
+        
+        has_metadata :name => "significant_passages", :type => ActiveFedora::SimpleDatastream do |m|
+          m.field "significant_passage", :text
         end
-
-        has_metadata :name => 'sensitive_passages', :type => ActiveFedora::SimpleDatastream do |m|
-          m.field 'sensitive_passage', :text
+        
+        has_metadata :name => "sensitive_passages", :type => ActiveFedora::SimpleDatastream do |m|
+          m.field "sensitive_passage", :text
         end
 
     end
-    sample_location = 'Boston, Massachusetts'
+    sample_location = "Boston, Massachusetts"
     sample_notes = 'Addelson, Frances. (1973?) "The Induced Abortion," American Journal of Ortho-Psychiatry,  Addelson, Frances. "Abortion: Source of Guilt or Growth," National Journal of Gynecology and Obstetrics., Addelson, Frances. "First Zionist Novel," Jewish Frontier.'
     sample_other_contributor = 'any other contributors, people or corporate names (eg. Temple Israel)'
     sample_transcript_editor = 'Siegel, Cheryl'
@@ -68,113 +68,112 @@ describe ActiveFedora::Base do
     sample_bio = <<-END
     Rochelle Ruthchild interviewed Frances Addleson on October 18, November 14, and December 10, 1997.   The interview thoroughly examined the trajectory of Frances\' life from birth until the time of the interview. As a young child, Frances\' father died during the influenza epidemic, and her mother was not equipped to care for her and her siblings.  Consequently, they were placed in a Jewish foster home. Although her experience was mostly positive, this experience would leave life-long effects.  Frances attended Radcliffe upon the urging of a mentor and later obtained her Master\'s degree in social work from Simmons College in 1954.  In the 1940\'s, she returned to work while her children were still young; a rather unusual event for that time period.  While working as a social worker at Beth Israel Hospital in the early 1970\'s, she helped counsel countless women who came to the hospital seeking abortions before the procedure was officially legalized during the landmark Roe vs. Wade decision in 1973.  Frances would later write two articles that were published in medical journals about her experience during this time.  Although not a very religious person, Frances felt connected to the Jewish notion of social justice and remained very active until an accident in the late 1990\'s.
     END
-    sample_interviewer = 'Ruthchild, & Rochelle'
-
-    @properties_sample_values2 = Hash[:narrator => 'Narrator1 & Narrator2', :interviewer => 'Interviewer', :transcript_editor => 'Transcript Editor', :bio => 'Biographic info',
-                                    :notes => "My Note\\\'s a good one", :hard_copy_availability => 'Yes', :hard_copy_location => 'Archives', :other_contributor => 'Sally Ride',
-                                    :restrictions => 'None', :series => 'My Series', :location => 'location']
-
-    @properties_sample_values = Hash[:narrator => sample_narrator, :interviewer => sample_interviewer, :transcript_editor => sample_transcript_editor, :bio => sample_bio,
-                                    :notes => sample_notes, :hard_copy_availability => sample_hard_copy_availability, :hard_copy_location => 'Archives', :other_contributor => sample_other_contributor,
-                                    :restrictions => 'None', :series => 'My Series', :location => sample_location]
-
-    @dublin_core_sample_values = Hash[:creator => 'Matt && McClain', :publisher => "Jewish Womens's Archive", :description => 'description', :identifier => 'jwa:sample_pid',
-                                    :title => 'title',
-                                    #:alt_title => "alt_title", :subject => "subject",
-                                    #:subject_heading => "subject heading",
-                                    #:creation_date => "2008-07-02T05:09:42.015Z",
-                                    :language => 'language',
-                                    #:spatial_coverage => "spatial coverage",
-                                    #:temporal_coverage => "temporal coverage",
+    sample_interviewer = "Ruthchild, & Rochelle"
+    
+    @properties_sample_values2 = Hash[:narrator => "Narrator1 & Narrator2", :interviewer => "Interviewer", :transcript_editor => "Transcript Editor", :bio => "Biographic info", 
+                                    :notes => "My Note\\\'s a good one", :hard_copy_availability => "Yes", :hard_copy_location => "Archives", :other_contributor => "Sally Ride", 
+                                    :restrictions => "None", :series => "My Series", :location => "location"]
+    
+    @properties_sample_values = Hash[:narrator => sample_narrator, :interviewer => sample_interviewer, :transcript_editor => sample_transcript_editor, :bio => sample_bio, 
+                                    :notes => sample_notes, :hard_copy_availability => sample_hard_copy_availability, :hard_copy_location => "Archives", :other_contributor => sample_other_contributor, 
+                                    :restrictions => "None", :series => "My Series", :location => sample_location]
+
+    @dublin_core_sample_values = Hash[:creator => 'Matt && McClain', :publisher => "Jewish Womens's Archive", :description => "description", :identifier => "jwa:sample_pid", 
+                                    :title => "title", 
+                                    #:alt_title => "alt_title", :subject => "subject", 
+                                    #:subject_heading => "subject heading", 
+                                    #:creation_date => "2008-07-02T05:09:42.015Z", 
+                                    :language => "language", 
+                                    #:spatial_coverage => "spatial coverage", 
+                                    #:temporal_coverage => "temporal coverage", 
                                     #:abstract => "abstract",
-                                    :rights => 'rights', :type => 'type',
-                                    #:extent => "extent",
-                                    :format => 'format', :medium => 'medium']
+                                    :rights => "rights", :type => "type",
+                                    #:extent => "extent", 
+                                    :format => "format", :medium => "medium"]
     @signigicant_passages_sample_values = {}
-    @sensitive_passages_sample_values = {}
-    @sample_xml = '<xml><fields><system_create_date>REMOVED</system_create_date><system_modified_date>REMOVED</system_modified_date><active_fedora_model_s>OralHistory</active_fedora_model_s><id>changeme:14527</id><subject_heading>subject heading</subject_heading><type>type</type><rights>rights</rights><publisher>publisher</publisher><creation_date>creation date</creation_date><identifier>jwa:sample_pid</identifier><format>format</format><extent>extent</extent><language>language</language><description>description</description><title>title</title><medium>medium</medium><spatial_coverage>spatial coverage</spatial_coverage><alt_title>alt_title</alt_title><temporal_coverage>temporal coverage</temporal_coverage><subject>subject</subject><creator>creator</creator><abstract>abstract</abstract><other_contributor>Sally Ride</other_contributor><transcript_editor>Transcript Editor</transcript_editor><restrictions>None</restrictions><bio>Biographic info</bio><series>My Series</series><notes>My Note</notes><location>location</location><hard_copy_availability>Yes</hard_copy_availability><narrator>Narrator</narrator><hard_copy_location>Archives</hard_copy_location><interviewer>Interviewer</interviewer></fields><content/></xml>'
-
-  end
-
+    @sensitive_passages_sample_values = {}     
+    @sample_xml = "<xml><fields><system_create_date>REMOVED</system_create_date><system_modified_date>REMOVED</system_modified_date><active_fedora_model_s>OralHistory</active_fedora_model_s><id>changeme:14527</id><subject_heading>subject heading</subject_heading><type>type</type><rights>rights</rights><publisher>publisher</publisher><creation_date>creation date</creation_date><identifier>jwa:sample_pid</identifier><format>format</format><extent>extent</extent><language>language</language><description>description</description><title>title</title><medium>medium</medium><spatial_coverage>spatial coverage</spatial_coverage><alt_title>alt_title</alt_title><temporal_coverage>temporal coverage</temporal_coverage><subject>subject</subject><creator>creator</creator><abstract>abstract</abstract><other_contributor>Sally Ride</other_contributor><transcript_editor>Transcript Editor</transcript_editor><restrictions>None</restrictions><bio>Biographic info</bio><series>My Series</series><notes>My Note</notes><location>location</location><hard_copy_availability>Yes</hard_copy_availability><narrator>Narrator</narrator><hard_copy_location>Archives</hard_copy_location><interviewer>Interviewer</interviewer></fields><content/></xml>"
+               
+  end  
+  
   before(:each) do
     @test_history = OralHistory.new
   end
-
+  
   after(:each) do
   end
-
+  
   after(:all) do
     Object.send(:remove_const, :OralHistory)
   end
-
-  it 'should be an instance of ActiveFedora::Base' do
-    expect(@test_history).to be_kind_of(ActiveFedora::Base)
-  end
-
-
-  it 'should create proxies to all the datastreams' do
-    properties_ds = @test_history.datastreams['properties']
-    dublin_core_ds = @test_history.datastreams['dublin_core']
-    expect(@test_history).not_to respond_to(:properties)
-    expect(@test_history.properties).to be properties_ds
-    expect(@test_history).to respond_to(:properties)
-    expect(OralHistory.new).to respond_to(:properties)
+  
+  it "should be an instance of ActiveFedora::Base" do
+    @test_history.should be_kind_of(ActiveFedora::Base)
   end
-
-
-  it 'should push all of the metadata fields into solr' do
+  
+
+  it "should create proxies to all the datastreams" do
+    properties_ds = @test_history.datastreams["properties"]
+    dublin_core_ds = @test_history.datastreams["dublin_core"]
+    @test_history.properties.should be properties_ds    
+    @test_history.should respond_to(:properties)
+    OralHistory.new.should respond_to(:properties)
+  end 
+
+    
+  it "should push all of the metadata fields into solr" do
     # TODO: test must test values using solr symbol names (ie. _field, _text and _date)
-    properties_ds = @test_history.datastreams['properties']
-    dublin_core_ds = @test_history.datastreams['dublin_core']
-
+    properties_ds = @test_history.datastreams["properties"]
+    dublin_core_ds = @test_history.datastreams["dublin_core"]
+        
     @properties_sample_values.each_pair do |field, value|
       next if field == :hard_copy_availability #FIXME HYDRA-824
       properties_ds.send("#{field.to_s}=", [value])
     end
-
+    
     @dublin_core_sample_values.each_pair do |field, value|
       next if [:format, :type].include?(field)  #format and type are methods declared on Object
       dublin_core_ds.send("#{field.to_s}=", [value])
     end
-
+    
     @test_history.save
-
-    @solr_result = OralHistory.find_with_conditions(:id => @test_history.pid)[0]
+    
+    @solr_result = OralHistory.find_with_conditions(:id=>@test_history.pid)[0]
     @properties_sample_values.each_pair do |field, value|
       next if field == :hard_copy_availability #FIXME HYDRA-824
-      next if field == :location #FIXME HYDRA-825
-      expect(@solr_result[ActiveFedora::SolrService.solr_name(field, type: :string)] || @solr_result[ActiveFedora::SolrService.solr_name(field, type: :date)]).to eq([::Solrizer::Extractor.format_node_value(value)])
+      next if field == :location #FIXME HYDRA-825 
+      (@solr_result[ActiveFedora::SolrService.solr_name(field, type: :string)] || @solr_result[ActiveFedora::SolrService.solr_name(field, type: :date)]).should == [::Solrizer::Extractor.format_node_value(value)] 
     end
-
+    
     @dublin_core_sample_values.each_pair do |field, value|
       next if [:format, :type].include?(field)  #format and type are methods declared on Object
-      expect(dublin_core_ds.send("#{field.to_s}")).to eq([value])
-    end
+      dublin_core_ds.send("#{field.to_s}").should == [value]
+    end    
   end
-
-  it 'should have Qualified Dublin core, with custom accessors' do
-
-    dublin_core_ds = @test_history.datastreams['dublin_core']
-
-    dublin_core_ds.subject = 'My Subject Heading'
+  
+  it "should have Qualified Dublin core, with custom accessors" do
+    
+    dublin_core_ds = @test_history.datastreams["dublin_core"]
+    
+    dublin_core_ds.subject = "My Subject Heading"
     dc_xml = REXML::Document.new(dublin_core_ds.to_xml)
-
-    expect(dc_xml.root.elements['dcterms:subject'].text).to eq('My Subject Heading')
-
+    
+    dc_xml.root.elements["dcterms:subject"].text.should == "My Subject Heading"
+    
   end
-
-  it 'should support #find_with_conditions' do
+  
+  it "should support #find_with_conditions" do
     solr_result = OralHistory.find_with_conditions({})
-    expect(solr_result).not_to be_nil
+    solr_result.should_not be_nil
   end
-
+  
   describe '#new' do
-    it 'should support custom pids' do
-      oh = OralHistory.new(:pid => 'uuid:blah-blah-blah')
-      expect(oh.pid).to eq('uuid:blah-blah-blah')
+    it "should support custom pids" do
+      oh = OralHistory.new(:pid=>"uuid:blah-blah-blah")
+      oh.pid.should == "uuid:blah-blah-blah"
     end
   end
+  
 
-
-
+  
 end

data/spec/integration/has_and_belongs_to_many_associations_spec.rb

@@ -0,0 +1,341 @@
+require 'spec_helper'
+
+describe ActiveFedora::Base do
+  describe "with inverse" do
+    before do
+      class Book < ActiveFedora::Base 
+        has_and_belongs_to_many :topics, :property=>:has_topic, :inverse_of=>:is_topic_of
+        has_and_belongs_to_many :collections, :property=>:is_member_of_collection
+      end
+
+      class SpecialInheritedBook < Book
+      end
+
+
+      class Collection < ActiveFedora::Base
+      end
+
+      class Topic < ActiveFedora::Base 
+        has_and_belongs_to_many :books, :property=>:is_topic_of
+      end
+    end
+
+    after do
+      Object.send(:remove_const, :SpecialInheritedBook)
+      Object.send(:remove_const, :Book)
+      Object.send(:remove_const, :Collection)
+      Object.send(:remove_const, :Topic)
+    end
+
+    describe "an unsaved instance" do
+      before do
+        @topic1 = Topic.create
+        @topic2 = Topic.create
+        @book = Book.create
+        @special_book = SpecialInheritedBook.create #TODO this isnt' needed in every test.
+      end
+      it "habtm should set and remove relationships bidirectionally" do
+        @book.topics << @topic1
+        @book.topics.should == [@topic1]
+        @topic1.books.should == [@book]
+        @topic1.reload.books.should == [@book]
+
+        @book.topics.delete(@topic1)
+        #@topic1.books.delete(@book)
+        @book.topics.should == []
+        @topic1.books.should == []
+      end
+      it "Should allow for more than 10 items" do
+
+        (0..11).each do
+          @book.topics << Topic.create
+        end
+        @book.save
+        @book.topics.count.should == 12
+        book2 = Book.find(@book.pid)
+        book2.topics.count.should == 12
+      end
+
+      it "Should find inherited objects along with base objects" do
+        @book.topics << @topic1
+        @special_book.topics << @topic1
+        @topic1.books.should == [@book, @special_book]
+        @topic1.reload.books.should == [@book, @special_book]
+      end
+
+      it "Should cast found books to the correct cmodel" do
+        @topic1.books[0].class == Book
+        @topic1.books[1].class == SpecialInheritedBook
+      end
+
+      after do
+        @topic1.delete
+        @topic2.delete
+        @book.delete
+        @special_book.delete
+      end
+    end
+    describe "a saved instance" do
+      before do
+        @book = Book.create
+        @topic1 = Topic.create
+        @topic2 = Topic.create
+      end
+      it "should set relationships bidirectionally" do
+        @book.topics << @topic1
+        @book.topics.should == [@topic1]
+        @book.relationships(:has_topic).should == [@topic1.internal_uri]
+        @topic1.relationships(:has_topic).should == []
+        @topic1.relationships(:is_topic_of).should == [@book.internal_uri]
+        Topic.find(@topic1.pid).books.should == [@book] #Can't have saved it because @book isn't saved yet.
+      end
+      it "should save new child objects" do
+        @book.topics << Topic.new
+        @book.topics.first.pid.should_not be_nil
+      end
+      it "should clear out the old associtions" do
+        @book.topics = [@topic1]
+        @book.topics = [@topic2]
+        @book.topic_ids.should == [@topic2.pid]
+      end
+      after do
+        @book.delete
+        @topic1.delete
+        @topic2.delete
+      end
+    end
+  end
+
+  describe "when inverse is not specified" do
+    before do
+      class Book < ActiveFedora::Base 
+        has_and_belongs_to_many :collections, :property=>:is_member_of_collection
+      end
+
+      class Collection < ActiveFedora::Base
+      end
+    end
+
+    after do
+      Object.send(:remove_const, :Book)
+      Object.send(:remove_const, :Collection)
+    end
+
+    let (:book) { Book.create }
+    let (:collection) { Collection.create }
+    before do
+      book.collections << collection
+      book.save!
+    end
+    after do
+      collection.delete
+      book.delete
+    end
+    it "should have a collection" do
+      book.relationships(:is_member_of_collection).should == [collection.internal_uri]
+      book.collections.should == [collection]
+    end
+    it "habtm should not set foreign relationships if :inverse_of is not specified" do
+       collection.relationships(:is_member_of_collection).should == []
+    end
+    it "should load the collections" do
+      reloaded = Book.find(book.pid)
+      reloaded.collections.should == [collection]
+    end
+  end
+
+
+  describe "when destroying the association" do
+    describe "without callbacks" do
+      before do
+        class Book < ActiveFedora::Base 
+          has_and_belongs_to_many :collections, property: :is_member_of_collection
+        end
+
+        class Collection < ActiveFedora::Base
+        end
+      end
+
+      after do
+        Object.send(:remove_const, :Book)
+        Object.send(:remove_const, :Collection)
+      end
+
+      let (:book) { Book.create }
+      let (:collection1) { Collection.create }
+      let (:collection2) { Collection.create }
+      before do
+        book.collections << collection1 << collection2
+        book.save!
+      end
+      after do
+        collection1.delete
+        collection2.delete
+        book.delete
+      end
+
+      it "delete should cause the entries to be removed from RELS-EXT, but not destroy the original record" do
+        book.collections.should == [collection1, collection2]
+        book.collections.delete(collection1)
+        book.collections.should == [collection2]
+        book.save!
+        book.reload
+        book.collections.should == [collection2]
+        expect {Collection.find(collection1.pid)}.to_not be_nil
+      end
+
+      it "destroy should cause the entries to be removed from RELS-EXT, but not destroy the original record" do
+        book.collections.should == [collection1, collection2]
+        book.collections.destroy(collection1)
+        book.collections.should == [collection2]
+        book.save!
+        book.reload
+        book.collections.should == [collection2]
+        expect {Collection.find(collection1.pid)}.to_not be_nil
+      end
+    end
+
+    describe "with remove callbacks" do
+      before do
+        class Book < ActiveFedora::Base 
+          has_and_belongs_to_many :collections, 
+                                  property: :is_member_of_collection,
+                                  before_remove: :foo, after_remove: :bar
+        end
+
+        class Collection < ActiveFedora::Base
+        end
+      end
+
+      after do
+        Object.send(:remove_const, :Book)
+        Object.send(:remove_const, :Collection)
+      end
+
+      let (:book) { Book.create }
+      let (:collection) { Collection.create }
+      before do
+        book.collections << collection
+        book.save!
+      end
+      after do
+        collection.delete
+        book.delete
+      end
+
+      it "destroy should cause the before_remove and after_remove callback to be triggered" do
+        book.should_receive(:foo).with(collection)
+        book.should_receive(:bar).with(collection)
+        book.collections.destroy(collection)
+      end
+
+      it "delete should cause the before_remove and after_remove callback to be triggered" do
+        book.should_receive(:foo).with(collection)
+        book.should_receive(:bar).with(collection)
+        book.collections.delete(collection)
+      end
+
+      it "should not remove if an exception is thrown in before_remove" do
+        book.should_receive(:foo).with(collection).and_raise
+        book.should_not_receive(:bar)
+        begin
+          book.collections.delete(collection)
+        rescue RuntimeError
+        end
+        book.collections.should == [collection]
+      end
+    end
+
+    describe "with add callbacks" do
+      before do
+        class Book < ActiveFedora::Base 
+          has_and_belongs_to_many :collections, 
+                                  property: :is_member_of_collection,
+                                  before_add: :foo, after_add: :bar
+        end
+
+        class Collection < ActiveFedora::Base
+        end
+      end
+
+      after do
+        Object.send(:remove_const, :Book)
+        Object.send(:remove_const, :Collection)
+      end
+
+      let (:book) { Book.create }
+      let (:collection) { Collection.create }
+      after do
+        collection.delete
+        book.delete
+      end
+
+      it "shift should cause the before_add and after_add callback to be triggered" do
+        book.should_receive(:foo).with(collection)
+        book.should_receive(:bar).with(collection)
+        book.collections << collection
+      end
+
+      it "assignment should cause the before_add and after_add callback to be triggered" do
+        book.should_receive(:foo).with(collection)
+        book.should_receive(:bar).with(collection)
+        book.collections = [collection]
+      end
+
+      it "should not add if an exception is thrown in before_add" do
+        book.should_receive(:foo).with(collection).and_raise
+        book.should_not_receive(:bar)
+        begin
+          book.collections << collection
+        rescue RuntimeError
+        end
+        book.collections.should == []
+      end
+    end
+
+  end
+
+end
+
+describe "Autosave" do
+  before do
+    class Item < ActiveFedora::Base
+      has_many :components
+      has_metadata "foo", type: ActiveFedora::SimpleDatastream do |m|
+        m.field "title", :string
+      end
+      has_attributes :title, datastream: 'foo'
+    end
+    class Component < ActiveFedora::Base
+      has_and_belongs_to_many :items, :property => :is_part_of
+      has_metadata "foo", type: ActiveFedora::SimpleDatastream do |m|
+        m.field "description", :string
+      end
+      has_attributes :description, datastream: 'foo'
+    end
+  end
+
+  after do
+      Object.send(:remove_const, :Item)
+      Object.send(:remove_const, :Component)
+  end
+
+  describe "From the has_and_belongs_to_many side" do
+    let(:component) { Component.create(items: [Item.new(title: 'my title')]) }
+
+    it "should save dependent records" do
+      component.reload
+      component.items.first.title.should == 'my title'
+    end
+  end
+  describe "From the has_many side" do
+    let(:item) { Item.create(components: [Component.new(description: 'my description')]) }
+
+    it "should save dependent records" do
+      item.reload
+      item.components.first.description.should == 'my description'
+    end
+  end
+
+end
+