hydra-works 0.0.1 → 0.1.0

data/spec/hydra/works/services/generic_file/add_file_spec.rb

@@ -1,28 +0,0 @@
-require 'spec_helper'
-
-describe Hydra::Works::AddFileToGenericFile do
-
-  let(:generic_file)        { Hydra::Works::GenericFile::Base.create }
-  let(:filename)            { "sample-file.pdf" }
-  let(:path)                { File.join(fixture_path, filename) }
-  let(:type)                { ::RDF::URI("http://pcdm.org/ExtractedText") }
-  let(:replace)             { false }
-
-  it "adds the given file and applies the specified type to it" do
-    Hydra::Works::AddFileToGenericFile.call(generic_file, path, type, replace)
-    expect(generic_file.filter_files_by_type(type).first.content).to start_with("%PDF-1.3")
-  end
-
-  context "type_to_uri" do
-    it "maps supported symbols to corresponding URIs" do
-      expect(Hydra::Works::AddFileToGenericFile.send(:type_to_uri, :original_file)).to eq(::RDF::URI("http://pcdm.org/OriginalFile"))
-      expect(Hydra::Works::AddFileToGenericFile.send(:type_to_uri, :thumbnail)).to eq(::RDF::URI("http://pcdm.org/ThumbnailImage"))
-      expect(Hydra::Works::AddFileToGenericFile.send(:type_to_uri, :extracted_text)).to eq(::RDF::URI("http://pcdm.org/ExtractedText"))
-    end
-
-    it "converts URI strings to RDF::URI" do
-      expect(Hydra::Works::AddFileToGenericFile.send(:type_to_uri, "http://example.com/CustomURI" )).to eq(::RDF::URI("http://example.com/CustomURI"))
-    end
-  end
-
-end

data/spec/hydra/works/services/generic_file/add_generic_file_spec.rb

