active-fedora 6.8.0 → 7.0.0.pre1

data/spec/unit/om_datastream_spec.rb

@@ -1,373 +1,367 @@
 require 'spec_helper'
 describe ActiveFedora::OmDatastream do
+  
   before(:all) do
-    @sample_fields = {
-      publisher:     { values: ['publisher1'], type: :string },
-      coverage:      { values: %w(coverage1 coverage2), type: :text },
-      creation_date: { values: 'fake-date', type: :date },
-      mydate:        { values: 'fake-date', type: :date },
-      empty_field:   { values: {} }
-    }
-    @sample_raw_xml = '<foo><xmlelement/></foo>'
-    @solr_doc = {
-      'id' => 'mods_article1',
-      ActiveFedora::SolrService.solr_name('name_role_roleTerm', type: :string) => %w(creator submitter teacher),
-      ActiveFedora::SolrService.solr_name('name_0_role', type: :string) => "\r\ncreator\r\nsubmitter\r\n",
-      ActiveFedora::SolrService.solr_name('name_1_role', type: :string) => "\r\n teacher \r\n",
-      ActiveFedora::SolrService.solr_name('name_0_role_0_roleTerm', type: :string) => 'creator',
-      ActiveFedora::SolrService.solr_name('name_0_role_1_roleTerm', type: :string) => 'submitter',
-      ActiveFedora::SolrService.solr_name('name_1_role_0_roleTerm', type: :string) => ['teacher']
-    }
+    @sample_fields = {:publisher => {:values => ["publisher1"], :type => :string}, 
+                      :coverage => {:values => ["coverage1", "coverage2"], :type => :text}, 
+                      :creation_date => {:values => "fake-date", :type => :date},
+                      :mydate => {:values => "fake-date", :type => :date},
+                      :empty_field => {:values => {}}
+                      } 
+    @sample_raw_xml = "<foo><xmlelement/></foo>"
+    @solr_doc = {"id"=>"mods_article1",
+      ActiveFedora::SolrService.solr_name("name_role_roleTerm", type: :string) =>["creator","submitter","teacher"],
+      ActiveFedora::SolrService.solr_name("name_0_role", type: :string)=>"\r\ncreator\r\nsubmitter\r\n",
+      ActiveFedora::SolrService.solr_name("name_1_role", type: :string)=>"\r\n teacher \r\n",
+      ActiveFedora::SolrService.solr_name("name_0_role_0_roleTerm", type: :string)=>"creator",
+      ActiveFedora::SolrService.solr_name("name_0_role_1_roleTerm", type: :string)=>"submitter",
+      ActiveFedora::SolrService.solr_name("name_1_role_0_roleTerm", type: :string)=>["teacher"]}
   end
-
+  
   before(:each) do
     @mock_inner = double('inner object')
-    @mock_repo  = double('repository')
-    @mock_repo.stub(datastream_dissemination: 'My Content', config: {}, datastream_profile: {}, datastream: '')
-    allow(@mock_inner).to receive(:repository).and_return(@mock_repo)
-    allow(@mock_inner).to receive(:pid)
-    @mock_inner.stub(new?: false)
-    @test_ds = ActiveFedora::OmDatastream.new(@mock_inner, 'descMetadata')
-    @test_ds.stub(new?: false, profile: {}, datastream_content: '<test_xml/>')
-    @test_ds.content = '<test_xml/>'
-    @test_ds.stub(new?: false)
-  end
-
-  describe '#metadata?' do
-    subject { super().metadata? }
-    it { is_expected.to be_truthy }
+    @mock_repo = double('repository')
+    @mock_repo.stub(:datastream_dissemination=>'My Content', :config=>{}, :datastream_profile=>{})
+    @mock_inner.stub(:repository).and_return(@mock_repo)
+    @mock_inner.stub(:pid)
+    @mock_inner.stub(:new_record? => false)
+    @test_ds = ActiveFedora::OmDatastream.new(@mock_inner, "descMetadata")
+    @test_ds.stub(:new? => false, :profile => {}, :datastream_content => '<test_xml/>')
+    @test_ds.content="<test_xml/>"
+    @test_ds.stub(:new? => false)
   end
+  
+  its(:metadata?) { should be_true}
 
-  describe '#controlGroup' do
-    subject { super().controlGroup }
-    it { is_expected.to eq('M') }
-  end
+  its(:controlGroup) { should == "M"}
 
