active-fedora 5.7.1 → 6.0.0.pre1

data/spec/integration/base_file_management_spec.rb

@@ -1,24 +0,0 @@
-require 'spec_helper'
-
-describe ActiveFedora::Base do
-  
-  before(:each) do
-    class  FileMgmt < ActiveFedora::Base
-      include ActiveFedora::FileManagement
-    end
-    @test_container = FileMgmt.new
-    @test_container.add_relationship(:has_collection_member, "info:fedora/foo:2")
-    @test_container.save
-  end
-  
-  after(:each) do
-    @test_container.delete
-    Object.send(:remove_const, :FileMgmt)
-  end
-  
-  it "should persist and re-load collection members" do
-    container_copy = FileMgmt.find(@test_container.pid)
-    container_copy.collection_members(:response_format=>:id_array).should == ["foo:2"]
-  end
-    
-end

data/spec/integration/has_many_associations_spec.rb

@@ -1,64 +0,0 @@
-require 'spec_helper'
-
-describe "When two or more relationships share the same property" do 
-  before do
-    class Book < ActiveFedora::Base 
-      has_many :collections, :property=>:is_part_of, :class_name=>'Collection'
-      has_many :people, :property=>:is_part_of
-    end
-
-    class Person < ActiveFedora::Base
-      belongs_to :book, :property=>:is_part_of
-    end
-
-    class Collection < ActiveFedora::Base
-      belongs_to :book, :property=>:is_part_of
-    end
-
-    @book = Book.create!
-    @person1 = Person.create!(:book=>@book)
-    @person2 = Person.create!(:book=>@book)
-  end
-  after do
-      Object.send(:remove_const, :Collection)
-      Object.send(:remove_const, :Person)
-      Object.send(:remove_const, :Book)
-  end
-
-  it "Should only return relationships of the correct class" do
-    @book.reload
-    @book.people.should == [@person1, @person2]
-    @book.collections.should == []
-  end
-end
-
-describe "When relationship is restricted to AF::Base" do
-  before do
-    class Email < ActiveFedora::Base 
-      has_many :attachments, :property=>:is_part_of, :class_name=>'ActiveFedora::Base'
-    end
-
-    class Image < ActiveFedora::Base
-      belongs_to :email, :property=>:is_part_of
-    end
-
-    class PDF < ActiveFedora::Base
-      belongs_to :email, :property=>:is_part_of
-    end
-
-    @book = Email.create!
-    @image = Image.create!(:email=>@book)
-    @pdf = PDF.create!(:email=>@book)
-  end
-  after do
-      Object.send(:remove_const, :Image)
-      Object.send(:remove_const, :PDF)
-      Object.send(:remove_const, :Email)
-  end
-
-
-  it "Should not restrict relationships " do
-    @book.reload
-    @book.attachments.should == [@image, @pdf]
-  end
-end

data/spec/integration/metadata_datastream_helper_spec.rb

