hydra-works 0.0.1

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

@@ -0,0 +1,82 @@
+require 'spec_helper'
+
+describe Hydra::Works::UploadFileToGenericFile do
+
+  let(:generic_work)        { Hydra::Works::GenericWork::Base.create }
+  let(:generic_file)        { Hydra::Works::GenericFile::Base.create }
+  let(:filename)            { "sample-file.pdf" }
+  let(:file)                { File.join(fixture_path, filename) }
+  let(:updated_filename)    { "updated-file.txt"}
+  let(:updated_file)        { File.join(fixture_path, updated_filename) }
+  let(:mime_type)           { "application/pdf" }
+  let(:updated_mime_type)   { "text/plain" }  
+  let(:additional_services) { [Hydra::Works::GenerateThumbnail] }
+
+  context "without a proper generic file" do
+    let(:error_message) { "supplied object must be a generic file" }
+    it "raises an error" do
+      expect( lambda { described_class.call(generic_work, file) }).to raise_error(ArgumentError, error_message)
+    end
+  end
+
+  context "without a proper path" do
+    let(:error_message) { "supplied path must be a string" }
+    it "raises an error" do
+      expect( lambda { described_class.call(generic_file, ["this isn't a path"]) }).to raise_error(ArgumentError, error_message)
+    end
+  end
+
+  context "with a non-existent file" do
+    let(:error_message) { "supplied path to file does not exist" }
+    it "raises an error" do
+      expect( lambda { described_class.call(generic_file, "/i/do/not/exist") }).to raise_error(ArgumentError, error_message)
+    end
+  end
+
+  context "with an empty generic file" do
+    before { described_class.call(generic_file, file, additional_services) }
+
+    describe "the uploaded file" do
+      subject { generic_file.original_file }
+      it "has content" do
+        expect(subject.content).to start_with("%PDF-1.3")
+      end
+      it "has a mime type" do
+        expect(subject.mime_type).to eql mime_type
+      end
+      it "has a name" do
+        expect(subject.original_name).to eql filename
+      end
+    end
+
+    describe "the generic file's generated files" do
+      subject { generic_file }
+      it "has a thumbnail" do
+        expect(subject.thumbnail.content).not_to be_nil
+      end
+    end
+  end
+
+  context "when replacing an existing file" do
+    before do
+      described_class.call(generic_file, file, additional_services)
+      described_class.call(generic_file, updated_file, additional_services, replace: true)
+      generic_file.reload
+    end
+
+    describe "the new file" do
+      subject { generic_file.original_file }
+      it "has updated content" do
+        expect(subject.content).to eql File.open(updated_file).read
+      end
+      it "has an updated name" do
+        expect(subject.original_name).to eql updated_filename
+      end
+      it "has a updated mime type" do
+        expect(subject.mime_type).to eql updated_mime_type
+      end 
+    end
+  end
+
+end
+  

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