-  it 'should include the Solrizer::XML::TerminologyBasedSolrizer for .to_solr support' do
-    expect(ActiveFedora::OmDatastream.included_modules).to include(OM::XML::TerminologyBasedSolrizer)
+  it "should include the Solrizer::XML::TerminologyBasedSolrizer for .to_solr support" do
+    ActiveFedora::OmDatastream.included_modules.should include(OM::XML::TerminologyBasedSolrizer)
   end
-
+  
   describe '#new' do
     it 'should provide #new' do
-      expect(ActiveFedora::OmDatastream).to respond_to(:new)
-      expect(@test_ds.ng_xml).to be_instance_of(Nokogiri::XML::Document)
+      ActiveFedora::OmDatastream.should respond_to(:new)
+      @test_ds.ng_xml.should be_instance_of(Nokogiri::XML::Document)
     end
     it 'should load xml from blob if provided' do
       test_ds1 = ActiveFedora::OmDatastream.new(nil, 'ds1')
-      test_ds1.content = '<xml><foo/></xml>'
-      expect(test_ds1.ng_xml.to_xml).to eq("<?xml version=\"1.0\"?>\n<xml>\n  <foo/>\n</xml>\n")
+      test_ds1.content="<xml><foo/></xml>"
+      test_ds1.ng_xml.to_xml.should == "<?xml version=\"1.0\"?>\n<xml>\n  <foo/>\n</xml>\n"
     end
-    it 'should initialize from #xml_template if no xml is provided' do
-      expect(ActiveFedora::OmDatastream).to receive(:xml_template).and_return('<fake template/>')
+    it "should initialize from #xml_template if no xml is provided" do
+      ActiveFedora::OmDatastream.should_receive(:xml_template).and_return("<fake template/>")
       n = ActiveFedora::OmDatastream.new
-      expect(n.ng_xml).to be_equivalent_to('<fake template/>')
+      n.ng_xml.should be_equivalent_to("<fake template/>")
     end
   end
-
+  
   describe '#xml_template' do
-    it 'should return an empty xml document' do
-      expect(ActiveFedora::OmDatastream.xml_template.to_xml).to eq("<?xml version=\"1.0\"?>\n<xml/>\n")
+    it "should return an empty xml document" do
+      ActiveFedora::OmDatastream.xml_template.to_xml.should == "<?xml version=\"1.0\"?>\n<xml/>\n"
     end
   end
 
-  describe 'an instance' do
-    it '#to_solr' do
-      obj = ActiveFedora::OmDatastream.new
-      expect(obj).to respond_to(:to_solr)
-      expect(obj.to_solr).to eq({})
-    end
+  describe "an instance" do
+    subject { ActiveFedora::OmDatastream.new }
+    it{ should.respond_to? :to_solr }
+    its(:to_solr) {should == { }}
   end
-
-  describe '.update_indexed_attributes' do
+  
+  describe ".update_indexed_attributes" do
+    
     before(:each) do
       @mods_ds = Hydra::ModsArticleDatastream.new(nil, 'descMetadata')
-      @mods_ds.content = fixture(File.join('mods_articles', 'mods_article1.xml')).read
+      @mods_ds.content=fixture(File.join("mods_articles","mods_article1.xml")).read
     end
-
-    it 'should apply submitted hash to corresponding datastream field values' do
-      result = @mods_ds.update_indexed_attributes([{ ':person' => '0' }, 'role'] => { '0' => 'role1', '1' => 'role2', '2' => 'role3' })
-      expect(result).to eq('person_0_role' => %w(role1 role2 role3))
-      expect(@mods_ds.property_values('//oxns:name[@type="personal"][1]/oxns:role')).to eq(%w(role1 role2 role3))
+    
+    it "should apply submitted hash to corresponding datastream field values" do
+      result = @mods_ds.update_indexed_attributes( {[{":person"=>"0"}, "role"]=>{"0"=>"role1", "1"=>"role2", "2"=>"role3"} })
+      result.should == {"person_0_role"=>["role1", "role2", "role3"]}
+      @mods_ds.property_values('//oxns:name[@type="personal"][1]/oxns:role').should == ["role1","role2","role3"]
     end
-    it 'should support single-value arguments (as opposed to a hash of values with array indexes as keys)' do
+    it "should support single-value arguments (as opposed to a hash of values with array indexes as keys)" do
       # In other words, { "fubar"=>"dork" } should have the same effect as { "fubar"=>{"0"=>"dork"} }
-      result = @mods_ds.update_indexed_attributes([{ ':person' => '0' }, 'role'] => 'the role')
-      expect(result).to eq('person_0_role' => ['the role'])
-      expect(@mods_ds.term_values('//oxns:name[@type="personal"][1]/oxns:role').first).to eq('the role')
+      result = @mods_ds.update_indexed_attributes( { [{":person"=>"0"}, "role"]=>"the role" } )
+      result.should == {"person_0_role"=>["the role"]}
+      @mods_ds.term_values('//oxns:name[@type="personal"][1]/oxns:role').first.should == "the role"
     end