@@ -1,100 +0,0 @@
-require 'spec_helper'
-
-require 'active-fedora'
-require "rexml/document"
-
-class MockMetaHelperSolr < ActiveFedora::Base
-  has_metadata :name => "properties", :type => ActiveFedora::SimpleDatastream do |m|
-    m.field "holding_id", :string
-  end
-  
-  has_metadata :name => "descMetadata", :type => ActiveFedora::QualifiedDublinCoreDatastream do |m|
-    m.field "created", :date, :xml_node => "created"
-    m.field "language", :string, :xml_node => "language"
-    m.field "creator", :string, :xml_node => "creator"
-    # Created remaining fields
-    m.field "geography", :string, :xml_node => "geography"
-    m.field "title", :string, :xml_node => "title"
-  end
-
-end
-
-describe ActiveFedora::MetadataDatastreamHelper do
-  
-  before(:each) do
-    @test_object = ActiveFedora::Base.new
-    @test_object.save
-  end
-  
-  after(:each) do
-    begin
-    @test_object.delete
-    rescue
-    end
-    begin
-    @test_object2.delete
-    rescue
-    end
-    begin
-    @test_object3.delete
-    rescue
-    end
-  end
-  
-  describe '#from_solr' do
-    it 'should return an object with the appropriate metadata fields filled in' do
-      @test_object = MockMetaHelperSolr.new
-      attributes = {"holding_id"=>{0=>"Holding 1"},
-                    "language" =>{0=>"Italian"},
-                    "creator"=>{0=>"Linguist, A."},
-                    "geography"=>{0=>"Italy"},
-                    "title"=>{0=>"Italian and Spanish: A Comparison of Common Phrases"}}
-      @test_object.update_indexed_attributes(attributes)
-      @test_object.save
-      
-      @test_object2 = MockMetaHelperSolr.new
-      attributes = {"holding_id"=>{0=>"Holding 2"},
-                    "language"=>{0=>"Spanish;Latin"},
-                    "creator"=>{0=>"Linguist, A."},
-                    "geography"=>{0=>"Spain"},
-                    "title"=>{0=>"A study of the evolution of Spanish from Latin"}}
-      @test_object2.update_indexed_attributes(attributes)
-      @test_object2.save      
-
-      @test_object3 = MockMetaHelperSolr.new
-      attributes = {"holding_id"=>{0=>"Holding 3"},
-                    "language"=>{0=>"Spanish;Latin"},
-                    "creator"=>{0=>"Linguist, A."},
-                    "geography"=>{0=>"Spain"},
-                    "title"=>{0=>"An obscure look into early nomadic tribes of Spain"}}
-      @test_object3.update_indexed_attributes(attributes)
-      @test_object3.save
-      
-      #from_solr gets called indirectly
-      test_from_solr_object = MockMetaHelperSolr.load_instance_from_solr(@test_object.pid)
-      test_from_solr_object2 = MockMetaHelperSolr.load_instance_from_solr(@test_object2.pid)
-      test_from_solr_object3 = MockMetaHelperSolr.load_instance_from_solr(@test_object3.pid)
-      
-      test_from_solr_object.descMetadata.language.should == ["Italian"]
-      test_from_solr_object.descMetadata.creator.should == ["Linguist, A."]
-      test_from_solr_object.descMetadata.geography.should == ["Italy"]
-      test_from_solr_object.descMetadata.title.should == ["Italian and Spanish: A Comparison of Common Phrases"]
-      #test_from_solr_object.fields[:holding_id][:values].should == ["Holding 1"]
-      
-      test_from_solr_object2.descMetadata.language.should == ["Spanish;Latin"]
-      test_from_solr_object2.descMetadata.creator.should == ["Linguist, A."]
-      test_from_solr_object2.descMetadata.geography.should == ["Spain"]
-      test_from_solr_object2.descMetadata.title.should == ["A study of the evolution of Spanish from Latin"]
-      #test_from_solr_object2.fields[:holding_id][:values].should == ["Holding 2"]
-      
-      test_from_solr_object3.descMetadata.language.should == ["Spanish;Latin"]
-      test_from_solr_object3.descMetadata.creator.should == ["Linguist, A."]
-      test_from_solr_object3.descMetadata.geography.should == ["Spain"]
-      test_from_solr_object3.descMetadata.title.should == ["An obscure look into early nomadic tribes of Spain"]
-      #test_from_solr_object3.properties.holding_id.should == ["Holding 3"]
-      
-      
-    end
-  end
-  
-end

data/spec/integration/semantic_node_spec.rb

