curation_concerns 1.0.0.beta1 → 1.0.0.beta2

data/spec/models/curation_concerns/collection_behavior_spec.rb

@@ -2,6 +2,8 @@ require 'spec_helper'
 require 'rspec/active_model/mocks'
 
 describe CurationConcerns::CollectionBehavior do
+  include CurationConcerns::FactoryHelpers
+
   # All behavior for Collection are defined in CC::CollectionBehavior, so we use
   # a Collection instance to test.
   let(:collection) { FactoryGirl.build(:collection) }
@@ -31,13 +33,20 @@ describe CurationConcerns::CollectionBehavior do
     # Calculating the size of the Collection should only hit Solr.
     # This base case querries solr in an integration test
     context 'with indexed Works and FileSets', :integration do
-      let(:file1) { FactoryGirl.build(:file_set, file_size: ['100']) }
-      let(:file2) { FactoryGirl.build(:file_set, file_size: ['100']) }
-      let(:file3) { FactoryGirl.build(:file_set, file_size: ['9000'], id: 'fumid') }
+      let(:file1) { FactoryGirl.build(:file_set) }
+      let(:file2) { FactoryGirl.build(:file_set) }
+      let(:file3) { FactoryGirl.build(:file_set, id: 'fumid') }
       let(:work1) { FactoryGirl.build(:generic_work) }
       let(:work2) { FactoryGirl.build(:generic_work) }
       let(:work3) { FactoryGirl.build(:generic_work, id: 'dumid') }
+      let(:of1)   { mock_file_factory(file_size: ['100']) }
+      let(:of2)   { mock_file_factory(file_size: ['100']) }
+      let(:of3)   { mock_file_factory(file_size: ['9000']) }
+
       before do
+        allow(file1).to receive(:original_file).and_return(of1)
+        allow(file2).to receive(:original_file).and_return(of2)
+        allow(file3).to receive(:original_file).and_return(of3)
         # Save collection to get ids
         collection.save
         # Create relationships so member_ids are created

data/spec/models/file_set_spec.rb

@@ -4,6 +4,8 @@ require 'spec_helper'
 # It includes the CurationConcerns::FileSetBehavior module and nothing else
 # So this test covers both the FileSetBehavior module and the generated FileSet model
 describe FileSet do
+  include CurationConcerns::FactoryHelpers
+
   let(:user) { create(:user) }
 
   describe 'rdf type' do
@@ -105,6 +107,7 @@ describe FileSet do
       expect(subject).to respond_to(:well_formed)
       expect(subject).to respond_to(:page_count)
       expect(subject).to respond_to(:file_title)
+      # :creator is characterization metadata?
       expect(subject).to respond_to(:creator)
     end
 
@@ -168,7 +171,7 @@ describe FileSet do
   end
 
   it 'supports multi-valued fields in solr' do
-    subject.tag = %w(tag1 tag2)
+    subject.keyword = %w(keyword1 keyword2)
     expect { subject.save }.not_to raise_error
     subject.delete
   end
@@ -357,7 +360,7 @@ describe FileSet do
     context 'when file contains a virus' do
       before do
         allow(subject).to receive(:warn) # suppress virus warnings
-        allow(ClamAV.instance).to receive(:scanfile).and_return('EL CRAPO VIRUS')
+        allow(Hydra::Works::VirusCheckerService).to receive(:file_has_virus?) { true }
         # TODO: Test that this works with Hydra::Works::UploadFileToFileSet. see https://github.com/projecthydra-labs/hydra-works/pull/139
         # Hydra::Works::UploadFileToFileSet.call(subject, file_path, original_name: 'small_file.txt')
         of = subject.build_original_file
@@ -366,7 +369,7 @@ describe FileSet do
 
       it 'populates the errors hash during validation' do
         expect(subject).to_not be_valid
-        expect(subject.errors.messages[:base].first).to match(/A virus was found in .*: EL CRAPO VIRUS/)
+        expect(subject.errors.messages[:base].first).to eq "Failed to verify uploaded file is not a virus"
       end
 
       it 'does not save the file or create a new version' do
@@ -555,77 +558,80 @@ describe FileSet do
   end
 
   describe 'mime type recognition' do
+    let(:mock_file) { mock_file_factory(mime_type: mime_type) }
+    before { allow(subject).to receive(:original_file).and_return(mock_file) }
+
     context '#image?' do
       context 'when image/jp2' do