-    it 'should do nothing if field key is a string (must be an array or symbol).  Will not accept xpath queries!' do
+    it "should do nothing if field key is a string (must be an array or symbol).  Will not accept xpath queries!" do
       xml_before = @mods_ds.to_xml
-      expect(logger).to receive(:warn).with "WARNING: descMetadata ignoring {\"fubar\" => \"the role\"} because \"fubar\" is a String (only valid OM Term Pointers will be used).  Make sure your html has the correct field_selector tags in it."
-      expect(@mods_ds.update_indexed_attributes('fubar' => 'the role')).to eq({})
-      expect(@mods_ds.to_xml).to eq(xml_before)
+      logger.should_receive(:warn).with "WARNING: descMetadata ignoring {\"fubar\" => \"the role\"} because \"fubar\" is a String (only valid OM Term Pointers will be used).  Make sure your html has the correct field_selector tags in it."
+      @mods_ds.update_indexed_attributes( { "fubar"=>"the role" } ).should == {}
+      @mods_ds.to_xml.should == xml_before
     end
-    it 'should do nothing if there is no accessor corresponding to the given field key' do
+    it "should do nothing if there is no accessor corresponding to the given field key" do
       xml_before = @mods_ds.to_xml
-      expect(@mods_ds.update_indexed_attributes([{ 'fubar' => '0' }] => 'the role')).to eq({})
-      expect(@mods_ds.to_xml).to eq(xml_before)
+      @mods_ds.update_indexed_attributes( { [{"fubar"=>"0"}]=>"the role" } ).should == {}
+      @mods_ds.to_xml.should == xml_before
     end
-
+    
     ### Examples copied over form metadata_datastream_spec
-
-    it 'should work for text fields' do
-      att = { [{ 'person' => '0' }, 'description'] => { '-1' => 'mork', '1' => 'york' } }
+    
+    it "should work for text fields" do 
+      att= {[{"person"=>"0"},"description"]=>{"-1"=>"mork", "1"=>"york"}}
       result = @mods_ds.update_indexed_attributes(att)
-      expect(result).to eq('person_0_description' => %w(mork york))
-      expect(@mods_ds.get_values([{ person: 0 }, :description])).to eq(%w(mork york))
-      att = { [{ 'person' => '0' }, 'description'] => { '-1' => 'dork' } }
+      result.should == {"person_0_description"=>["mork","york"]}
+      @mods_ds.get_values([{:person=>0},:description]).should == ['mork', 'york']
+      att= {[{"person"=>"0"},"description"]=>{"-1"=>"dork"}}
       result2 = @mods_ds.update_indexed_attributes(att)
-      expect(result2).to eq('person_0_description' => ['dork'])
-      expect(@mods_ds.get_values([{ person: 0 }, :description])).to eq(['dork'])
+      result2.should == {"person_0_description"=>["dork"]}
+      @mods_ds.get_values([{:person=>0},:description]).should == ['dork']
     end
-
-    it 'should allow deleting of values and should delete values so that to_xml does not return emtpy nodes' do
-      att = { [{ 'person' => '0' }, 'description'] => { '0' => 'york', '1' => 'mangle', '2' => 'mork' } }
+    
+    it "should allow deleting of values and should delete values so that to_xml does not return emtpy nodes" do
+      att= {[{"person"=>"0"},"description"]=>{"0"=>"york", "1"=>"mangle","2"=>"mork"}}
       @mods_ds.update_indexed_attributes(att)
