curation_concerns 0.1.0 → 0.2.0

data/spec/routing/curation_concerns/routes_spec.rb

@@ -1,35 +1,61 @@
-require "spec_helper"
+require 'spec_helper'
 
 module CurationConcerns
-  describe "routing" do
+  describe 'routing' do
     before(:all) do
-      @generic_work =  FactoryGirl.create(:generic_work) # Preload once to save time
+      @generic_work = FactoryGirl.create(:generic_work) # Preload once to save time
     end
 
-    describe "Classify concerns" do
+    describe 'Classify concerns' do
       routes { CurationConcerns::Engine.routes }
-      it "routes to #new" do
+      it 'routes to #new' do
         expect(new_classify_concern_path).to eq '/classify_concerns/new'
-        expect(get("/classify_concerns/new")).to route_to("curation_concerns/classify_concerns#new")
+        expect(get('/classify_concerns/new')).to route_to('curation_concerns/classify_concerns#new')
+      end
+    end
+
+    describe 'Single Use Link Viewer' do
+      routes { CurationConcerns::Engine.routes }
+      it 'routes to #show' do
+        expect(show_single_use_link_path('abc123')).to eq '/single_use_link/show/abc123'
+        expect(get("/single_use_link/show/abc123")).to route_to("curation_concerns/single_use_links_viewer#show", id: 'abc123')
+      end
+
+      it 'routes to #download' do
+        expect(download_single_use_link_path('abc123')).to eq '/single_use_link/download/abc123'
+        expect(get("/single_use_link/download/abc123")).to route_to("curation_concerns/single_use_links_viewer#download", id: 'abc123')
+      end
+    end
+
+    describe 'Single Use Link Generator' do
+      routes { CurationConcerns::Engine.routes }
+      it 'routes to #show' do
+        expect(generate_show_single_use_link_path('abc123')).to eq '/single_use_link/generate_show/abc123'
+        expect(get("/single_use_link/generate_show/abc123")).to route_to("curation_concerns/single_use_links#new_show", id: 'abc123')
+      end
+
+      it 'routes to #download' do
+        expect(generate_download_single_use_link_path('abc123')).to eq '/single_use_link/generate_download/abc123'
+        expect(get("/single_use_link/generate_download/abc123")).to route_to("curation_concerns/single_use_links#new_download", id: 'abc123')
       end
     end
 
-    describe "generic work" do
+    describe 'generic work' do
       routes { Rails.application.routes }
       let(:generic_work) { @generic_work }
       it 'routes to #new' do
         expect(new_curation_concerns_generic_work_path).to eq '/concern/generic_works/new'
-        expect(get("/concern/generic_works/new")).to route_to("curation_concerns/generic_works#new")
+        expect(get('/concern/generic_works/new')).to route_to('curation_concerns/generic_works#new')
       end
       it 'routes to #show' do
         expect(curation_concerns_generic_work_path(generic_work)).to eq "/concern/generic_works/#{generic_work.id}"
-        expect(get("/concern/generic_works/#{generic_work.id}")).to route_to(:controller => "curation_concerns/generic_works", :action => "show", id:generic_work.id)
-        expect(url_for([:curation_concerns, generic_work, only_path:true])).to eq "/concern/generic_works/#{generic_work.id}"
+        expect(get("/concern/generic_works/#{generic_work.id}")).to route_to(controller: 'curation_concerns/generic_works', action: 'show', id: generic_work.id)
+        expect(url_for([:curation_concerns, generic_work, only_path: true])).to eq "/concern/generic_works/#{generic_work.id}"
       end
       it 'routes to #edit' do
         expect(edit_curation_concerns_generic_work_path(generic_work)).to eq "/concern/generic_works/#{generic_work.id}/edit"
-        expect(get("/concern/generic_works/#{generic_work.id}/edit")).to route_to(controller:"curation_concerns/generic_works", action: 'edit', id: generic_work.id)
-        expect(url_for([:edit, :curation_concerns, generic_work, only_path:true])).to eq "/concern/generic_works/#{generic_work.id}/edit"
+        expect(get("/concern/generic_works/#{generic_work.id}/edit")).to route_to(controller: 'curation_concerns/generic_works', action: 'edit', id: generic_work.id)
+        expect(url_for([:edit, :curation_concerns, generic_work, only_path: true])).to eq "/concern/generic_works/#{generic_work.id}/edit"
       end
     end
   end

data/spec/routing/route_spec.rb