-        before { subject.mime_type = 'image/jp2' }
+        let(:mime_type) { 'image/jp2' }
         it { should be_image }
       end
       context 'when image/jpg' do
-        before { subject.mime_type = 'image/jpg' }
+        let(:mime_type) { 'image/jpg' }
         it { should be_image }
       end
       context 'when image/png' do
-        before { subject.mime_type = 'image/png' }
+        let(:mime_type) { 'image/png' }
         it { should be_image }
       end
       context 'when image/tiff' do
-        before { subject.mime_type = 'image/tiff' }
+        let(:mime_type) { 'image/tiff' }
         it { should be_image }
       end
     end
 
     describe '#pdf?' do
-      before { subject.mime_type = 'application/pdf' }
+      let(:mime_type) { 'application/pdf' }
       it { should be_pdf }
     end
 
     describe '#audio?' do
       context 'when x-wave' do
-        before { subject.mime_type = 'audio/x-wave' }
+        let(:mime_type) { 'audio/x-wave' }
         it { should be_audio }
       end
       context 'when x-wav' do
-        before { subject.mime_type = 'audio/x-wav' }
+        let(:mime_type) { 'audio/x-wav' }
         it { should be_audio }
       end
       context 'when mpeg' do
-        before { subject.mime_type = 'audio/mpeg' }
+        let(:mime_type) { 'audio/mpeg' }
         it { should be_audio }
       end
       context 'when mp3' do
-        before { subject.mime_type = 'audio/mp3' }
+        let(:mime_type) { 'audio/mp3' }
         it { should be_audio }
       end
       context 'when ogg' do
-        before { subject.mime_type = 'audio/ogg' }
+        let(:mime_type) { 'audio/ogg' }
         it { should be_audio }
       end
     end
 
     describe '#video?' do
       context 'should be true for avi' do
-        before { subject.mime_type = 'video/avi' }
+        let(:mime_type) { 'video/avi' }
         it { should be_video }
       end
 
       context 'should be true for webm' do
-        before { subject.mime_type = 'video/webm' }
+        let(:mime_type) { 'video/webm' }
         it { should be_video }
       end
       context 'should be true for mp4' do
-        before { subject.mime_type = 'video/mp4' }
+        let(:mime_type) { 'video/mp4' }
         it { should be_video }
       end
       context 'should be true for mpeg' do
-        before { subject.mime_type = 'video/mpeg' }
+        let(:mime_type) { 'video/mpeg' }
         it { should be_video }
       end
       context 'should be true for quicktime' do
-        before { subject.mime_type = 'video/quicktime' }
+        let(:mime_type) { 'video/quicktime' }
         it { should be_video }
       end
       context 'should be true for mxf' do
-        before { subject.mime_type = 'application/mxf' }
+        let(:mime_type) { 'application/mxf' }
         it { should be_video }
       end
     end

data/spec/presenters/curation_concerns/collection_presenter_spec.rb

@@ -18,7 +18,7 @@ describe CurationConcerns::CollectionPresenter do
 
   describe '#title' do
     subject { presenter.title }
-    it { is_expected.to eq 'A clever title' }
+    it { is_expected.to eq ['A clever title'] }
   end
 
   describe '#to_key' do

data/spec/presenters/curation_concerns/file_set_presenter_spec.rb

@@ -45,7 +45,7 @@ describe CurationConcerns::FileSetPresenter do
 
   describe "properties delegated to solr_document" do
     let(:solr_properties) do