-      expect(@mods_ds.get_values([{ 'person' => '0' }, 'description'])).to eq(%w(york mangle mork))
-
-      @mods_ds.update_indexed_attributes([{ 'person' => '0' }, { 'description' => '1' }] => nil)
-      expect(@mods_ds.get_values([{ 'person' => '0' }, 'description'])).to eq(%w(york mork))
-
-      @mods_ds.update_indexed_attributes([{ 'person' => '0' }, { 'description' => '0' }] => :delete)
-      expect(@mods_ds.get_values([{ 'person' => '0' }, 'description'])).to eq(['mork'])
-    end
-
-    it 'should set changed to true' do
-      expect(@mods_ds.get_values([{ title_info: 0 }, :main_title])).to eq(['ARTICLE TITLE', 'TITLE OF HOST JOURNAL'])
-      @mods_ds.update_indexed_attributes [{ 'title_info' => '0' }, 'main_title'] => { '-1' => 'mork' }
-      expect(@mods_ds).to be_changed
+      @mods_ds.get_values([{"person"=>"0"},"description"]).should == ['york', 'mangle', 'mork']
+      
+      @mods_ds.update_indexed_attributes({[{"person"=>"0"},{"description" => '1'} ]=> nil})
+      @mods_ds.get_values([{"person"=>"0"},"description"]).should == ['york', 'mork']
+      
+      @mods_ds.update_indexed_attributes({[{"person"=>"0"},{"description" => '0'}]=>:delete})
+      @mods_ds.get_values([{"person"=>"0"},"description"]).should == ['mork']
+    end
+    
+    it "should set changed to true" do
+      @mods_ds.get_values([{:title_info=>0},:main_title]).should == ["ARTICLE TITLE", "TITLE OF HOST JOURNAL"]
+      @mods_ds.update_indexed_attributes [{"title_info"=>"0"},"main_title"]=>{"-1"=>"mork"}
+      @mods_ds.should be_changed
     end
   end
-
-  describe '.get_values' do
+  
+  describe ".get_values" do
+    
     before(:each) do
       @mods_ds = Hydra::ModsArticleDatastream.new(nil, 'modsDs')
-      @mods_ds.content = fixture(File.join('mods_articles', 'mods_article1.xml')).read
+      @mods_ds.content=fixture(File.join("mods_articles","mods_article1.xml")).read
     end
-
-    it 'should call lookup with field_name and return the text values from each resulting node' do
-      expect(@mods_ds).to receive(:term_values).with('--my xpath--').and_return(%w(value1 value2))
-      expect(@mods_ds.get_values('--my xpath--')).to eq(%w(value1 value2))
+    
+    it "should call lookup with field_name and return the text values from each resulting node" do
+      @mods_ds.should_receive(:term_values).with("--my xpath--").and_return(["value1", "value2"])
+      @mods_ds.get_values("--my xpath--").should == ["value1", "value2"]
     end
-    it 'should assume that field_names that are strings are xpath queries' do
-      expect(ActiveFedora::OmDatastream).to receive(:accessor_xpath).never
-      expect(@mods_ds).to receive(:term_values).with('--my xpath--').and_return(%w(abstract1 abstract2))
-      expect(@mods_ds.get_values('--my xpath--')).to eq(%w(abstract1 abstract2))
+    it "should assume that field_names that are strings are xpath queries" do
+      ActiveFedora::OmDatastream.should_receive(:accessor_xpath).never
+      @mods_ds.should_receive(:term_values).with("--my xpath--").and_return(["abstract1", "abstract2"])
+      @mods_ds.get_values("--my xpath--").should == ["abstract1", "abstract2"]
     end
   end
 
   describe '.save' do
-    it 'should provide .save' do
-      expect(@test_ds).to respond_to(:save)
-    end
-    it 'should persist the product of .to_xml in fedora' do
-      allow(@mock_repo).to receive(:datastream).and_return('')
-      @test_ds.stub(new?: true)
-      @test_ds.stub(ng_xml_changed?: true)
-      @test_ds.stub(to_xml: 'fake xml')
-      expect(@mock_repo).to receive(:add_datastream).with(pid: nil, dsid: 'descMetadata', versionable: true, content: 'fake xml', controlGroup: 'M', dsState: 'A', mimeType: 'text/xml')
+    it "should provide .save" do
+      @test_ds.should respond_to(:save)
+    end
+    it "should persist the product of .to_xml in fedora" do
+      @mock_repo.stub(:datastream).and_return('')
+      @test_ds.stub(:new? => true)
+      @test_ds.stub(:ng_xml_changed? => true)
+      @test_ds.stub(:to_xml => "fake xml")
+      @mock_repo.should_receive(:add_datastream).with(:pid => nil, :dsid => 'descMetadata', :versionable => true, :content => 'fake xml', :controlGroup => 'M', :dsState => 'A', :mimeType=>'text/xml')
 
       @test_ds.serialize!
       @test_ds.save
-      expect(@test_ds.mimeType).to eq('text/xml')
+      @test_ds.mimeType.should == 'text/xml'
     end
   end
-
+  
   describe 'setting content' do
-    subject { ActiveFedora::OmDatastream.new(@mock_inner, 'descMetadata') }
-    it 'should update the content' do
-      subject.stub(new?: false)
-      subject.content = '<a />'
-      expect(subject.content).to eq('<a/>')
+    subject { ActiveFedora::OmDatastream.new(@mock_inner, "descMetadata") }
+    it "should update the content" do
+      subject.stub(:new? => false )
+      subject.content = "<a />"
+      subject.content.should == '<a/>'
     end
 