@@ -1,457 +0,0 @@
-require 'spec_helper'
-
-require 'active_fedora'
-
-describe ActiveFedora::SemanticNode do
-        before(:all) do
-        @behavior = ActiveFedora::Relationships.deprecation_behavior
-        @c_behavior = ActiveFedora::Relationships::ClassMethods.deprecation_behavior
-        ActiveFedora::Relationships.deprecation_behavior = :silence
-        ActiveFedora::Relationships::ClassMethods.deprecation_behavior = :silence
-      end
-  
-      after :all do
-        ActiveFedora::Relationships.deprecation_behavior = @behavior
-        ActiveFedora::Relationships::ClassMethods.deprecation_behavior = @c_behavior
-      end
-  before(:all) do 
-    class SNSpecNode < ActiveFedora::Base
-      include ActiveFedora::FileManagement
-    end
-    @node = SNSpecNode.new
-    class SNSpecModel < ActiveFedora::Base
-      include ActiveFedora::Relationships
-      has_relationship("parts", :is_part_of, :inbound => true)
-      has_relationship("containers", :is_member_of)
-      has_bidirectional_relationship("bi_containers", :is_member_of, :has_member)
-    end
-    class SpecNodeSolrFilterQuery < ActiveFedora::Base
-      include ActiveFedora::Relationships
-      has_relationship("parts", :is_part_of, :inbound => true)
-      has_relationship("special_parts", :is_part_of, :inbound => true, :solr_fq=>"#{ActiveFedora::SolrService.solr_name("has_model", :symbol)}:info\\:fedora\\/afmodel\\:SpecialPart")
-      has_relationship("containers", :is_member_of)
-      has_relationship("special_containers", :is_member_of, :solr_fq=>"#{ActiveFedora::SolrService.solr_name("has_model", :symbol)}:info\\:fedora\\/afmodel\\:SpecialContainer")
-      has_bidirectional_relationship("bi_containers", :is_member_of, :has_member)
-      has_bidirectional_relationship("bi_special_containers", :is_member_of, :has_member, :solr_fq=>"#{ActiveFedora::SolrService.solr_name("has_model", :symbol)}:info\\:fedora\\/afmodel\\:SpecialContainer")
-    end
-
-    @test_object = SNSpecModel.new
-    @test_object.save
-    
-    @part1 = ActiveFedora::Base.new()
-    @part1.add_relationship(:is_part_of, @test_object)
-    @part1.save
-    @part2 = ActiveFedora::Base.new()
-    @part2.add_relationship(:is_part_of, @test_object)
-    @part2.save
-    
-    
-    @container1 = ActiveFedora::Base.new()
-    @container1.save
-    @container2 = ActiveFedora::Base.new()    
-    @container2.save
-    @container3 = ActiveFedora::Base.new()    
-    @container3.save
-    @container4 = ActiveFedora::Base.new()    
-    @container4.save
-    
-    @test_object.add_relationship(:is_member_of, @container1)
-    @test_object.add_relationship(:is_member_of, @container2)
-    @test_object.add_relationship(:is_member_of, @container3)
-    @test_object.save
-
-    @container4.add_relationship(:has_member,@test_object)
-    @container4.save
-
-    class SpecialContainer < ActiveFedora::Base; end;
-    class SpecialPart < ActiveFedora::Base; end;
-    @special_container = ActiveFedora::Base.new()
-    @special_container.add_relationship(:has_model, SpecialContainer.to_class_uri)
-    @special_container.save
-
-    @special_container3 = ActiveFedora::Base.new()
-    @special_container3.add_relationship(:has_model, SpecialContainer.to_class_uri)
-    @special_container3.save
-
-    @special_container4 = ActiveFedora::Base.new()
-    @special_container4.add_relationship(:has_model, SpecialContainer.to_class_uri)
-    @special_container4.save
-
-    #even though adding container3 and 3 special containers, it should only include the special containers when returning via relationship name finder methods
-    #also should only return special part similarly
-    @test_object_query = SpecNodeSolrFilterQuery.new
-    @test_object_query.add_relationship(:is_member_of, @container3)
-    @test_object_query.add_relationship(:is_member_of, @special_container)
-    @test_object_query.add_relationship(:is_member_of, @special_container3)
-    @test_object_query.add_relationship(:is_member_of, @special_container4)
-    @test_object_query.save
-
-    @special_container2 = ActiveFedora::Base.new()
-    @special_container2.add_relationship(:has_model, SpecialContainer.to_class_uri)
-    @special_container2.add_relationship(:has_member, 'info:fedora/'+@test_object_query.pid)
-    @special_container2.save
-
-    @part3 = ActiveFedora::Base.new()
-    @part3.add_relationship(:is_part_of, @test_object_query)
-    @part3.save
-
-    @special_part = ActiveFedora::Base.new()
-    @special_part.add_relationship(:has_model, SpecialPart.to_class_uri)
-    @special_part.add_relationship(:is_part_of, @test_object_query)
-    @special_part.save
-    
-    @special_container_query = "#{ActiveFedora::SolrService.solr_name("has_model", :symbol)}:#{solr_uri("info:fedora/afmodel:SpecialContainer")}"
-    @special_part_query = "#{ActiveFedora::SolrService.solr_name("has_model", :symbol)}:#{solr_uri("info:fedora/afmodel:SpecialPart")}"
-  end
-  
-  after(:all) do
-    begin
-    @part1.delete 
-    rescue
-    end
-    begin
-    @part2.delete
-    rescue
-    end
-    begin
-    @part3.delete
-    rescue
-    end
-    begin
-    @container1.delete
-    rescue
-    end
-    begin
-    @container2.delete
-    rescue
-    end
-    begin
-    @container3.delete
-    rescue
-    end
-    begin
-    @test_object.delete
-    rescue
-    end
-    begin
-    @test_object_query.delete
-    rescue
-    end
-    begin
-    @special_part.delete
-    rescue
-    end
-    begin
-    @special_container.delete
-    rescue
-    end
-    begin
-    @special_container2.delete
-    rescue
-    end
-    Object.send(:remove_const, :SNSpecModel)
-
-  end
-  
-  describe '#has_relationship' do
-    it "should create useable finders" do
-      spec_node = SNSpecNode.new
-      spec_node.collection_members.should == []
-      
-      spec_node.add_relationship(:has_collection_member, @test_object.pid)
-      collection_members = spec_node.collection_members
-      collection_members.length.should == 1
-      collection_members.first.pid.should == @test_object.pid
-      collection_members.first.class.should == @test_object.class
-    end
-  end
-  
-  describe "inbound relationship finders" do
-    describe "when inheriting from parents" do
-      before do
-        class Test1 < ActiveFedora::Base
-          include ActiveFedora::FileManagement
-        end
-        @part4 = Test1.new()
-        @part4.parts 
-
-        class Test2 < ActiveFedora::Base
-          include ActiveFedora::FileManagement
-        end
-        class Test3 < Test2
-          include ActiveFedora::Relationships
-          has_relationship "testing", :has_member
-        end
-
-        class Test4 < Test3
-          include ActiveFedora::Relationships
-          has_relationship "testing_inbound", :is_member_of, :inbound=>true
-        end
-
-        class Test5 < Test4
-          include ActiveFedora::Relationships
-          has_relationship "testing_inbound", :is_part_of, :inbound=>true
-        end 
- 
-        @test_object2 = Test2.new
-        @test_object2.save
-      end
-      it "should have relationships defined" do
-        Test1.relationships_desc.should have_key(:inbound)
-        Test2.relationships_desc.should have_key(:inbound)
-        Test2.relationships_desc[:inbound]["parts_inbound"].should == {:inbound=>true, :predicate=>:is_part_of, :singular=>nil}
-      end
-
-      it "should have relationships defined from more than one ancestor class" do
-        Test4.relationships_desc[:self].should have_key("collection_members")
-        Test4.relationships_desc[:self].should have_key("testing")
-        Test4.relationships_desc[:inbound].should have_key("testing_inbound")
-      end
-
-      it "should override a parents relationship description if defined in the child" do
-        #check parent description
-        Test4.relationships_desc[:inbound]["testing_inbound"][:predicate].should == :is_member_of
-        #check child with overwritten relationship description has different predicate
-        Test5.relationships_desc[:inbound]["testing_inbound"][:predicate].should == :is_part_of
-      end
-
-      it "should not have relationships bleeding over from other sibling classes" do
-        SpecNodeSolrFilterQuery.relationships_desc[:inbound].should have_key("bi_special_containers_inbound")
-        Test1.relationships_desc[:inbound].should_not have_key("bi_special_containers_inbound")
-        Test2.relationships_desc[:inbound].should_not have_key("bi_special_containers_inbound")
-      end
-      it "should return an empty set" do
-        @test_object2.parts.should == []
-        @test_object2.parts_outbound.should == []
-      end
-      it "should return stuff" do
-        @part4.add_relationship(:is_part_of, @test_object2)
-        @test_object2.add_relationship(:has_part, @test_object)
-        @part4.save
-        @test_object2.parts_inbound.map(&:pid).should == [@part4.pid]
-        @test_object2.parts_outbound.map(&:pid).should == [@test_object.pid]
-        @test_object2.parts.map(&:pid).should == [@part4.pid, @test_object.pid]
-      end 
-      after do
-        @test_object2.delete
-        begin
-          @part4.delete
-        rescue
-        end
-      end
-    end
-    it "should return an array of Base objects" do
-      parts = @test_object.parts
-      parts.each do |part|
-        part.should be_kind_of(ActiveFedora::Base)
-      end  
-    end
-    it "_ids should return an array of pids" do
-      ids = @test_object.parts_ids
-      ids.size.should == 2
-      ids.include?(@part1.pid).should == true
-      ids.include?(@part2.pid).should == true
-    end
-    it "should return an array of Base objects with some filtered out if using solr_fq" do
-      @test_object_query.special_parts_ids.should == [@special_part.pid]
-    end
-
-    it "should return an array of all Base objects with relationship if not using solr_fq" do
-      @test_object_query.parts_ids.size.should == 2
-      @test_object_query.parts_ids.include?(@special_part.pid).should == true
-      @test_object_query.parts_ids.include?(@part3.pid).should == true
-    end
-
-    it "should return a solr query for an inbound relationship" do
-      @test_object_query.special_parts_query.should == "#{ActiveFedora::SolrService.solr_name(@test_object_query.relationship_predicates[:inbound]['special_parts'], :symbol)}:#{solr_uri(@test_object_query.internal_uri)} AND #{@special_part_query}"
-    end
-  end
-
-  describe "inbound relationship query" do
-    it "should return a properly formatted query for a relationship that has a solr filter query defined" do
-      SpecNodeSolrFilterQuery.inbound_relationship_query(@test_object_query.pid,"special_parts").should == "#{ActiveFedora::SolrService.solr_name(@test_object_query.relationship_predicates[:inbound]['special_parts'], :symbol)}:#{solr_uri(@test_object_query.internal_uri)} AND #{@special_part_query}"
-    end
-
-    it "should return a properly formatted query for a relationship that does not have a solr filter query defined" do
-      SpecNodeSolrFilterQuery.inbound_relationship_query(@test_object_query.pid,"parts").should == "#{ActiveFedora::SolrService.solr_name('is_part_of', :symbol)}:#{solr_uri(@test_object_query.internal_uri)}"
-    end
-  end
-
-  describe "outbound relationship query" do
-    it "should return a properly formatted query for a relationship that has a solr filter query defined" do
-      expected_string = ""
-      @test_object_query.containers_ids.each_with_index do |id,index|
-        expected_string << " OR " if index > 0
-        expected_string << "(id:" + id.gsub(/(:)/, '\\:') + " AND #{@special_container_query})"
-      end
-      SpecNodeSolrFilterQuery.outbound_relationship_query("special_containers",@test_object_query.containers_ids).should == expected_string
-    end
-
-    it "should return a properly formatted query for a relationship that does not have a solr filter query defined" do
-      expected_string = ""
-      @test_object_query.containers_ids.each_with_index do |id,index|
-        expected_string << " OR " if index > 0
-        expected_string << "id:" + id.gsub(/(:)/, '\\:')
-      end
-      SpecNodeSolrFilterQuery.outbound_relationship_query("containers",@test_object_query.containers_ids).should == expected_string
-    end
-  end
-
-  describe "bidirectional relationship query" do
-    it "should return a properly formatted query for a relationship that has a solr filter query defined" do
-      expected_string = ""
-      @test_object_query.bi_containers_outbound_ids.each_with_index do |id,index|
-        expected_string << " OR " if index > 0
-        expected_string << "(id:" + id.gsub(/(:)/, '\\:') + " AND #{@special_container_query})"
-      end
-      expected_string << " OR "
-      expected_string << "(#{ActiveFedora::SolrService.solr_name(@test_object_query.relationship_predicates[:inbound]['bi_special_containers_inbound'], :symbol)}:#{solr_uri(@test_object_query.internal_uri)} AND #{@special_container_query})"
-      SpecNodeSolrFilterQuery.bidirectional_relationship_query(@test_object_query.pid,"bi_special_containers",@test_object_query.bi_containers_outbound_ids).should == expected_string
-    end
-
-    it "should return a properly formatted query for a relationship that does not have a solr filter query defined" do
-      expected_string = ""
-      @test_object_query.bi_containers_outbound_ids.each_with_index do |id,index|
-        expected_string << " OR " if index > 0
-        expected_string << "id:" + id.gsub(/(:)/, '\\:')
-      end
-      expected_string << " OR "
-      expected_string << "(#{ActiveFedora::SolrService.solr_name(@test_object_query.relationship_predicates[:inbound]['bi_special_containers_inbound'], :symbol)}:#{solr_uri(@test_object_query.internal_uri)})"
-      SpecNodeSolrFilterQuery.bidirectional_relationship_query(@test_object_query.pid,"bi_containers",@test_object_query.bi_containers_outbound_ids).should == expected_string
-    end
-  end
-
-  describe "outbound relationship finders" do
-    it "should return an array of Base objects" do
-      containers = @test_object.containers
-      containers.length.should > 0
-      containers.each do |container|
-        container.should be_kind_of(ActiveFedora::Base)
-      end  
-    end
-    it "_ids should return an array of pids" do
-      ids = @test_object.containers_ids
-      ids.size.should == 3
-      ids.include?(@container1.pid).should == true
-      ids.include?(@container2.pid).should == true
-      ids.include?(@container3.pid).should == true
-      ids.include?(@container4.pid).should == false
-    end
-
-    it "should return an array of Base objects with some filtered out if using solr_fq" do
-      @test_object_query.special_containers_ids.size.should == 3
-      @test_object_query.special_containers_ids.include?(@container3.pid).should == false
-      @test_object_query.special_containers_ids.include?(@special_container.pid).should == true
-      @test_object_query.special_containers_ids.include?(@special_container3.pid).should == true
-      @test_object_query.special_containers_ids.include?(@special_container4.pid).should == true
-    end
-
-    it "should return an array of all Base objects with relationship if not using solr_fq" do
-      @test_object_query.containers_ids.size.should == 4
-      @test_object_query.containers_ids.include?(@special_container2.pid).should == false
-      @test_object_query.containers_ids.include?(@special_container.pid).should == true
-      @test_object_query.containers_ids.include?(@container3.pid).should == true
-      @test_object_query.containers_ids.include?(@special_container3.pid).should == true
-      @test_object_query.containers_ids.include?(@special_container4.pid).should == true
-    end
-
-    it "should return a solr query for an outbound relationship" do
-    end
-
-    it "should return an array of Base objects with some filtered out if using solr_fq" do
-      @test_object_query.special_containers_ids.size.should == 3
-      @test_object_query.special_containers_ids.include?(@container3.pid).should == false
-      @test_object_query.special_containers_ids.include?(@special_container.pid).should == true
-      @test_object_query.special_containers_ids.include?(@special_container3.pid).should == true
-      @test_object_query.special_containers_ids.include?(@special_container4.pid).should == true
-    end
-
-    it "should return an array of all Base objects with relationship if not using solr_fq" do
-      @test_object_query.containers_ids.size.should == 4
-      @test_object_query.containers_ids.include?(@special_container2.pid).should == false
-      @test_object_query.containers_ids.include?(@special_container.pid).should == true
-      @test_object_query.containers_ids.include?(@container3.pid).should == true
-      @test_object_query.containers_ids.include?(@special_container3.pid).should == true
-      @test_object_query.containers_ids.include?(@special_container4.pid).should == true
-    end
-
-    it "should return a solr query for an outbound relationship" do
-      expected_string = ""
-      @test_object_query.containers_ids.each_with_index do |id,index|
-        expected_string << " OR " if index > 0
-        expected_string << "(id:" + id.gsub(/(:)/, '\\:') + " AND #{@special_container_query})"
-      end
-      @test_object_query.special_containers_query.should == expected_string
-    end 
-  end  
-
-  describe "bidirectional relationship finders" do
-    it "should return an array of Base objects" do
-      containers = @test_object.bi_containers
-      containers.length.should > 0
-      containers.each do |container|
-        container.should be_kind_of(ActiveFedora::Base)
-      end  
-    end
-    it "_ids should return an array of pids" do
-      ids = @test_object.bi_containers_ids
-      ids.size.should == 4
-      ids.include?(@container1.pid).should == true
-      ids.include?(@container2.pid).should == true
-      ids.include?(@container3.pid).should == true
-      ids.include?(@container4.pid).should == true
-    end
-
-    it "should return an array of Base objects with some filtered out if using solr_fq" do
-      ids = @test_object_query.bi_special_containers_ids
-      ids.size.should == 4
-      ids.include?(@container1.pid).should == false
-      ids.include?(@container2.pid).should == false
-      ids.include?(@container3.pid).should == false
-      ids.include?(@special_container.pid).should == true
-      ids.include?(@special_container2.pid).should == true
-      ids.include?(@special_container3.pid).should == true
-      ids.include?(@special_container4.pid).should == true
-    end
-
-    it "should return an array of all Base objects with relationship if not using solr_fq" do
-      @test_object_query.bi_containers_ids.should include @container3.pid, @special_container.pid, @special_container2.pid, @special_container3.pid, @special_container4.pid
-
-      @test_object_query.bi_containers_ids.size.should == 5
-    end
-
-    it "should return a solr query for a bidirectional relationship" do
-      expected_string = ""
-      @test_object_query.bi_containers_outbound_ids.each_with_index do |id,index|
-        expected_string << " OR " if index > 0
-        expected_string << "(id:" + id.gsub(/(:)/, '\\:') + " AND #{@special_container_query})"
-      end
-      expected_string << " OR "
-      expected_string << "(#{ActiveFedora::SolrService.solr_name(@test_object_query.relationship_predicates[:inbound]['bi_special_containers_inbound'], :symbol)}:#{solr_uri(@test_object_query.internal_uri)} AND #{@special_container_query})"
-      @test_object_query.bi_special_containers_query.should == expected_string
-    end
-  end
-
-  #putting this test here instead of relationships_helper because testing that relationships_by_name hash gets refreshed if the relationships hash is changed
-  describe "relationships_by_name" do
-    before do
-      class MockSemNamedRelationships  < ActiveFedora::Base
-        include ActiveFedora::FileManagement
-        has_relationship "testing", :has_part
-        has_relationship "testing2", :has_member
-        has_relationship "testing_inbound", :has_part, :inbound=>true
-      end
-    end
-
-    it 'should automatically update the relationships_by_name if relationships has changed (no refresh of relationships_by_name hash unless relationships hash has changed' do
-      @test_object2 = MockSemNamedRelationships.new
-      @test_object2.add_relationship(:has_model, MockSemNamedRelationships.to_class_uri)
-      #should return expected named relationships
-      @test_object2.relationships_by_name.should == {:self=>{"testing"=>[],"testing2"=>[], "collection_members"=>[], "part_of"=>[], "parts_outbound"=>[]}}
-      @test_object2.add_relationship(:has_part, @test_object.internal_uri)
-      @test_object2.relationships_by_name.should == {:self=>{"testing"=>[@test_object.internal_uri],"testing2"=>[], "collection_members"=>[], "part_of"=>[],  "parts_outbound"=>[@test_object.internal_uri]}}
-      @test_object2.add_relationship(:has_member, "3", true)
-      @test_object2.relationships_by_name.should == {:self=>{"testing"=>[@test_object.internal_uri],"testing2"=>["3"], "collection_members"=>[], "part_of"=>[],  "parts_outbound"=>[@test_object.internal_uri]}}
-    end
-  end
-end