@@ -1,174 +0,0 @@
-require 'spec_helper'
-
-describe Hydra::Works::AddGenericFileToGenericFile do
-
-  let(:subject) { Hydra::Works::GenericFile::Base.create }
-
-  describe '#call' do
-    context 'with acceptable generic_files' do
-      let(:generic_file1)   { Hydra::Works::GenericFile::Base.create }
-      let(:generic_file2)   { Hydra::Works::GenericFile::Base.create }
-      let(:generic_file3)   { Hydra::Works::GenericFile::Base.create }
-      let(:generic_file4)   { Hydra::Works::GenericFile::Base.create }
-      let(:generic_file5)   { Hydra::Works::GenericFile::Base.create }
-
-      it 'should aggregate generic_files in a sub-generic_file of a generic_file' do
-        Hydra::Works::AddGenericFileToGenericFile.call( generic_file1, generic_file2 )
-        generic_file1.save
-        Hydra::Works::AddGenericFileToGenericFile.call( generic_file2, generic_file3 )
-        generic_file2.save
-        expect( Hydra::Works::GetGenericFilesFromGenericFile.call( generic_file1 ) ).to eq [generic_file2]
-        expect( Hydra::Works::GetGenericFilesFromGenericFile.call( generic_file2 ) ).to eq [generic_file3]
-      end
-
-      it 'should allow generic_files to repeat' do
-        Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file1 )
-        Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file2 )
-        Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file1 )
-        expect( Hydra::Works::GetGenericFilesFromGenericFile.call( subject ) ).to eq [generic_file1,generic_file2,generic_file1]
-      end
-
-      context 'with files and generic_files' do
-        let(:file1) { subject.files.build }
-        let(:file2) { subject.files.build }
-
-        before do
-          file1.content = "I'm a file"
-          file2.content = "I am too"
-          Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file1 )
-          Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file2 )
-          subject.save!
-        end
-
-        it 'should add generic_file to generic_file with generic_files and files' do
-          Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file3 )
-          expect( Hydra::Works::GetGenericFilesFromGenericFile.call( subject ) ).to eq [generic_file1,generic_file2,generic_file3]
-        end
-
-        it 'should solrize member ids' do
-          skip 'skipping this test because issue #109 needs to be addressed' do
-          expect(subject.to_solr["generic_files_ssim"]).to include(generic_file2.id,generic_file1.id)
-          expect(subject.to_solr["generic_files_ssim"]).not_to include(file1.id,file2.id)
-          expect(subject.to_solr["files_ssim"]).to include(file1.id,file2.id)
-          expect(subject.to_solr["files_ssim"]).not_to include(generic_file1.id,generic_file2.id)
-        end
-        end
-      end
-
-      describe 'aggregates generic_files that implement Hydra::Works::GenericFileBehavior' do
-        before do
-          class DummyIncFile < ActiveFedora::Base
-            include Hydra::Works::GenericFileBehavior
-          end
-        end
-        after { Object.send(:remove_const, :DummyIncFile) }
-        let(:ifile1) { DummyIncFile.create }
-
-        it 'should accept implementing generic_file as a child' do
-          Hydra::Works::AddGenericFileToGenericFile.call( subject, ifile1 )
-          subject.save
-          expect( Hydra::Works::GetGenericFilesFromGenericFile.call( subject ) ).to eq [ifile1]
-        end
-      end
-
-      describe 'aggregates generic_files that extend Hydra::Works::GenericFile::Base' do
-        before do
-          class DummyExtFile < Hydra::Works::GenericFile::Base
-          end
-        end
-        after { Object.send(:remove_const, :DummyExtFile) }
-        let(:efile1) { DummyExtFile.create }
-
-        it 'should accept extending generic_file as a child' do
-          Hydra::Works::AddGenericFileToGenericFile.call( subject, efile1 )
-          subject.save
-          expect( Hydra::Works::GetGenericFilesFromGenericFile.call( subject ) ).to eq [efile1]
-        end
-      end
-    end
-
-    context 'with unacceptable child generic_files' do
-      let(:collection1)      { Hydra::Works::Collection.create }
-      let(:generic_work1)    { Hydra::Works::GenericWork::Base.create }
-      let(:pcdm_collection1) { Hydra::PCDM::Collection.create }
-      let(:pcdm_object1)     { Hydra::PCDM::Object.create }
-      let(:pcdm_file1)       { Hydra::PCDM::File.new }
-      let(:non_PCDM_object)  { "I'm not a PCDM object" }
-      let(:af_base_object)   { ActiveFedora::Base.create }
-
-      let(:error_message) { 'child_generic_file must be a hydra-works generic file' }
-
-      it 'should NOT aggregate Hydra::Works::Collection in generic files aggregation' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( subject, collection1 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT aggregate Hydra::Works::GenericWork in generic files aggregation' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_work1 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT aggregate Hydra::PCDM::Collections in generic files aggregation' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( subject, pcdm_collection1 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT aggregate Hydra::PCDM::Objects in generic files aggregation' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( subject, pcdm_object1 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT aggregate Hydra::PCDM::Files in generic files aggregation' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( subject, pcdm_file1 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT aggregate non-PCDM objects in generic files aggregation' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( subject, non_PCDM_object ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT aggregate AF::Base objects in generic files aggregation' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( subject, af_base_object ) }.to raise_error(ArgumentError,error_message)
-      end
-    end
-
-    context 'with unacceptable parent generic works' do
-      let(:collection1)      { Hydra::Works::Collection.create }
-      let(:generic_work1)    { Hydra::Works::GenericWork::Base.create }
-      let(:generic_file1)    { Hydra::Works::GenericFile::Base.create }
-      let(:generic_file2)    { Hydra::Works::GenericFile::Base.create }
-      let(:pcdm_collection1) { Hydra::PCDM::Collection.create }
-      let(:pcdm_object1)     { Hydra::PCDM::Object.create }
-      let(:pcdm_file1)       { Hydra::PCDM::File.new }
-      let(:non_PCDM_object)  { "I'm not a PCDM object" }
-      let(:af_base_object)   { ActiveFedora::Base.create }
-
-      let(:error_message) { 'parent_generic_file must be a hydra-works generic file' }
-
-      it 'should NOT accept Hydra::Works::Collection as parent generic file' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( collection1, generic_file2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept Hydra::Works::GenericWork as parent generic file' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( generic_work1, generic_file2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept Hydra::PCDM::Collections as parent generic file' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( pcdm_collection1, generic_file2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept Hydra::PCDM::Objects as parent generic file' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( pcdm_object1, generic_file2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept Hydra::PCDM::Files as parent generic file' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( pcdm_file1, generic_file2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept non-PCDM objects as parent generic file' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( non_PCDM_object, generic_file2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept AF::Base objects as parent generic file' do
-        expect{ Hydra::Works::AddGenericFileToGenericFile.call( af_base_object, generic_file2 ) }.to raise_error(ArgumentError,error_message)
-      end
-    end
-
-  end
-
-end

data/spec/hydra/works/services/generic_file/add_related_object_spec.rb

@@ -1,150 +0,0 @@
-require 'spec_helper'
-
-describe Hydra::Works::AddRelatedObjectToGenericFile do
-
-  let(:subject) { Hydra::Works::GenericFile::Base.create }
-
-  describe '#call' do
-
-    context 'with acceptable related objects' do
-      let(:object1) { Hydra::PCDM::Object.create }
-      let(:object2) { Hydra::PCDM::Object.create }
-      let(:generic_work1) { Hydra::Works::GenericWork::Base.create }
-      let(:generic_work2) { Hydra::Works::GenericWork::Base.create }
-      let(:generic_file1) { Hydra::Works::GenericFile::Base.create }
-      let(:generic_file2) { Hydra::Works::GenericFile::Base.create }
-
-      it 'should add various types of related objects to generic_file' do
-        Hydra::Works::AddRelatedObjectToGenericFile.call( subject, generic_work1 )
-        Hydra::Works::AddRelatedObjectToGenericFile.call( subject, generic_file1 )
-        Hydra::Works::AddRelatedObjectToGenericFile.call( subject, object1 )
-        related_objects = Hydra::Works::GetRelatedObjectsFromGenericFile.call( subject )
-        expect( related_objects.include? generic_work1 ).to be true
-        expect( related_objects.include? generic_file1 ).to be true
-        expect( related_objects.include? object1 ).to be true
-        expect( related_objects.size ).to eq 3
-      end
-
-      context 'with files and generic_files' do
-        let(:file1) { subject.files.build }
-        let(:file2) { subject.files.build }
-
-        before do
-          file1.content = "I'm a file"
-          file2.content = "I am too"
-          Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file1 )
-          Hydra::Works::AddRelatedObjectToGenericFile.call( subject, object1 )
-          subject.save!
-        end
-
-        it 'should add a related object to a generic_file with files and generic_files' do
-          Hydra::Works::AddRelatedObjectToGenericFile.call( subject, object2 )
-          related_objects = Hydra::Works::GetRelatedObjectsFromGenericFile.call( subject )
-          expect( related_objects.include? object1 ).to be true
-          expect( related_objects.include? object2 ).to be true
-          expect( related_objects.size ).to eq 2
-        end
-
-        it 'should solrize member ids' do
-          skip 'skipping this test because issue #109 needs to be addressed' do
-          expect(subject.to_solr["generic_files_ssim"]).to include(generic_file2.id,generic_file1.id)
-          expect(subject.to_solr["generic_files_ssim"]).not_to include(object1.id,object2.id,file1.id,file2.id)
-          expect(subject.to_solr["files_ssim"]).to include(file1.id,file2.id)
-          expect(subject.to_solr["files_ssim"]).not_to include(object1.id,object2.id,generic_file1.id,generic_file2.id)
-          expect(subject.to_solr["related_objects_ssim"]).to include(object1.id,object2.id)
-          expect(subject.to_solr["related_objects_ssim"]).not_to include(generic_file2.id,generic_file1.id,file1.id,file2.id)
-        end
-        end
-      end
-    end
-
-    context 'with unacceptable child related objects' do
-      let(:collection1)      { Hydra::Works::Collection.create }
-      let(:pcdm_collection1) { Hydra::PCDM::Collection.create }
-      let(:pcdm_file1)       { Hydra::PCDM::File.new }
-      let(:non_PCDM_object)  { "I'm not a PCDM object" }
-      let(:af_base_object)   { ActiveFedora::Base.create }
-
-      let(:error_message) { 'child_related_object must be a pcdm object' }
-
-      it 'should NOT aggregate Hydra::Works::Collection in related objects aggregation' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( subject, collection1 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT aggregate Hydra::PCDM::Collections in related objects aggregation' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( subject, pcdm_collection1 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT aggregate Hydra::PCDM::Files in related objects aggregation' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( subject, pcdm_file1 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT aggregate non-PCDM objects in related objects aggregation' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( subject, non_PCDM_object ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT aggregate AF::Base objects in related objects aggregation' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( subject, af_base_object ) }.to raise_error(ArgumentError,error_message)
-      end
-    end
-
-    context 'with unacceptable parent generic work' do
-      let(:collection1)      { Hydra::Works::Collection.create }
-      let(:generic_work1)    { Hydra::Works::GenericWork::Base.create }
-      let(:pcdm_collection1) { Hydra::PCDM::Collection.create }
-      let(:pcdm_object1)     { Hydra::PCDM::Object.create }
-      let(:pcdm_object2)     { Hydra::PCDM::Object.create }
-      let(:pcdm_file1)       { Hydra::PCDM::File.new }
-      let(:non_PCDM_object)  { "I'm not a PCDM object" }
-      let(:af_base_object)   { ActiveFedora::Base.create }
-
-      let(:error_message) { 'parent_generic_file must be a hydra-works generic file' }
-
-      it 'should NOT accept Hydra::Works::Collection as parent generic file' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( collection1, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept Hydra::Works::GenericWork as parent generic file' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( generic_work1, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept Hydra::PCDM::Collections as parent generic file' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( pcdm_collection1, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept Hydra::PCDM::Objects as parent generic file' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( pcdm_object1, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept Hydra::PCDM::Files as parent generic file' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( pcdm_file1, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept non-PCDM objects as parent generic file' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( non_PCDM_object, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
-      end
-
-      it 'should NOT accept AF::Base objects as parent generic file' do
-        expect{ Hydra::Works::AddRelatedObjectToGenericFile.call( af_base_object, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
-      end
-    end
-
-
-    context 'with invalid behaviors' do
-      let(:object1) { Hydra::PCDM::Object.create }
-      let(:object2) { Hydra::PCDM::Object.create }
-
-      it 'should NOT allow related objects to repeat' do
-        skip 'skipping this test because issue pcdm#92 needs to be addressed' do
-        Hydra::Works::AddRelatedObjectToGenericFile.call( subject, object1 )
-        Hydra::Works::AddRelatedObjectToGenericFile.call( subject, object2 )
-        Hydra::Works::AddRelatedObjectToGenericFile.call( subject, object1 )
-        related_objects = Hydra::Works::GetRelatedObjectsFromGenericFile.call( subject )
-        expect( related_objects.include? object1 ).to be true
-        expect( related_objects.include? object2 ).to be true
-        expect( related_objects.size ).to eq 2
-      end
-      end
-    end
-  end
-end

data/spec/hydra/works/services/generic_file/get_generic_files_spec.rb

@@ -1,19 +0,0 @@
-require 'spec_helper'
-
-describe Hydra::Works::GetGenericFilesFromGenericFile do
-
-  subject { Hydra::Works::GenericFile::Base.create }
-
-  let(:generic_file1) { Hydra::Works::GenericFile::Base.create }
-  let(:generic_file2) { Hydra::Works::GenericFile::Base.create }
-
-  describe '#call' do
-    it 'should return generic_files when generic_files are aggregated' do
-      Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file1 )
-      Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file2 )
-      subject.save
-      expect(Hydra::Works::GetGenericFilesFromGenericFile.call( subject )).to eq [generic_file1,generic_file2]
-    end
-
-  end
-end

data/spec/hydra/works/services/generic_file/get_related_objects_spec.rb

@@ -1,45 +0,0 @@
-require 'spec_helper'
-
-describe Hydra::Works::GetRelatedObjectsFromGenericFile do
-
-  subject { Hydra::Works::GenericFile::Base.create }
-
-  let(:object1) { Hydra::PCDM::Object.create }
-  let(:object2) { Hydra::PCDM::Object.create }
-
-  let(:generic_work1) { Hydra::Works::GenericWork::Base.create }
-  let(:generic_file1) { Hydra::Works::GenericFile::Base.create }
-  let(:generic_file2) { Hydra::Works::GenericFile::Base.create }
-
-  describe '#call' do
-    context 'with generic files' do
-      before do
-        Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file1 )
-        Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file2 )
-        subject.save
-      end
-
-      it 'should return empty array when only generic files are aggregated' do
-        expect(Hydra::Works::GetRelatedObjectsFromGenericFile.call( subject )).to eq []
-      end
-
-      it 'should only return related objects' do
-        Hydra::Works::AddRelatedObjectToGenericFile.call( subject, object2 )
-        expect(Hydra::Works::GetRelatedObjectsFromGenericFile.call( subject )).to eq [object2]
-      end
-
-      it 'should return related objects of various types' do
-        Hydra::Works::AddRelatedObjectToGenericFile.call( subject, generic_work1 )
-        Hydra::Works::AddRelatedObjectToGenericFile.call( subject, generic_file1 )
-        Hydra::Works::AddRelatedObjectToGenericFile.call( subject, object1 )
-        related_objects = Hydra::Works::GetRelatedObjectsFromGenericFile.call( subject )
-        expect( related_objects.include? object1 ).to be true
-        expect( related_objects.include? generic_work1 ).to be true
-        expect( related_objects.include? generic_file1 ).to be true
-        expect( related_objects.size ).to eq 3
-      end
-   end
-  end
-end
-
-

data/spec/hydra/works/services/generic_file/remove_generic_file_spec.rb

@@ -1,124 +0,0 @@
-require 'spec_helper'
-
-describe Hydra::Works::RemoveGenericFileFromGenericFile do
-
-  subject { Hydra::Works::GenericFile::Base.create }
-
-  let(:generic_file1) { Hydra::Works::GenericFile::Base.create }
-  let(:generic_file2) { Hydra::Works::GenericFile::Base.create }
-  let(:generic_file3) { Hydra::Works::GenericFile::Base.create }
-  let(:generic_file4) { Hydra::Works::GenericFile::Base.create }
-  let(:generic_file5) { Hydra::Works::GenericFile::Base.create }
-
-  describe '#call' do
-    context 'when multiple collections' do
-      before do
-        Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file1 )
-        Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file2 )
-        Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file3 )
-        Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file4 )
-        Hydra::Works::AddGenericFileToGenericFile.call( subject, generic_file5 )
-        subject.save
-        expect( Hydra::Works::GetGenericFilesFromGenericFile.call( subject.reload )).to eq [generic_file1,generic_file2,generic_file3,generic_file4,generic_file5]
-      end
-
-      it 'should remove first collection' do
-        expect( Hydra::Works::RemoveGenericFileFromGenericFile.call( subject, generic_file1 ) ).to eq generic_file1
-        expect( Hydra::Works::GetGenericFilesFromGenericFile.call( subject.reload )).to eq [generic_file2,generic_file3,generic_file4,generic_file5]
-      end
-
-      it 'should remove last collection' do
-        expect( Hydra::Works::RemoveGenericFileFromGenericFile.call( subject, generic_file5 ) ).to eq generic_file5
-        expect( Hydra::Works::GetGenericFilesFromGenericFile.call( subject.reload )).to eq [generic_file1,generic_file2,generic_file3,generic_file4]
-      end
-
-      it 'should remove middle collection' do
-        expect( Hydra::Works::RemoveGenericFileFromGenericFile.call( subject, generic_file3 ) ).to eq generic_file3
-        expect( Hydra::Works::GetGenericFilesFromGenericFile.call( subject.reload )).to eq [generic_file1,generic_file2,generic_file4,generic_file5]
-      end
-    end
-  end
-
-  context 'with unacceptable generic files' do
-    let(:collection1)    { Hydra::Works::Collection.create }
-    let(:generic_work1)    { Hydra::Works::GenericWork::Base.create }
-    let(:pcdm_collection1) { Hydra::PCDM::Collection.create }
-    let(:pcdm_object1)     { Hydra::PCDM::Object.create }
-    let(:pcdm_file1)       { Hydra::PCDM::File.new }
-    let(:non_PCDM_object)  { "I'm not a PCDM object" }
-    let(:af_base_object)   { ActiveFedora::Base.create }
-
-    let(:error_type)    { ArgumentError }
-    let(:error_message) { 'child_generic_file must be a hydra-works generic file' }
-
-    it 'should NOT remove Hydra::Works::Collection from generic_files aggregation' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( subject, collection1 ) }.to raise_error(ArgumentError,error_message)
-    end
-
-    it 'should NOT remove Hydra::Works::GenericWork from generic_files aggregation' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( subject, generic_work1 ) }.to raise_error(ArgumentError,error_message)
-    end
-
-    it 'should NOT remove Hydra::PCDM::Collections from generic_files aggregation' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( subject, pcdm_collection1 ) }.to raise_error(ArgumentError,error_message)
-    end
-
-    it 'should NOT remove Hydra::PCDM::Objects from generic_files aggregation' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( subject, pcdm_object1 ) }.to raise_error(error_type,error_message)
-    end
-
-    it 'should NOT remove Hydra::PCDM::Files from generic_files aggregation' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( subject, pcdm_file1 ) }.to raise_error(error_type,error_message)
-    end
-
-    it 'should NOT remove non-PCDM objects from generic_files aggregation' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( subject, non_PCDM_object ) }.to raise_error(error_type,error_message)
-    end
-
-    it 'should NOT remove AF::Base objects from generic_works aggregation' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( subject, af_base_object ) }.to raise_error(error_type,error_message)
-    end
-  end
-
-  context 'with unacceptable parent generic file' do
-    let(:generic_file2)    { Hydra::Works::GenericFile::Base.create }
-    let(:collection1)      { Hydra::Works::Collection.create }
-    let(:generic_work1)    { Hydra::Works::GenericWork::Base.create }
-    let(:pcdm_collection1) { Hydra::PCDM::Collection.create }
-    let(:pcdm_object1)     { Hydra::PCDM::Object.create }
-    let(:pcdm_file1)       { Hydra::PCDM::File.new }
-    let(:non_PCDM_object)  { "I'm not a PCDM object" }
-    let(:af_base_object)   { ActiveFedora::Base.create }
-
-    let(:error_type)    { ArgumentError }
-    let(:error_message) { 'parent_generic_file must be a hydra-works generic file' }
-
-    it 'should NOT accept Hydra::Works::Collection as parent generic file' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( collection1, generic_file2 ) }.to raise_error(ArgumentError,error_message)
-    end
-
-    it 'should NOT accept Hydra::Works::GenericWork as parent generic file' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( generic_work1, generic_file2 ) }.to raise_error(ArgumentError,error_message)
-    end
-
-    it 'should NOT accept Hydra::PCDM::Collections as parent generic file' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( pcdm_collection1, generic_file2 ) }.to raise_error(ArgumentError,error_message)
-    end
-
-    it 'should NOT accept Hydra::PCDM::Objects as parent generic file' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( pcdm_object1, generic_file2 ) }.to raise_error(error_type,error_message)
-    end
-
-    it 'should NOT accept Hydra::PCDM::Files as parent generic file' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( pcdm_file1, generic_file2 ) }.to raise_error(error_type,error_message)
-    end
-
-    it 'should NOT accept non-PCDM objects as parent generic file' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( non_PCDM_object, generic_file2 ) }.to raise_error(error_type,error_message)
-    end
-
-    it 'should NOT accept AF::Base objects as parent generic file' do
-      expect{ Hydra::Works::RemoveGenericFileFromGenericFile.call( af_base_object, generic_file2 ) }.to raise_error(error_type,error_message)
-    end
-  end
-end