-    it 'should mark the object as changed' do
-      subject.stub(new?: false, controlGroup: 'M')
-      subject.content = '<a />'
-      expect(subject).to be_changed
+    it "should mark the object as changed" do
+      subject.stub(:new? => false, :controlGroup => 'M')
+      subject.content = "<a />"
+      subject.should be_changed
     end
 
-    it 'update ngxml and mark the xml as loaded' do
-      subject.stub(new?: false)
-      subject.content = '<a />'
-      expect(subject.ng_xml.to_xml).to match(/<a\/>/)
-      expect(subject.xml_loaded).to be_truthy
+    it "update ngxml and mark the xml as loaded" do
+      subject.stub(:new? => false )
+      subject.content = "<a />"
+      subject.ng_xml.to_xml.should =~ /<a\/>/
+      subject.xml_loaded.should be_true
     end
   end
-
+  
   describe 'ng_xml=' do
     before do
-      @mock_inner.stub(new_record?: true)
-      @test_ds2 = ActiveFedora::OmDatastream.new(@mock_inner, 'descMetadata')
+      @mock_inner.stub(:new_record? => true)
+      @test_ds2 = ActiveFedora::OmDatastream.new(@mock_inner, "descMetadata")
     end
-    it 'should parse raw xml for you' do
+    it "should parse raw xml for you" do
       @test_ds2.ng_xml = @sample_raw_xml
-      expect(@test_ds2.ng_xml.class).to eq(Nokogiri::XML::Document)
-      expect(@test_ds2.ng_xml.to_xml).to be_equivalent_to(@sample_raw_xml)
+      @test_ds2.ng_xml.class.should == Nokogiri::XML::Document
+      @test_ds2.ng_xml.to_xml.should be_equivalent_to(@sample_raw_xml)
     end
 
-    it 'Should always set a document when an Element is passed' do
+    it "Should always set a document when an Element is passed" do
       @test_ds2.ng_xml = Nokogiri::XML(@sample_raw_xml).xpath('//xmlelement').first
-      expect(@test_ds2.ng_xml).to be_kind_of Nokogiri::XML::Document
-      expect(@test_ds2.ng_xml.to_xml).to be_equivalent_to('<xmlelement/>')
+      @test_ds2.ng_xml.should be_kind_of Nokogiri::XML::Document
+      @test_ds2.ng_xml.to_xml.should be_equivalent_to("<xmlelement/>")
     end
-    it 'should mark the datastream as changed' do
-      @test_ds2.stub(new?: false, controlGroup: 'M')
-      expect(@test_ds2).not_to be_changed
+    it "should mark the datastream as changed" do
+      @test_ds2.stub(:new? => false, :controlGroup => 'M')
+      @test_ds2.should_not be_changed 
       @test_ds2.ng_xml = @sample_raw_xml
-      expect(@test_ds2).to be_changed
+      @test_ds2.should be_changed
     end
   end
-
+  
   describe '.to_xml' do
-    it 'should provide .to_xml' do
-      expect(@test_ds).to respond_to(:to_xml)
+    it "should provide .to_xml" do
+      @test_ds.should respond_to(:to_xml)
     end
-
-    it 'should ng_xml.to_xml' do
-      @test_ds.stub(ng_xml: Nokogiri::XML::Document.parse('<text_document/>'))
-      expect(@test_ds.to_xml).to eq('<text_document/>')
+    
+    it "should ng_xml.to_xml" do
+      @test_ds.stub(:ng_xml => Nokogiri::XML::Document.parse("<text_document/>"))
+      @test_ds.to_xml.should == "<text_document/>"       
     end
-
+    
     it 'should accept an optional Nokogiri::XML Document as an argument and insert its fields into that (mocked test)' do
-      doc = Nokogiri::XML::Document.parse('<test_document/>')
-      expect(doc.root).to receive(:add_child) # .with(@test_ds.ng_xml.root)
+      doc = Nokogiri::XML::Document.parse("<test_document/>")
+      doc.root.should_receive(:add_child)#.with(@test_ds.ng_xml.root)
       @test_ds.to_xml(doc)
     end
-
+    
     it 'should accept an optional Nokogiri::XML Document as an argument and insert its fields into that (functional test)' do
-      expected_result = '<test_document><foo/><test_xml/></test_document>'
-      doc = Nokogiri::XML::Document.parse('<test_document><foo/></test_document>')
+      expected_result = "<test_document><foo/><test_xml/></test_document>"
+      doc = Nokogiri::XML::Document.parse("<test_document><foo/></test_document>")
       result = @test_ds.to_xml(doc)