@@ -1,86 +1,85 @@
 require 'spec_helper'
 
-describe 'Routes', :type => :routing do
-
+describe 'Routes', type: :routing do
   describe 'Homepage' do
-    it 'should route the root url to the catalog controller' do
-      expect({ get: '/' }).to route_to(controller: 'catalog', action: 'index')
+    it 'routes the root url to the welcome controller' do
+      expect(get: '/').to route_to(controller: 'welcome', action: 'index')
     end
   end
 
   describe 'GenericWork' do
-    it 'should route to create' do
-      expect({ post: 'concern/generic_works' }).to route_to(controller: 'curation_concerns/generic_works', action: 'create')
+    it 'routes to create' do
+      expect(post: 'concern/generic_works').to route_to(controller: 'curation_concerns/generic_works', action: 'create')
     end
 
-    it 'should route to new' do
-      expect({ get: 'concern/generic_works/new' }).to route_to(controller: 'curation_concerns/generic_works', action: 'new')
+    it 'routes to new' do
+      expect(get: 'concern/generic_works/new').to route_to(controller: 'curation_concerns/generic_works', action: 'new')
     end
 
-    it 'should route to edit' do
-      expect({ get: 'concern/generic_works/3/edit' }).to route_to(controller: 'curation_concerns/generic_works', action: 'edit', id: '3')
+    it 'routes to edit' do
+      expect(get: 'concern/generic_works/3/edit').to route_to(controller: 'curation_concerns/generic_works', action: 'edit', id: '3')
     end
 
-    it "should route to show" do
-      expect({ get: 'concern/generic_works/4' }).to route_to(controller: 'curation_concerns/generic_works', action: 'show', id: '4')
+    it 'routes to show' do
+      expect(get: 'concern/generic_works/4').to route_to(controller: 'curation_concerns/generic_works', action: 'show', id: '4')
     end
 
-    it "should route to update" do
-      expect({ put: 'concern/generic_works/5' }).to route_to(controller: 'curation_concerns/generic_works', action: 'update', id: '5')
+    it 'routes to update' do
+      expect(put: 'concern/generic_works/5').to route_to(controller: 'curation_concerns/generic_works', action: 'update', id: '5')
     end
 
-    it "should route to destroy" do
-      expect({ delete: 'concern/generic_works/6' }).to route_to(controller: 'curation_concerns/generic_works', action: 'destroy', id: '6')
+    it 'routes to destroy' do
+      expect(delete: 'concern/generic_works/6').to route_to(controller: 'curation_concerns/generic_works', action: 'destroy', id: '6')
     end
 
-    it "should *not* route to index" do
-      expect({ get: 'concern/generic_works' }).not_to route_to(controller: 'curation_concerns/generic_works', action: 'index')
-    end    
+    it '*not*s route to index' do
+      expect(get: 'concern/generic_works').not_to route_to(controller: 'curation_concerns/generic_works', action: 'index')
+    end
   end
 
   describe 'Permissions' do
-    it "should route to confirm" do
-      expect({ get: '/concern/permissions/1/confirm' }).to route_to(controller: 'curation_concerns/permissions', action: 'confirm', id: '1')
+    it 'routes to confirm' do
+      expect(get: '/concern/permissions/1/confirm').to route_to(controller: 'curation_concerns/permissions', action: 'confirm', id: '1')
     end
 
-    it "should route to copy" do
-      expect({ post: '/concern/permissions/2/copy' }).to route_to(controller: 'curation_concerns/permissions', action: 'copy', id: '2')
+    it 'routes to copy' do
+      expect(post: '/concern/permissions/2/copy').to route_to(controller: 'curation_concerns/permissions', action: 'copy', id: '2')
     end
   end
 
   describe 'GenericFile' do
-    it 'should route to create' do
-      expect({ post: 'concern/container/1/generic_files' }).to route_to(controller: 'curation_concerns/generic_files', action: 'create', parent_id: '1')
+    it 'routes to create' do
+      expect(post: 'concern/container/1/file_sets').to route_to(controller: 'curation_concerns/file_sets', action: 'create', parent_id: '1')
     end
 
-    it 'should route to new' do
-      expect({ get: 'concern/container/2/generic_files/new' }).to route_to(controller: 'curation_concerns/generic_files', action: 'new', parent_id: '2')
+    it 'routes to new' do
+      expect(get: 'concern/container/2/file_sets/new').to route_to(controller: 'curation_concerns/file_sets', action: 'new', parent_id: '2')
     end
 