@@ -0,0 +1,156 @@
+require 'spec_helper'
+
+describe Hydra::Works::AddGenericFileToGenericWork do
+
+  let(:subject) { Hydra::Works::GenericWork::Base.create }
+
+  describe '#call' do
+    context 'with acceptable generic_works' do
+      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 }
+      let(:generic_file3)   { Hydra::Works::GenericFile::Base.create }
+
+      context 'with generic_files and generic_works' do
+        before do
+          Hydra::Works::AddGenericFileToGenericWork.call( subject, generic_file1 )
+          Hydra::Works::AddGenericFileToGenericWork.call( subject, generic_file2 )
+          Hydra::Works::AddGenericWorkToGenericWork.call( subject, generic_work1 )
+          Hydra::Works::AddGenericWorkToGenericWork.call( subject, generic_work2 )
+          subject.save
+        end
+
+        it 'should add generic_file to generic_work with generic_files and generic_works' do
+          Hydra::Works::AddGenericFileToGenericWork.call( subject, generic_file3 )
+          expect( Hydra::Works::GetGenericFilesFromGenericWork.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_works_ssim"]).to include(generic_work1.id,generic_work2.id)
+          expect(subject.to_solr["generic_works_ssim"]).not_to include(generic_file1.id,generic_file2.id,generic_file3.id)
+          expect(subject.to_solr["generic_files_ssim"]).to include(generic_file1.id,generic_file2.id,generic_file3.id)
+          expect(subject.to_solr["generic_files_ssim"]).not_to include(generic_work1.id,generic_work2.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::AddGenericFileToGenericWork.call( subject, ifile1 )
+          subject.save
+          expect( Hydra::Works::GetGenericFilesFromGenericWork.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::AddGenericFileToGenericWork.call( subject, efile1 )
+          subject.save
+          expect( Hydra::Works::GetGenericFilesFromGenericWork.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::AddGenericFileToGenericWork.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::AddGenericFileToGenericWork.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::AddGenericFileToGenericWork.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::AddGenericFileToGenericWork.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::AddGenericFileToGenericWork.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::AddGenericFileToGenericWork.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::AddGenericFileToGenericWork.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_work2)    { Hydra::Works::GenericWork::Base.create }
+      let(:generic_file1)    { 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_work must be a hydra-works generic work' }
+
+      it 'should NOT accept Hydra::Works::Collection as parent generic work' do
+        expect{ Hydra::Works::AddGenericFileToGenericWork.call( collection1, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::Works::GenericFile as parent generic work' do
+        expect{ Hydra::Works::AddGenericFileToGenericWork.call( generic_file1, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::PCDM::Collections as parent generic work' do
+        expect{ Hydra::Works::AddGenericFileToGenericWork.call( pcdm_collection1, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::PCDM::Objects as parent generic work' do
+        expect{ Hydra::Works::AddGenericFileToGenericWork.call( pcdm_object1, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::PCDM::Files as parent generic work' do
+        expect{ Hydra::Works::AddGenericFileToGenericWork.call( pcdm_file1, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept non-PCDM objects as parent generic work' do
+        expect{ Hydra::Works::AddGenericFileToGenericWork.call( non_PCDM_object, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept AF::Base objects as parent generic work' do
+        expect{ Hydra::Works::AddGenericFileToGenericWork.call( af_base_object, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+    end
+
+  end
+
+end

data/spec/hydra/works/services/generic_work/add_generic_work_spec.rb

@@ -0,0 +1,157 @@
+require 'spec_helper'
+
+describe Hydra::Works::AddGenericWorkToGenericWork do
+
+  let(:subject) { Hydra::Works::GenericWork::Base.create }
+
+  describe '#call' do
+    context 'with acceptable generic_works' do
+      let(:generic_work1) { Hydra::Works::GenericWork::Base.create }
+      let(:generic_work2) { Hydra::Works::GenericWork::Base.create }
+      let(:generic_work3) { Hydra::Works::GenericWork::Base.create }
+      let(:generic_work4) { Hydra::Works::GenericWork::Base.create }
+      let(:generic_work5) { Hydra::Works::GenericWork::Base.create }
+      let(:generic_file1)   { Hydra::Works::GenericFile::Base.create }
+      let(:generic_file2)   { Hydra::Works::GenericFile::Base.create }
+
+      context 'with generic_files and generic_works' do
+        before do
+          Hydra::Works::AddGenericFileToGenericWork.call( subject, generic_file1 )
+          Hydra::Works::AddGenericFileToGenericWork.call( subject, generic_file2 )
+          Hydra::Works::AddGenericWorkToGenericWork.call( subject, generic_work1 )
+          Hydra::Works::AddGenericWorkToGenericWork.call( subject, generic_work2 )
+          subject.save
+        end
+
+        it 'should add generic_work to generic_work with generic_files and generic_works' do
+          Hydra::Works::AddGenericWorkToGenericWork.call( subject, generic_work3 )
+          expect( Hydra::Works::GetGenericWorksFromGenericWork.call( subject ) ).to eq [generic_work1,generic_work2,generic_work3]
+        end
+
+        it 'should solrize member ids' do
+          skip 'skipping this test because issue #109 needs to be addressed' do
+          expect(subject.to_solr['generic_works_ssim']).to include(generic_work1.id,generic_work2.id,generic_work3.id)
+          expect(subject.to_solr['generic_works_ssim']).not_to include(generic_file1.id,generic_file2.id)
+          expect(subject.to_solr['generic_files_ssim']).to include(generic_file1.id,generic_file2.id)
+          expect(subject.to_solr['generic_files_ssim']).not_to include(generic_work1.id,generic_work2.id,generic_work3.id)
+        end
+        end
+      end
+
+      describe 'aggregates generic_works that implement Hydra::Works::GenericWorkBehavior' do
+        before do
+          class DummyIncWork < ActiveFedora::Base
+            include Hydra::Works::GenericWorkBehavior
+          end
+        end
+        after { Object.send(:remove_const, :DummyIncWork) }
+        let(:iwork1) { DummyIncWork.create }
+
+        it 'should accept implementing generic_work as a child' do
+          Hydra::Works::AddGenericWorkToGenericWork.call( subject, iwork1 )
+          subject.save
+          expect( Hydra::Works::GetGenericWorksFromGenericWork.call( subject ) ).to eq [iwork1]
+        end
+      end
+
+      describe 'aggregates generic_works that extend Hydra::Works::GenericWork::Base' do
+        before do
+          class DummyExtWork < Hydra::Works::GenericWork::Base
+          end
+        end
+        after { Object.send(:remove_const, :DummyExtWork) }
+        let(:ework1) { DummyExtWork.create }
+
+        it 'should accept extending generic_work as a child' do
+          Hydra::Works::AddGenericWorkToGenericWork.call( subject, ework1 )
+          subject.save
+          expect( Hydra::Works::GetGenericWorksFromGenericWork.call( subject ) ).to eq [ework1]
+        end
+      end
+    end
+
+    context 'with unacceptable child generic_works' do
+      let(:collection1)      { Hydra::Works::Collection.create }
+      let(:generic_file1)    { 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) { 'child_generic_work must be a hydra-works generic work' }
+
+      it 'should NOT aggregate Hydra::Works::Collection in generic works aggregation' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( subject, collection1 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT aggregate Hydra::Works::GenericFile in generic works aggregation' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( subject, generic_file1 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT aggregate Hydra::PCDM::Collections in generic works aggregation' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( subject, pcdm_collection1 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT aggregate Hydra::PCDM::Objects in generic works aggregation' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( subject, pcdm_object1 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT aggregate Hydra::PCDM::Files in generic works aggregation' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( subject, pcdm_file1 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT aggregate non-PCDM objects in generic works aggregation' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( subject, non_PCDM_object ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT aggregate AF::Base objects in generic works aggregation' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.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_work2)    { Hydra::Works::GenericWork::Base.create }
+      let(:generic_file1)    { 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_work must be a hydra-works generic work' }
+
+      it 'should NOT accept Hydra::Works::Collection as parent generic work' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( collection1, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::Works::GenericFile as parent generic work' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( generic_file1, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::PCDM::Collections as parent generic work' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( pcdm_collection1, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::PCDM::Objects as parent generic work' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( pcdm_object1, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::PCDM::Files as parent generic work' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( pcdm_file1, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept non-PCDM objects as parent generic work' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( non_PCDM_object, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept AF::Base objects as parent generic work' do
+        expect{ Hydra::Works::AddGenericWorkToGenericWork.call( af_base_object, generic_work2 ) }.to raise_error(ArgumentError,error_message)
+      end
+    end
+
+  end
+
+end

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

@@ -0,0 +1,148 @@
+require 'spec_helper'
+
+describe Hydra::Works::AddRelatedObjectToGenericWork do
+
+  let(:subject) { Hydra::Works::GenericWork::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_work' do
+        Hydra::Works::AddRelatedObjectToGenericWork.call( subject, generic_work1 )
+        Hydra::Works::AddRelatedObjectToGenericWork.call( subject, generic_file1 )
+        Hydra::Works::AddRelatedObjectToGenericWork.call( subject, object1 )
+        related_objects = Hydra::Works::GetRelatedObjectsFromGenericWork.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 generic_works and generic_files' do
+        before do
+          Hydra::Works::AddGenericFileToGenericWork.call( subject, generic_file1 )
+          Hydra::Works::AddGenericFileToGenericWork.call( subject, generic_file2 )
+          Hydra::Works::AddGenericWorkToGenericWork.call( subject, generic_work1 )
+          Hydra::Works::AddGenericWorkToGenericWork.call( subject, generic_work2 )
+          Hydra::Works::AddRelatedObjectToGenericWork.call( subject, object1 )
+          subject.save
+        end
+
+        it 'should add a related object to generic_work with generic_works and generic_files' do
+          Hydra::Works::AddRelatedObjectToGenericWork.call( subject, object2 )
+          related_objects = Hydra::Works::GetRelatedObjectsFromGenericWork.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_works_ssim"]).to include(generic_work1.id,generic_work2.id)
+          expect(subject.to_solr["generic_works_ssim"]).not_to include(generic_file2.id,generic_file1.id,object1.id,object2.id)
+          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,generic_work1.id,generic_work2.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,generic_work1.id,generic_work2.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::AddRelatedObjectToGenericWork.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::AddRelatedObjectToGenericWork.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::AddRelatedObjectToGenericWork.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::AddRelatedObjectToGenericWork.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::AddRelatedObjectToGenericWork.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_file1)    { Hydra::Works::GenericFile::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_work must be a hydra-works generic work' }
+
+      it 'should NOT accept Hydra::Works::Collection as parent generic work' do
+        expect{ Hydra::Works::AddRelatedObjectToGenericWork.call( collection1, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::Works::GenericFile as parent generic work' do
+        expect{ Hydra::Works::AddRelatedObjectToGenericWork.call( generic_file1, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::PCDM::Collections as parent generic work' do
+        expect{ Hydra::Works::AddRelatedObjectToGenericWork.call( pcdm_collection1, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::PCDM::Objects as parent generic work' do
+        expect{ Hydra::Works::AddRelatedObjectToGenericWork.call( pcdm_object1, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept Hydra::PCDM::Files as parent generic work' do
+        expect{ Hydra::Works::AddRelatedObjectToGenericWork.call( pcdm_file1, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept non-PCDM objects as parent generic work' do
+        expect{ Hydra::Works::AddRelatedObjectToGenericWork.call( non_PCDM_object, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
+      end
+
+      it 'should NOT accept AF::Base objects as parent generic work' do
+        expect{ Hydra::Works::AddRelatedObjectToGenericWork.call( af_base_object, pcdm_object2 ) }.to raise_error(ArgumentError,error_message)
+      end
+    end
+
+
+    context 'with invalid bahaviors' 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::AddRelatedObjectToGenericWork.call( subject, object1 )
+        Hydra::Works::AddRelatedObjectToGenericWork.call( subject, object2 )
+        Hydra::Works::AddRelatedObjectToGenericWork.call( subject, object1 )
+        related_objects = Hydra::Works::GetRelatedObjectsFromGenericWork.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