-      expect(doc).to be_equivalent_to expected_result
-      expect(result).to be_equivalent_to expected_result
+      doc.should be_equivalent_to expected_result
+      result.should be_equivalent_to expected_result
     end
-
+    
     it 'should add to root of Nokogiri::XML::Documents, but add directly to the elements if a Nokogiri::XML::Node is passed in' do
-      doc = Nokogiri::XML::Document.parse('<test_document/>')
-      el = Nokogiri::XML::Node.new('test_element', Nokogiri::XML::Document.new)
-      expect(@test_ds.to_xml(doc)).to be_equivalent_to '<test_document><test_xml/></test_document>'
-      expect(@test_ds.to_xml(el)).to be_equivalent_to '<test_element/>'
+      doc = Nokogiri::XML::Document.parse("<test_document/>")
+      el = Nokogiri::XML::Node.new("test_element", Nokogiri::XML::Document.new)
+      @test_ds.to_xml(doc).should be_equivalent_to "<test_document><test_xml/></test_document>"
+      @test_ds.to_xml(el).should be_equivalent_to "<test_element/>"
     end
+    
   end
-
+  
   describe '.from_solr' do
-    it 'should set the internal_solr_doc attribute to the solr document passed in' do
+    it "should set the internal_solr_doc attribute to the solr document passed in" do 
       @test_ds.from_solr(@solr_doc)
-      expect(@test_ds.internal_solr_doc).to eq(@solr_doc)
+      @test_ds.internal_solr_doc.should == @solr_doc
     end
   end
 
   describe '.get_values_from_solr' do
     before(:each) do
       @mods_ds = ActiveFedora::OmDatastream.new
-      @mods_ds.content = fixture(File.join('mods_articles', 'mods_article1.xml')).read
+      @mods_ds.content=fixture(File.join("mods_articles","mods_article1.xml")).read
     end
 
-    it 'should return empty array if internal_solr_doc not set' do
-      @mods_ds.get_values_from_solr(:name, :role, :roleTerm)
+    it "should return empty array if internal_solr_doc not set" do
+      @mods_ds.get_values_from_solr(:name,:role,:roleTerm)
     end
-
-    it 'should return correct values from solr_doc given different term pointers' do
-      mock_term = double('OM::XML::Term')
-      allow(mock_term).to receive(:type).and_return(:text)
-      mock_terminology = double('OM::XML::Terminology')
-      allow(mock_terminology).to receive(:retrieve_term).and_return(mock_term)
-      allow(ActiveFedora::OmDatastream).to receive(:terminology).and_return(mock_terminology)
+ 
+    it "should return correct values from solr_doc given different term pointers" do
+      mock_term = double("OM::XML::Term")
+      mock_term.stub(:type).and_return(:text)
+      mock_terminology = double("OM::XML::Terminology")
+      mock_terminology.stub(:retrieve_term).and_return(mock_term)
+      ActiveFedora::OmDatastream.stub(:terminology).and_return(mock_terminology)
       @mods_ds.from_solr(@solr_doc)