-    it 'should route to edit' do
-      expect({ get: 'concern/generic_files/3/edit' }).to route_to(controller: 'curation_concerns/generic_files', action: 'edit', id: '3')
+    it 'routes to edit' do
+      expect(get: 'concern/file_sets/3/edit').to route_to(controller: 'curation_concerns/file_sets', action: 'edit', id: '3')
     end
 
-    it "should route to show" do
-      expect({ get: 'concern/generic_files/4' }).to route_to(controller: 'curation_concerns/generic_files', action: 'show', id: '4')
+    it 'routes to show' do
+      expect(get: 'concern/file_sets/4').to route_to(controller: 'curation_concerns/file_sets', action: 'show', id: '4')
     end
 
-    it "should route to update" do
-      expect({ put: 'concern/generic_files/5' }).to route_to(controller: 'curation_concerns/generic_files', action: 'update', id: '5')
+    it 'routes to update' do
+      expect(put: 'concern/file_sets/5').to route_to(controller: 'curation_concerns/file_sets', action: 'update', id: '5')
     end
 
-    it "should route to destroy" do
-      expect({ delete: 'concern/generic_files/6' }).to route_to(controller: 'curation_concerns/generic_files', action: 'destroy', id: '6')
+    it 'routes to destroy' do
+      expect(delete: 'concern/file_sets/6').to route_to(controller: 'curation_concerns/file_sets', action: 'destroy', id: '6')
     end
 
-    it "should *not* route to index" do
-      expect({ get: 'concern/generic_files' }).not_to route_to(controller: 'curation_concerns/generic_files', action: 'index')
+    it '*not*s route to index' do
+      expect(get: 'concern/file_sets').not_to route_to(controller: 'curation_concerns/file_sets', action: 'index')
     end
   end
 
   describe 'Download' do
-    it "should route to show" do
-      expect({ get: '/downloads/9' }).to route_to(controller: 'downloads', action: 'show', id: '9')
+    it 'routes to show' do
+      expect(get: '/downloads/9').to route_to(controller: 'downloads', action: 'show', id: '9')
     end
   end
 end

data/spec/search_builders/curation_concerns/embargo_search_builder_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe CurationConcerns::EmbargoSearchBuilder do
+  let(:context) { double }
+  let(:search_builder) { described_class.new(context) }
+
+  describe "#processor_chain" do
+    subject { search_builder.processor_chain }
+    it { is_expected.to eq [:with_pagination, :with_sorting, :only_active_embargoes] }
+  end
+
+  describe "#with_sorting" do
+    subject { {} }
+    before { search_builder.with_sorting(subject) }
+    it { is_expected.to eq(sort: 'embargo_release_date_dtsi desc') }
+  end
+end

data/spec/search_builders/curation_concerns/lease_search_builder_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe CurationConcerns::LeaseSearchBuilder do
+  let(:context) { double }
+  let(:search_builder) { described_class.new(context) }
+
+  describe "#processor_chain" do
+    subject { search_builder.processor_chain }
+    it { is_expected.to eq [:with_pagination, :with_sorting, :only_active_leases] }
+  end
+
+  describe "#with_sorting" do
+    subject { {} }
+    before { search_builder.with_sorting(subject) }
+    it { is_expected.to eq(sort: 'lease_expiration_date_dtsi desc') }
+  end
+end

data/spec/search_builders/curation_concerns/search_builder_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+
+describe CurationConcerns::SearchBuilder do
+  let(:processor_chain) { [:filter_models] }
+  let(:ability) { double('ability') }
+  let(:context) { double('context') }
+  let(:user) { double('user') }
+  let(:solr_params) { { fq: [] } }
+
+  subject { described_class.new(processor_chain, context) }
+
+  describe '#only_file_sets' do
+    before { subject.only_file_sets(solr_params) }
+
+    it 'adds FileSet to query' do
+      expect(solr_params[:fq].first).to include('{!raw f=has_model_ssim}FileSet')
+    end
+  end
+
+  describe '#find_one' do
+    before do
+      allow(subject).to receive(:blacklight_params).and_return(id: '12345')
+      subject.find_one(solr_params)
+    end
+
+    it 'adds id to query' do
+      expect(solr_params[:fq].first).to include('{!raw f=id}12345')
+    end
+  end
+
+  describe '#gated_discovery_filters' do
+    before do
+      allow(subject).to receive(:current_ability).and_return(ability)
+      allow(ability).to receive(:current_user).and_return(user)
+      allow(user).to receive(:groups).and_return(['admin'])
+    end
+
+    it 'does not filter results for admin users' do
+      expect(subject.gated_discovery_filters).to eq([])
+    end
+  end
+
+  describe '#discovery_permissions' do
+    context 'when showing my works' do
+      before { allow(subject).to receive(:blacklight_params).and_return(works: 'mine') }
+
+      it 'limits query to edit permissions' do
+        expect(subject.discovery_permissions).to eq(['edit'])
+      end
+    end
+  end
+
+  describe '#filter_models' do
+    before { subject.filter_models(solr_params) }
+
+    it 'limits query to collection and generic work' do
+      expect(solr_params[:fq].first).to match(/{!raw f=has_model_ssim}GenericWork.*OR.*{!raw f=has_model_ssim}Collection/)
+    end
+  end
+end