-      ["date_uploaded", "depositor", "tags", "title_or_label",
+      ["date_uploaded", "depositor", "keyword", "title_or_label",
        "contributor", "creator", "title", "description", "publisher",
        "subject", "language", "rights"]
     end

data/spec/presenters/curation_concerns/work_show_presenter_spec.rb

@@ -131,8 +131,8 @@ describe CurationConcerns::WorkShowPresenter do
 
     context 'with an existing field' do
       before do
-        allow(CurationConcerns::AttributeRenderer).to receive(:new)
-          .with(:title, "foo bar", {})
+        allow(CurationConcerns::Renderers::AttributeRenderer).to receive(:new)
+          .with(:title, ["foo bar"], {})
           .and_return(renderer)
       end
 

data/spec/renderers/curation_concerns/{attribute_renderer_spec.rb → renderers/attribute_renderer_spec.rb}

@@ -1,9 +1,9 @@
 require 'spec_helper'
 
-describe CurationConcerns::AttributeRenderer do
+describe CurationConcerns::Renderers::AttributeRenderer do
   let(:field) { :name }
   let(:renderer) { described_class.new(field, ['Bob', 'Jessica']) }
-  let(:yml_path) { File.join(File.dirname(__FILE__), '..', '..', 'fixtures', 'locales', '*.{rb,yml}') }
+  let(:yml_path) { File.join(File.dirname(__FILE__), '..', '..', '..', 'fixtures', 'locales', '*.{rb,yml}') }
   before do
     I18n.load_path += Dir[File.join(yml_path)]
     I18n.reload!

data/spec/services/curation_concern_spec.rb

@@ -6,6 +6,6 @@ describe CurationConcerns::CurationConcern do
 
   describe ".actor" do
     subject { described_class.actor(work, user) }
-    it { is_expected.to be_kind_of CurationConcerns::ActorStack }
+    it { is_expected.to be_kind_of CurationConcerns::Actors::ActorStack }
   end
 end

data/spec/services/thumbnail_path_service_spec.rb

@@ -1,35 +1,39 @@
 require 'spec_helper'
 
 describe CurationConcerns::ThumbnailPathService do
+  include CurationConcerns::FactoryHelpers
+
   subject { described_class.call(object) }
 
   context "with a FileSet" do
-    let(:object) { FileSet.new(id: '999', mime_type: mime_type) }
-    let(:mime_type) { 'image/jpeg' }
+    let(:object) { build(:file_set, id: '999') }
+    before { allow(object).to receive(:original_file).and_return(original_file) }
     context "that has a thumbnail" do
-      before do
-        allow(File).to receive(:exist?).and_return(true)
-      end
+      let(:original_file) { mock_file_factory(mime_type: 'image/jpeg') }
+      before { allow(File).to receive(:exist?).and_return(true) }
       it { is_expected.to eq '/downloads/999?file=thumbnail' }
     end
 
     context "that is an audio" do
-      let(:mime_type) { 'audio/x-wav' }
+      let(:original_file) { mock_file_factory(mime_type: 'audio/x-wav') }
       it { is_expected.to match %r{/assets/audio-.+.png} }
     end
 
     context "that has no thumbnail" do
+      let(:original_file) { mock_model('MockFile', mime_type: nil) }
       it { is_expected.to match %r{/assets/default-.+.png} }
     end
   end
 
   context "with a Work" do
     context "that has a thumbnail" do
-      let(:object) { GenericWork.new(thumbnail_id: '999') }
-      let(:representative) { FileSet.new(id: '777') }
+      let(:object)         { GenericWork.new(thumbnail_id: '999') }
+      let(:representative) { build(:file_set, id: '777') }
+      let(:original_file)  { mock_file_factory(mime_type: 'image/jpeg') }
       before do
         allow(File).to receive(:exist?).and_return(true)
-        allow(FileSet).to receive(:load_instance_from_solr).with('999').and_return(representative)
+        allow(FileSet).to receive(:find).with('999').and_return(representative)
+        allow(representative).to receive(:original_file).and_return(original_file)
       end
 
       it { is_expected.to eq '/downloads/999?file=thumbnail' }

data/spec/support/curation_concerns/factory_helpers.rb

@@ -11,5 +11,23 @@ module CurationConcerns
         let(:public_work_factory_name) { "public_#{curation_concern_type_underscore}".to_sym }
       end
     end
+
+    def mock_file_factory(opts = {})
+      mock_model('MockFile',
+                 mime_type:         opts.fetch(:mime_type, 'text/plain'),
+                 content:           opts.fetch(:content, 'content'),
+                 file_size:         opts.fetch(:file_size, []),
+                 format_label:      opts.fetch(:format_label, []),
+                 height:            opts.fetch(:height, []),
+                 width:             opts.fetch(:width, []),
+                 filename:          opts.fetch(:filename, []),
+                 well_formed:       opts.fetch(:well_formed, []),
+                 page_count:        opts.fetch(:page_count, []),
+                 file_title:        opts.fetch(:file_title, []),
+                 last_modified:     opts.fetch(:last_modified, []),
+                 original_checksum: opts.fetch(:original_checksum, []),
+                 digest:            opts.fetch(:digest, [])
+                )
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: curation_concerns
 version: !ruby/object:Gem::Version
-  version: 1.0.0.beta1
+  version: 1.0.0.beta2
 platform: ruby
 authors:
 - Matt Zumwalt
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-05-11 00:00:00.000000000 Z
+date: 2016-05-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hydra-head
@@ -176,42 +176,16 @@ dependencies:
   name: hydra-works
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.8'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.1
+        version: 0.10.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.8'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.8.1
-- !ruby/object:Gem::Dependency
-  name: hydra-pcdm
-  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.0.beta1
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '1'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.8.0.beta1
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '1'
+        version: 0.10.0
 - !ruby/object:Gem::Dependency
   name: active_fedora-noid
   requirement: !ruby/object:Gem::Requirement
@@ -612,21 +586,22 @@ files:
 - RELEASING.md
 - Rakefile
 - app/actors/concerns/curation_concerns/manages_embargoes_actor.rb
-- app/actors/curation_concerns/abstract_actor.rb
-- app/actors/curation_concerns/actor_stack.rb
-- app/actors/curation_concerns/add_to_collection_actor.rb
-- app/actors/curation_concerns/apply_order_actor.rb
-- app/actors/curation_concerns/assign_identifier_actor.rb
-- app/actors/curation_concerns/assign_representative_actor.rb
-- app/actors/curation_concerns/attach_files_actor.rb
-- app/actors/curation_concerns/base_actor.rb
-- app/actors/curation_concerns/embargo_actor.rb
-- app/actors/curation_concerns/file_actor.rb
-- app/actors/curation_concerns/file_set_actor.rb
-- app/actors/curation_concerns/interpret_visibility_actor.rb
-- app/actors/curation_concerns/lease_actor.rb
-- app/actors/curation_concerns/root_actor.rb
-- app/actors/curation_concerns/work_actor_behavior.rb
+- app/actors/curation_concerns/actors.rb
+- app/actors/curation_concerns/actors/abstract_actor.rb
+- app/actors/curation_concerns/actors/actor_stack.rb
+- app/actors/curation_concerns/actors/add_to_collection_actor.rb
+- app/actors/curation_concerns/actors/apply_order_actor.rb
+- app/actors/curation_concerns/actors/assign_identifier_actor.rb
+- app/actors/curation_concerns/actors/assign_representative_actor.rb
+- app/actors/curation_concerns/actors/attach_files_actor.rb
+- app/actors/curation_concerns/actors/base_actor.rb
+- app/actors/curation_concerns/actors/embargo_actor.rb
+- app/actors/curation_concerns/actors/file_actor.rb
+- app/actors/curation_concerns/actors/file_set_actor.rb
+- app/actors/curation_concerns/actors/interpret_visibility_actor.rb
+- app/actors/curation_concerns/actors/lease_actor.rb
+- app/actors/curation_concerns/actors/root_actor.rb
+- app/actors/curation_concerns/actors/work_actor_behavior.rb
 - app/assets/images/audio.png
 - app/assets/images/default.png
 - app/assets/images/loading.gif
@@ -696,6 +671,7 @@ files:
 - app/controllers/embargoes_controller.rb
 - app/controllers/leases_controller.rb
 - app/controllers/welcome_controller.rb
+- app/forms/curation_concerns/forms.rb
 - app/forms/curation_concerns/forms/collection_edit_form.rb
 - app/forms/curation_concerns/forms/file_set_edit_form.rb
 - app/forms/curation_concerns/forms/work_form.rb
@@ -734,6 +710,7 @@ files:
 - app/models/collection.rb
 - app/models/concerns/curation_concerns/ability.rb
 - app/models/concerns/curation_concerns/basic_metadata.rb
+- app/models/concerns/curation_concerns/characterization.rb
 - app/models/concerns/curation_concerns/collection.rb
 - app/models/concerns/curation_concerns/collection_behavior.rb
 - app/models/concerns/curation_concerns/file_set/belongs_to_works.rb
@@ -770,8 +747,9 @@ files:
 - app/presenters/curation_concerns/version_list_presenter.rb
 - app/presenters/curation_concerns/version_presenter.rb
 - app/presenters/curation_concerns/work_show_presenter.rb
-- app/renderers/curation_concerns/attribute_renderer.rb
-- app/renderers/curation_concerns/configured_microdata.rb
+- app/renderers/curation_concerns/renderers/attribute_renderer.rb
+- app/renderers/curation_concerns/renderers/configured_microdata.rb
+- app/renderers/renderers.rb
 - app/search_builders/curation_concerns/collection_member_search_builder.rb
 - app/search_builders/curation_concerns/collection_search_builder.rb
 - app/search_builders/curation_concerns/deactivated_embargo_search_builder.rb
@@ -788,7 +766,7 @@ files:
 - app/search_builders/curation_concerns/single_result.rb
 - app/search_builders/curation_concerns/single_use_link_search_builder.rb
 - app/search_builders/curation_concerns/work_search_builder.rb
-- app/services/curation_concerns/actor_factory.rb
+- app/services/curation_concerns/actors/actor_factory.rb
 - app/services/curation_concerns/curation_concern.rb
 - app/services/curation_concerns/derivative_path.rb
 - app/services/curation_concerns/embargo_service.rb
@@ -1142,7 +1120,7 @@ files:
 - spec/presenters/curation_concerns/work_show_presenter_spec.rb
 - spec/presenters/embargo_presenter_spec.rb
 - spec/presenters/lease_presenter_spec.rb
-- spec/renderers/curation_concerns/attribute_renderer_spec.rb
+- spec/renderers/curation_concerns/renderers/attribute_renderer_spec.rb
 - spec/routing/curation_concerns/routes_spec.rb
 - spec/routing/route_spec.rb
 - spec/search_builders/curation_concerns/embargo_search_builder_spec.rb
@@ -1343,7 +1321,7 @@ test_files:
 - spec/presenters/curation_concerns/work_show_presenter_spec.rb
 - spec/presenters/embargo_presenter_spec.rb
 - spec/presenters/lease_presenter_spec.rb
-- spec/renderers/curation_concerns/attribute_renderer_spec.rb
+- spec/renderers/curation_concerns/renderers/attribute_renderer_spec.rb
 - spec/routing/curation_concerns/routes_spec.rb
 - spec/routing/route_spec.rb
 - spec/search_builders/curation_concerns/embargo_search_builder_spec.rb

data/app/actors/curation_concerns/abstract_actor.rb

@@ -1,28 +0,0 @@
-module CurationConcerns
-  # The CurationConcern Abstract actor responds to two primary actions:
-  # * #create
-  # * #update
-  #
-  # and the following attributes
-  #
-  # * next_actor
-  # * curation_concern
-  # * user
-  #
-  # it must instantiate the next actor in the chain and instantiate it.
-  # it should respond to curation_concern, user and attributes.
-  # it ha to next_actor
-  class AbstractActor
-    attr_reader :next_actor
-
-    def initialize(_curation_concern, _user, next_actor)
-      @next_actor = next_actor
-    end
-
-    delegate :curation_concern, :user, to: :next_actor
-
-    delegate :create, to: :next_actor
-
-    delegate :update, to: :next_actor
-  end
-end

data/app/actors/curation_concerns/actor_stack.rb

@@ -1,27 +0,0 @@
-module CurationConcerns
-  class ActorStack
-    attr_reader :curation_concern, :user, :first_actor_class, :more_actors
-    def initialize(curation_concern, user, more_actors)
-      @curation_concern = curation_concern
-      @user = user
-      @more_actors = more_actors
-      @first_actor_class = @more_actors.shift || RootActor
-    end
-
-    def inner_stack
-      ActorStack.new(curation_concern, user, more_actors)
-    end
-
-    def actor
-      first_actor_class.new(curation_concern, user, inner_stack)
-    end
-
-    def create(attributes)
-      actor.create(attributes.with_indifferent_access)
-    end
-
-    def update(attributes)
-      actor.update(attributes.with_indifferent_access)
-    end
-  end
-end

data/app/actors/curation_concerns/add_to_collection_actor.rb

@@ -1,38 +0,0 @@
-module CurationConcerns
-  class AddToCollectionActor < AbstractActor
-    def create(attributes)
-      collection_ids = attributes.delete(:collection_ids)
-      next_actor.create(attributes) && add_to_collections(collection_ids)
-    end
-
-    def update(attributes)
-      collection_ids = attributes.delete(:collection_ids)
-      add_to_collections(collection_ids) && next_actor.update(attributes)
-    end
-
-    private
-
-      # The default behavior of active_fedora's aggregates association,
-      # when assigning the id accessor (e.g. collection_ids = ['foo:1']) is to add
-      # to new collections, but not remove from old collections.
-      # This method ensures it's removed from the old collections.
-      def add_to_collections(new_collection_ids)
-        return true unless new_collection_ids
-        # remove from old collections
-        # TODO: Implement in_collection_ids https://github.com/projecthydra-labs/hydra-pcdm/issues/157
-        (curation_concern.in_collections.map(&:id) - new_collection_ids).each do |old_id|
-          collection = ::Collection.find(old_id)
-          collection.members.delete(curation_concern)
-          collection.save
-        end
-
-        # add to new
-        new_collection_ids.each do |coll_id|
-          collection = ::Collection.find(coll_id)
-          collection.members << curation_concern
-          collection.save
-        end
-        true
-      end
-  end
-end

data/app/actors/curation_concerns/apply_order_actor.rb

@@ -1,24 +0,0 @@
-module CurationConcerns
-  class ApplyOrderActor < AbstractActor
-    def update(attributes)
-      ordered_member_ids = attributes.delete(:ordered_member_ids)
-      apply_order(ordered_member_ids) && next_actor.update(attributes)
-    end
-
-    private
-
-      def apply_order(new_order)
-        return true unless new_order
-        curation_concern.ordered_member_proxies.each_with_index do |proxy, index|
-          unless new_order[index]
-            proxy.prev.next = curation_concern.ordered_member_proxies.last.next
-            break
-          end
-          proxy.proxy_for = ActiveFedora::Base.id_to_uri(new_order[index])
-          proxy.target = nil
-        end
-        curation_concern.list_source.order_will_change!
-        true
-      end
-  end
-end

data/app/actors/curation_concerns/assign_identifier_actor.rb

@@ -1,7 +0,0 @@
-module CurationConcerns
-  class AssignIdentifierActor < AbstractActor
-    def create(attributes)
-      curation_concern.assign_id && next_actor.create(attributes)
-    end
-  end
-end

data/app/actors/curation_concerns/assign_representative_actor.rb

@@ -1,18 +0,0 @@
-module CurationConcerns
-  class AssignRepresentativeActor < AbstractActor
-    def create(attributes)
-      next_actor.create(attributes) && assign_representative
-    end
-
-    private
-
-      def assign_representative
-        unless curation_concern.representative_id
-          # TODO: Possible optimization here. Does this cause a fetch of ordered_members if they're already loaded?
-          representative = nil # curation_concern.ordered_members.association.reader.first.target
-          curation_concern.representative = representative if representative
-        end
-        curation_concern.save
-      end
-  end
-end

data/app/actors/curation_concerns/attach_files_actor.rb

@@ -1,39 +0,0 @@
-module CurationConcerns
-  class AttachFilesActor < AbstractActor
-    def create(attributes)
-      files = [attributes.delete(:files)].flatten.compact
-      attach_files(files, visibility_attributes(attributes)) &&
-        next_actor.create(attributes)
-    end
-
-    def update(attributes)
-      files = [attributes.delete(:files)].flatten.compact
-      next_actor.update(attributes) &&
-        attach_files(files, visibility_attributes(attributes))
-    end
-
-    private
-
-      def attach_files(files, visibility_attr)
-        files.all? do |file|
-          attach_file(file, visibility_attr)
-        end
-      end
-
-      def attach_file(file, visibility_attr)
-        file_set = ::FileSet.new
-        file_set_actor = CurationConcerns::FileSetActor.new(file_set, user)
-        file_set_actor.create_metadata(curation_concern, visibility_attr)
-        file_set_actor.create_content(file)
-      end
-
-      # The attributes used for visibility - used to send as initial params to
-      # created FileSets.
-      def visibility_attributes(attributes)
-        attributes.slice(:visibility, :visibility_during_lease,
-                         :visibility_after_lease, :lease_expiration_date,
-                         :embargo_release_date, :visibility_during_embargo,
-                         :visibility_after_embargo)
-      end
-  end
-end