-      term_pointer = [:name, :role, :roleTerm]
-      expect(@mods_ds.get_values_from_solr(:name, :role, :roleTerm)).to eq(%w(creator submitter teacher))
-      ar = @mods_ds.get_values_from_solr({ name: 0 }, :role, :roleTerm)
-      expect(ar.length).to eq(2)
-      expect(ar.include?('creator')).to eq(true)
-      expect(ar.include?('submitter')).to eq(true)
-      expect(@mods_ds.get_values_from_solr({ name: 1 }, :role, :roleTerm)).to eq(['teacher'])
-      expect(@mods_ds.get_values_from_solr({ name: 0 }, { role: 0 }, :roleTerm)).to eq(['creator'])
-      expect(@mods_ds.get_values_from_solr({ name: 0 }, { role: 1 }, :roleTerm)).to eq(['submitter'])
-      expect(@mods_ds.get_values_from_solr({ name: 0 }, { role: 2 }, :roleTerm)).to eq([])
-      expect(@mods_ds.get_values_from_solr({ name: 1 }, { role: 0 }, :roleTerm)).to eq(['teacher'])
-      expect(@mods_ds.get_values_from_solr({ name: 1 }, { role: 1 }, :roleTerm)).to eq([])
-      ar = @mods_ds.get_values_from_solr(:name, { role: 0 }, :roleTerm)
-      expect(ar.length).to eq(2)
-      expect(ar.include?('creator')).to eq(true)
-      expect(ar.include?('teacher')).to eq(true)
-      expect(@mods_ds.get_values_from_solr(:name, { role: 1 }, :roleTerm)).to eq(['submitter'])
+      term_pointer = [:name,:role,:roleTerm]
+      @mods_ds.get_values_from_solr(:name,:role,:roleTerm).should == ["creator","submitter","teacher"]
+      ar = @mods_ds.get_values_from_solr({:name=>0},:role,:roleTerm)
+      ar.length.should == 2
+      ar.include?("creator").should == true
+      ar.include?("submitter").should == true
+      @mods_ds.get_values_from_solr({:name=>1},:role,:roleTerm).should == ["teacher"]
+      @mods_ds.get_values_from_solr({:name=>0},{:role=>0},:roleTerm).should == ["creator"]
+      @mods_ds.get_values_from_solr({:name=>0},{:role=>1},:roleTerm).should == ["submitter"]
+      @mods_ds.get_values_from_solr({:name=>0},{:role=>2},:roleTerm).should == []
+      @mods_ds.get_values_from_solr({:name=>1},{:role=>0},:roleTerm).should == ["teacher"]
+      @mods_ds.get_values_from_solr({:name=>1},{:role=>1},:roleTerm).should == []
+      ar = @mods_ds.get_values_from_solr(:name,{:role=>0},:roleTerm)
+      ar.length.should == 2
+      ar.include?("creator").should == true
+      ar.include?("teacher").should == true
+      @mods_ds.get_values_from_solr(:name,{:role=>1},:roleTerm).should == ["submitter"]
     end
   end
 
   describe '.has_solr_name?' do
-    it 'should return true if the given key exists in the solr document passed in' do
-      expect(@test_ds.has_solr_name?(ActiveFedora::SolrService.solr_name('name_0_role_0_roleTerm', type: :string), @solr_doc)).to eq(true)
-      expect(@test_ds.has_solr_name?(ActiveFedora::SolrService.solr_name('name_0_role_0_roleTerm', type: :string).to_sym, @solr_doc)).to eq(true)
-      expect(@test_ds.has_solr_name?(ActiveFedora::SolrService.solr_name('name_1_role_1_roleTerm', type: :string), @solr_doc)).to eq(false)
-      # if not doc passed in should be new empty solr doc and always return false
-      expect(@test_ds.has_solr_name?(ActiveFedora::SolrService.solr_name('name_0_role_0_roleTerm', type: :string))).to eq(false)
+    it "should return true if the given key exists in the solr document passed in" do
+      @test_ds.has_solr_name?(ActiveFedora::SolrService.solr_name("name_0_role_0_roleTerm", type: :string),@solr_doc).should == true
+      @test_ds.has_solr_name?(ActiveFedora::SolrService.solr_name("name_0_role_0_roleTerm", type: :string).to_sym,@solr_doc).should == true
+      @test_ds.has_solr_name?(ActiveFedora::SolrService.solr_name("name_1_role_1_roleTerm", type: :string),@solr_doc).should == false
+      #if not doc passed in should be new empty solr doc and always return false
+      @test_ds.has_solr_name?(ActiveFedora::SolrService.solr_name("name_0_role_0_roleTerm", type: :string)).should == false
     end
   end
 
   describe '.is_hierarchical_term_pointer?' do
-    it 'should return true only if the pointer passed in is an array that contains a hash' do
-      expect(@test_ds.is_hierarchical_term_pointer?(*[:image, { tag1: 1 }, :tag2])).to eq(true)
-      expect(@test_ds.is_hierarchical_term_pointer?(*[:image, :tag1, { tag2: 1 }])).to eq(true)
-      expect(@test_ds.is_hierarchical_term_pointer?(*[:image, :tag1, :tag2])).to eq(false)
-      expect(@test_ds.is_hierarchical_term_pointer?(nil)).to eq(false)
+    it "should return true only if the pointer passed in is an array that contains a hash" do
+      @test_ds.is_hierarchical_term_pointer?(*[:image,{:tag1=>1},:tag2]).should == true
+      @test_ds.is_hierarchical_term_pointer?(*[:image,:tag1,{:tag2=>1}]).should == true
+      @test_ds.is_hierarchical_term_pointer?(*[:image,:tag1,:tag2]).should == false
+      @test_ds.is_hierarchical_term_pointer?(nil).should == false      
     end
   end
 
   describe '.update_values' do
     before(:each) do
       @mods_ds = ActiveFedora::OmDatastream.new
-      @mods_ds.content = fixture(File.join('mods_articles', 'mods_article1.xml')).read
+      @mods_ds.content= fixture(File.join("mods_articles","mods_article1.xml")).read
     end
 