data/spec/services/derivative_path_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+describe CurationConcerns::DerivativePath do
+  before do
+    allow(CurationConcerns.config).to receive(:derivatives_path).and_return('tmp')
+  end
+
+  describe '.derivative_path_for_reference' do
+    subject { described_class.derivative_path_for_reference(object, destination_name) }
+
+    let(:object) { double(id: '123') }
+    let(:destination_name) { 'thumbnail' }
+
+    it { is_expected.to eq 'tmp/12/3-thumbnail.jpeg' }
+  end
+end

data/spec/services/embargo_service_spec.rb

@@ -1,37 +1,38 @@
 require 'spec_helper'
 
 describe CurationConcerns::EmbargoService do
+  subject { described_class }
   let(:future_date) { 2.days.from_now }
   let(:past_date) { 2.days.ago }
 
   let!(:work_with_expired_embargo1) do
     FactoryGirl.build(:generic_work, embargo_release_date: past_date.to_s).tap do |work|
-      work.save(validate:false)
+      work.save(validate: false)
     end
   end
 
   let!(:work_with_expired_embargo2) do
     FactoryGirl.build(:generic_work, embargo_release_date: past_date.to_s).tap do |work|
-      work.save(validate:false)
+      work.save(validate: false)
     end
   end
 
-  let!(:work_with_embargo_in_effect) { FactoryGirl.create(:generic_work, embargo_release_date: future_date.to_s)}
-  let!(:work_without_embargo) { FactoryGirl.create(:generic_work)}
+  let!(:work_with_embargo_in_effect) { FactoryGirl.create(:generic_work, embargo_release_date: future_date.to_s) }
+  let!(:work_without_embargo) { FactoryGirl.create(:generic_work) }
 
-  describe "#assets_with_expired_embargoes" do
-    it "returns an array of assets with expired embargoes" do
-      returned_pids = subject.assets_with_expired_embargoes.map {|a| a.id}
-      expect(returned_pids).to include work_with_expired_embargo1.id,work_with_expired_embargo2.id
-      expect(returned_pids).to_not include work_with_embargo_in_effect.id,work_without_embargo.id
+  describe '#assets_with_expired_embargoes' do
+    it 'returns an array of assets with expired embargoes' do
+      returned_pids = subject.assets_with_expired_embargoes.map(&:id)
+      expect(returned_pids).to include work_with_expired_embargo1.id, work_with_expired_embargo2.id
+      expect(returned_pids).to_not include work_with_embargo_in_effect.id, work_without_embargo.id
     end
   end
 
-  describe "#assets_under_embargo" do
-    it "returns all assets with embargo release date set" do
-      result = subject.assets_under_embargo
-      returned_pids = subject.assets_under_embargo.map {|a| a.id}
-      expect(returned_pids).to include work_with_expired_embargo1.id,work_with_expired_embargo2.id,work_with_embargo_in_effect.id
+  describe '#assets_under_embargo' do
+    it 'returns all assets with embargo release date set' do
+      subject.assets_under_embargo
+      returned_pids = subject.assets_under_embargo.map(&:id)
+      expect(returned_pids).to include work_with_expired_embargo1.id, work_with_expired_embargo2.id, work_with_embargo_in_effect.id
       expect(returned_pids).to_not include work_without_embargo.id
     end
   end

data/spec/services/file_set_audit_service_spec.rb

@@ -0,0 +1,71 @@
+require 'spec_helper'
+
+describe CurationConcerns::FileSetAuditService do
+  let(:f)       { create(:file_set, content: File.open(fixture_file_path('world.png'))) }
+  let(:service) { described_class.new(f) }
+
+  describe '#audit' do
+    context 'when a file has two versions' do
+      before do
+        CurationConcerns::VersioningService.create(f.original_file) # create a second version -- the factory creates the first version when it attaches +content+
+      end
+      subject { service.audit[f.original_file.id] }
+      specify 'returns two log results' do
+        expect(subject.length).to eq(2)
+      end
+    end
+  end
+
+  describe '#audit_file' do
+    subject { service.send(:audit_file, f.original_file) }
+    specify 'returns a single result' do
+      expect(subject.length).to eq(1)
+    end
+  end
+
+  describe '#audit_file_version' do
+    subject { service.send(:audit_file_version, f.original_file.id, f.original_file.uri) }
+    specify 'returns a single ChecksumAuditLog for the given file' do
+      expect(subject).to be_kind_of ChecksumAuditLog
+      expect(subject.file_set_id).to eq(f.id)
+      expect(subject.version).to eq(f.original_file.uri)
+    end
+  end
+
+  describe '#audit_stat' do
+    subject { service.send(:audit_stat, f.original_file) }
+    context 'when no audits have been run' do
+      it 'reports that audits have not been run' do
+        expect(subject).to eq 'Audits have not yet been run on this file.'
+      end
+    end
+
+    context 'when no audit is pasing' do
+      before do
+        CurationConcerns::VersioningService.create(f.original_file)
+        ChecksumAuditLog.create!(pass: 1, file_set_id: f.id, version: f.original_file.versions.first.uri, file_id: 'original_file')
+      end
+
+      it 'reports that audits have not been run' do
+        expect(subject).to eq 'Some audits have not been run, but the ones run were passing.'
+      end
+    end
+  end
+
+  describe '#human_readable_audit_status' do
+    context 'when audit_stat is 0' do
+      subject { service.human_readable_audit_status 0 }
+      it { is_expected.to eq 'failing' }
+    end
+
+    context 'when audit_stat is 1' do
+      subject { service.human_readable_audit_status 1 }
+      it { is_expected.to eq 'passing' }
+    end
+
+    context 'when audit_stat is something else' do
+      subject { service.human_readable_audit_status 'something else' }
+      it { is_expected.to eq 'something else' }
+    end
+  end
+end

data/spec/services/lease_service_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe CurationConcerns::LeaseService do
-
+  subject { described_class }
   before { GenericWork.destroy_all }
 
   let(:future_date) { 2.days.from_now }
@@ -19,35 +19,35 @@ describe CurationConcerns::LeaseService do
     end
   end
 
-  let!(:work_with_lease_in_effect) { create(:generic_work, lease_expiration_date: future_date.to_s)}
-  let!(:work_without_lease) { create(:generic_work)}
+  let!(:work_with_lease_in_effect) { create(:generic_work, lease_expiration_date: future_date.to_s) }
+  let!(:work_without_lease) { create(:generic_work) }
 
-  describe "#assets_with_expired_leases" do
-    it "returns an array of assets with expired lease" do
+  describe '#assets_with_expired_leases' do
+    it 'returns an array of assets with expired lease' do
       returned_pids = subject.assets_with_expired_leases.map(&:id)
-      expect(returned_pids).to include work_with_expired_lease1.id,work_with_expired_lease2.id
-      expect(returned_pids).to_not include work_with_lease_in_effect.id,work_without_lease.id
+      expect(returned_pids).to include work_with_expired_lease1.id, work_with_expired_lease2.id
+      expect(returned_pids).to_not include work_with_lease_in_effect.id, work_without_lease.id
     end
   end
 
-  describe "#assets_under_lease" do
-    it "returns an array of assets with active leases" do
+  describe '#assets_under_lease' do
+    it 'returns an array of assets with active leases' do
       returned_pids = subject.assets_under_lease.map(&:id)
-      expect(returned_pids).to include work_with_expired_lease1.id,work_with_expired_lease2.id,work_with_lease_in_effect.id
+      expect(returned_pids).to include work_with_expired_lease1.id, work_with_expired_lease2.id, work_with_lease_in_effect.id
       expect(returned_pids).to_not include work_without_lease.id
     end
   end
 
-  describe "#assets_with_deactivated_leases" do
+  describe '#assets_with_deactivated_leases' do
     before do
       work_with_expired_lease1.deactivate_lease!
       work_with_expired_lease1.save!
       work_with_lease_in_effect.deactivate_lease!
       work_with_lease_in_effect.save!
     end