-    it 'should throw an exception if we have initialized the internal_solr_doc.' do
+    it "should throw an exception if we have initialized the internal_solr_doc." do
       @mods_ds.from_solr(@solr_doc)
       found_exception = false
       begin
-        @mods_ds.update_values([{ ':person' => '0' }, 'role', 'text'] => { '0' => 'role1', '1' => 'role2', '2' => 'role3' })
+        @mods_ds.update_values([{":person"=>"0"}, "role", "text"]=>{"0"=>"role1", "1"=>"role2", "2"=>"role3"})
       rescue
         found_exception = true
       end
-      expect(found_exception).to eq(true)
+      found_exception.should == true
     end
 
-    it 'should update a value internally call OM::XML::TermValueOperators::update_values if internal_solr_doc is not set' do
-      allow(@mods_ds).to receive(:om_update_values).once
-      term_pointer = [:name, :role, :roleTerm]
-      @mods_ds.update_values([{ ':person' => '0' }, 'role', 'text'] => { '0' => 'role1', '1' => 'role2', '2' => 'role3' })
+    it "should update a value internally call OM::XML::TermValueOperators::update_values if internal_solr_doc is not set" do
+      @mods_ds.stub(:om_update_values).once()
+      term_pointer = [:name,:role,:roleTerm]
+      @mods_ds.update_values([{":person"=>"0"}, "role", "text"]=>{"0"=>"role1", "1"=>"role2", "2"=>"role3"})
     end
 
-    it 'should set changed to true' do
+    it "should set changed to true" do
       mods_ds = Hydra::ModsArticleDatastream.new
-      mods_ds.content = fixture(File.join('mods_articles', 'mods_article1.xml')).read
-      mods_ds.update_values([{ ':person' => '0' }, 'role', 'text'] => { '0' => 'role1', '1' => 'role2', '2' => 'role3' })
-      expect(mods_ds).to be_changed
+      mods_ds.content=fixture(File.join("mods_articles","mods_article1.xml")).read
+      mods_ds.update_values([{":person"=>"0"}, "role", "text"]=>{"0"=>"role1", "1"=>"role2", "2"=>"role3"})
+      mods_ds.should be_changed
     end
   end
 
   describe '.term_values' do
+
     before(:each) do
       @mods_ds = ActiveFedora::OmDatastream.new
-      @mods_ds.content = fixture(File.join('mods_articles', 'mods_article1.xml')).read
+      @mods_ds.content=fixture(File.join("mods_articles","mods_article1.xml")).read
     end
 
-    it 'should call OM::XML::term_values if internal_solr_doc is not set and return values from xml' do
-      allow(@mods_ds).to receive(:om_term_values).once
-      term_pointer = [:name, :role, :roleTerm]
+    it "should call OM::XML::term_values if internal_solr_doc is not set and return values from xml" do
+      @mods_ds.stub(:om_term_values).once()
+      term_pointer = [:name,:role,:roleTerm]
       @mods_ds.term_values(*term_pointer)
     end
 
     # we will know this is working because solr_doc and xml are not synced so that wrong return mechanism can be detected
-    it 'should call get_values_from_solr if internal_solr_doc is set' do
+    it "should call get_values_from_solr if internal_solr_doc is set" do
       @mods_ds.from_solr(@solr_doc)
-      term_pointer = [:name, :role, :roleTerm]
-      allow(@mods_ds).to receive(:get_values_from_solr).once
+      term_pointer = [:name,:role,:roleTerm]
+      @mods_ds.stub(:get_values_from_solr).once()
       @mods_ds.term_values(*term_pointer)
     end
   end
 
   describe "an instance that exists in the datastore, but hasn't been loaded" do
-    before do
+    before do 
       class MyObj < ActiveFedora::Base
         has_metadata 'descMetadata', type: Hydra::ModsArticleDatastream
       end
@@ -379,10 +373,10 @@ describe ActiveFedora::OmDatastream do
       @obj.destroy
       Object.send(:remove_const, :MyObj)
     end
-    subject { @obj.reload.descMetadata }
-    it 'should not load the descMetadata datastream when calling content_changed?' do
-      expect(@obj.inner_object.repository).not_to receive(:datastream_dissemination).with(hash_including(dsid: 'descMetadata'))
-      expect(subject).not_to be_content_changed
+    subject { @obj.reload.descMetadata } 
+    it "should not load the descMetadata datastream when calling content_changed?" do
+      @obj.inner_object.repository.should_not_receive(:datastream_dissemination).with(hash_including(:dsid=>'descMetadata'))
+      subject.should_not be_content_changed
     end
   end
 end