-    it "returns an array of assets with deactivated leases" do
+    it 'returns an array of assets with deactivated leases' do
       returned_pids = subject.assets_with_deactivated_leases.map(&:id)
-      expect(returned_pids).to eq [work_with_expired_lease1.id, work_with_lease_in_effect.id ]
+      expect(returned_pids).to eq [work_with_expired_lease1.id, work_with_lease_in_effect.id]
     end
   end
 end

data/spec/services/lock_manager_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+describe CurationConcerns::LockManager do
+  subject { described_class.new(CurationConcerns.config.lock_time_to_live,
+                                CurationConcerns.config.lock_retry_count,
+                                CurationConcerns.config.lock_retry_delay) }
+  describe "lock", unless: $in_travis do
+    it "calls the block" do
+      expect { |probe| subject.lock('foobar', &probe) }.to yield_with_no_args
+    end
+  end
+end

data/spec/services/parent_service_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+RSpec.describe CurationConcerns::ParentService do
+  subject { described_class }
+
+  describe ".parent_for" do
+    context "when there's no ID" do
+      it "is nil" do
+        expect(subject.parent_for(nil)).to eq nil
+      end
+    end
+  end
+end

data/spec/services/persist_derivatives_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe CurationConcerns::PersistDerivatives do
+  before do
+    allow(CurationConcerns.config).to receive(:derivatives_path).and_return('tmp')
+  end
+
+  describe '.output_file' do
+    subject { described_class.output_file(directives, &block) }
+
+    let(:directives) { { url: "file:/tmp/12/34/56/7-thumbnail.jpeg" } }
+    let(:destination_name) { 'thumbnail' }
+
+    let(:block) { lambda { true } }
+
+    it 'yields to the file' do
+      expect(FileUtils).to receive(:mkdir_p).with('/tmp/12/34/56')
+      expect(File).to receive(:open).with('/tmp/12/34/56/7-thumbnail.jpeg', 'wb') do |*_, &blk|
+        expect(blk).to be(block)
+      end
+      subject
+    end
+  end
+end

data/spec/services/persist_directly_contained_output_file_service_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe CurationConcerns::PersistDirectlyContainedOutputFileService do
+  let(:object)            { FileSet.create! { |fs| fs.apply_depositor_metadata('justin') } }
+  let(:file_path)         { File.join(fixture_path, 'test.tif') }
+  let(:file)              { File.new(file_path) }
+  let(:destination_name)  { 'the_derivative_name' }
+  let(:stream) { StringIO.new("fake file content") }
+
+  it "persists the file to the specified destination on the given object" do
+    described_class.call(stream, format: 'txt', url: object.uri, container: 'extracted_text')
+    expect(object.reload.extracted_text.content).to eq("fake file content")
+  end
+end

data/spec/services/repository_audit_service_spec.rb

@@ -3,16 +3,17 @@ require 'spec_helper'
 describe CurationConcerns::RepositoryAuditService do
   let(:user) { FactoryGirl.create(:user) }
   let!(:file) do
-    GenericFile.create! do |file|
-      file.add_file(File.open(fixture_path + '/world.png'), path: 'content', original_name: 'world.png')
-      file.apply_depositor_metadata(user)
+    fs = FileSet.create! do |f|
+      f.apply_depositor_metadata(user)
     end
+    Hydra::Works::AddFileToFileSet.call(fs, File.open(fixture_path + '/world.png'), :original_file)
+    fs
   end
 
-  describe "#audit_everything" do
-    it "should audit everything" do
-      expect_any_instance_of(GenericFile).to receive(:audit)
-      CurationConcerns::RepositoryAuditService.audit_everything
+  describe '#audit_everything' do
+    it 'audits everything' do
+      expect_any_instance_of(FileSet).to receive(:audit)
+      described_class.audit_everything
     end
   end
 end

data/spec/services/rights_service_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe RightsService do
+  describe "select_options" do
+    subject { described_class.select_options }
+
+    it "has a select list" do
+      expect(subject.first).to eq ["Attribution 3.0 United States", "http://creativecommons.org/licenses/by/3.0/us/"]
+    end
+  end
+
+  describe "label" do
+    subject { described_class.label("http://www.europeana.eu/portal/rights/rr-r.html") }
+
+    it { is_expected.to eq 'All rights reserved' }
+  end
+end