ddr-models 2.3.2 → 2.4.0.rc1

data/spec/jobs/fixity_check_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+module Ddr::Jobs
+  RSpec.describe FixityCheck do
+
+    it "should use the :fixity queue" do
+      expect(described_class.queue).to eq(:fixity)
+    end
+
+    describe ".perform" do
+      let!(:obj) { double(fixity_check: nil) }
+      before do
+        allow(ActiveFedora::Base).to receive(:find).with("test:1") { obj }
+      end
+      it "should call `fixity_check` on the object" do
+        expect(obj).to receive(:fixity_check)
+        described_class.perform("test:1")
+      end
+    end
+
+  end
+end

data/spec/jobs/job_spec.rb

@@ -0,0 +1,40 @@
+module Ddr::Jobs
+  RSpec.describe Job do
+
+    before(:all) do
+      class TestJob
+        extend Job
+
+        @queue = :test
+
+        def perform(object_id)
+          puts object_id
+        end
+      end
+    end
+
+    after(:all) do
+      Ddr::Jobs.send(:remove_const, :TestJob)
+    end
+
+    let(:queued) do
+      [{"class"=>"Ddr::Jobs::TestJob", "args"=>["test-1"]},
+       {"class"=>"Ddr::Jobs::OtherJob", "args"=>["test-2"]},
+       {"class"=>"Ddr::Jobs::TestJob", "args"=>["test-3"]},
+      ]
+    end
+
+    before(:each) do
+      allow(Resque).to receive(:size).with(:test) { 3 }
+      allow(Resque).to receive(:peek).with(:test, 0, 3) { queued }
+    end
+
+    describe ".queued_object_ids" do
+      it "returns the list of object_ids for queued jobs of this type" do
+        expect(TestJob.queued_object_ids)
+          .to contain_exactly("test-1", "test-3")
+      end
+    end
+
+  end
+end

data/spec/jobs/update_index_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+module Ddr::Jobs
+  RSpec.describe UpdateIndex do
+
+    it "should use the :index queue" do
+      expect(described_class.queue).to eq(:index)
+    end
+
+    describe ".perform" do
+      let!(:obj) { double(update_index: nil) }
+      before do
+        allow(ActiveFedora::Base).to receive(:find).with("test:1") { obj }
+      end
+      it "should call `update_index` on the object" do
+        expect(obj).to receive(:update_index)
+        described_class.perform("test:1")
+      end
+    end
+
+  end
+end

data/spec/managers/derivatives_manager_spec.rb

@@ -28,7 +28,7 @@ module Ddr
         context "all derivatives" do
           context "not multires_image_able" do
             let(:object) { ContentBearing.new }
-            context "content is image" do
+            context "content is an image" do
               let(:file) { fixture_file_upload("imageA.tif", "image/tiff") }
               it "should generate a thumbnail and not a ptif" do
                 expect(object.derivatives).to receive(:generate_derivative!).with(Ddr::Derivatives::DERIVATIVES[:thumbnail])
@@ -36,15 +36,7 @@ module Ddr
                 object.derivatives.update_derivatives(:now)
               end
             end
-            context "content is a PDF" do
-              let(:file) { fixture_file_upload("sample.pdf", "application/pdf") }
-              it "should generate a thumbnail and not a ptif" do
-                expect(object.derivatives).to receive(:generate_derivative!).with(Ddr::Derivatives::DERIVATIVES[:thumbnail])
-                expect(object.derivatives).to_not receive(:generate_derivative!).with(Ddr::Derivatives::DERIVATIVES[:multires_image])
-                object.derivatives.update_derivatives(:now)
-              end
-            end
-            context "content is neither an image nor a PDF" do
+            context "content is not an image" do
               let(:file) { fixture_file_upload("sample.docx", "application/vnd.openxmlformats-officedocument.wordprocessingml.document") }
               it "should generate neither a thumbnail nor a ptif" do
                 expect(object.derivatives).to_not receive(:generate_derivative!).with(Ddr::Derivatives::DERIVATIVES[:thumbnail])
@@ -64,7 +56,7 @@ module Ddr
               end
             end
             context "content is not tiff image" do
-              let(:file) { fixture_file_upload("sample.pdf", "application/pdf") }
+              let(:file) { fixture_file_upload("bird.jpg", "image/jpeg") }
               it "should generate a thumbnail but not a ptif" do
                 expect(object.derivatives).to receive(:generate_derivative!).with(Ddr::Derivatives::DERIVATIVES[:thumbnail])
                 expect(object.derivatives).to_not receive(:generate_derivative!).with(Ddr::Derivatives::DERIVATIVES[:multires_image])
@@ -85,7 +77,7 @@ module Ddr
             object.derivatives.update_derivatives(:now)
           end
         end
-end
+      end
 
       describe "derivative generation" do
         let(:file) { fixture_file_upload("imageA.tif", "image/tiff") }
@@ -120,6 +112,18 @@ end
         end
       end
 
+      describe "exception during derivative generation" do
+        let(:object) { ContentBearing.create }
+        before do
+          allow(Dir::Tmpname).to receive(:make_tmpname).with('', nil) { 'test-temp-dir' }
+          # simulate raising of exception during derivative generation
+          allow_any_instance_of(Ddr::Managers::DerivativesManager).to receive(:generate_derivative!).and_raise(StandardError)
+        end
+        it "should delete the temporary work directory" do
+          expect(File.exist?(File.join(Dir.tmpdir, 'test-temp-dir'))).to be false
+        end
+      end
+
     end
   end
 end

data/spec/models/admin_set_spec.rb

@@ -3,11 +3,12 @@ module Ddr::Models
 
     describe ".call" do
       subject { described_class.call(obj) }
+      let(:obj) { Item.new }
 
       describe "when the object has an admin set" do
-        let(:obj) { double(admin_set: "dvs") }
         before do
-          allow(described_class).to receive(:find).with(code: "dvs") do
+          obj.admin_set = "dvs"
+          allow(described_class).to receive(:get).with(:find, code: "dvs") do
             described_class.new("id"=>1, "code"=>"dvs", "title"=>"Data and Visualization Services", "created_at"=>"2015-09-15T16:15:58.514Z", "updated_at"=>"2015-09-15T16:15:58.514Z")
           end
         end
@@ -16,7 +17,6 @@ module Ddr::Models
       end
 
       describe "when the object does not have an admin set" do
-        let(:obj) { double(admin_set: nil) }
         it { is_expected.to be_nil }
       end
     end

data/spec/models/contact_spec.rb

@@ -0,0 +1,22 @@
+module Ddr::Models
+  RSpec.describe Contact do
+
+    describe ".call" do
+      subject { described_class.call(slug) }
+      let(:slug) { 'abc' }
+
+      before do
+        allow(described_class).to receive(:get).with(:find, slug: slug) do
+          described_class.new(
+              "id"=>1, "slug"=>"abc", "name"=>"A, B, and C Services", "short_name"=>"ABCS",
+              "url"=>"http://library.inst.edu/abc", "phone"=>"555-1234", "email"=>"abc@library.inst.edu",
+              "ask"=>"http://library.inst.edu/abc-ask", "created_at"=>"2015-09-15T16:15:58.514Z",
+              "updated_at"=>"2015-09-15T16:15:58.514Z")
+        end
+      end
+
+      its(:to_s) { is_expected.to eq("A, B, and C Services") }
+    end
+
+  end
+end

data/spec/models/effective_license_spec.rb

@@ -6,34 +6,41 @@ module Ddr::Models
     let(:url) { "https://creativecommons.org/licenses/by-nc-nd/4.0/" }
 
     let(:license) { License.new(url: url) }
-    before { allow(License).to receive(:find).with(url: url) { license } }
 
-    let(:obj) { double(pid: "test:1", id: "test:1", license: nil, parent: nil, admin_policy: nil, admin_policy_id: nil) }
+    let(:obj) { Component.new(pid: "test:1") }
+    let(:parent) { Item.new(pid: "test:2") }
+    let(:admin_policy) { Collection.new(pid: "test:3") }
 
     describe "when the object has a license" do
-      before { allow(obj).to receive(:license) { url } }
+      before do
+        allow(License).to receive(:call).with(obj) { license }
+      end
       it { is_expected.to eq(license) }
     end
 
     describe "when the object does not have a license" do
+      before do
+        allow(License).to receive(:call).with(obj) { nil }
+      end
       describe "and the object has a parent" do
-        let(:parent) { double(pid: "test:2", license: nil) }
         before do
           allow(obj).to receive(:parent) { parent }
         end
         describe "and the parent has a license" do
           before do
-            allow(parent).to receive(:license) { url }
+            allow(License).to receive(:call).with(parent) { license }
           end
           it { is_expected.to eq(license) }
         end
         describe "and the parent does not have a license" do
+          before do
+            allow(License).to receive(:call).with(parent) { nil }
+          end
           it { is_expected.to be_nil }
         end
       end
       describe "and the object does not have a parent" do
         describe "and the object has an admin policy" do
-          let(:admin_policy) { double(pid: "test:3", id: "test:3", license: nil) }
           before { allow(obj).to receive(:admin_policy) { admin_policy } }
           describe "and the admin policy has a different id from the object" do
             before do
@@ -41,11 +48,14 @@ module Ddr::Models
             end
             describe "and the admin policy has a license" do
               before do
-                allow(admin_policy).to receive(:license) { url }
+                allow(License).to receive(:call).with(admin_policy) { license }
               end
               it { is_expected.to eq(license) }
             end
             describe "and the admin policy does not have a license" do
+              before do
+                allow(License).to receive(:call).with(admin_policy) { nil }
+              end
               it { is_expected.to be_nil }
             end
           end

data/spec/models/file_characterization_spec.rb

@@ -0,0 +1,39 @@
+module Ddr::Models
+  RSpec.describe FileCharacterization do
+
+    subject { described_class.new(obj) }
+
+    before {
+      obj.content.checksumType = "SHA-1"
+      obj.save!
+    }
+
+    let(:obj) { FactoryGirl.create(:component) }
+    let(:fits_output) { "<fits/>" }
+
+    describe "when there is an error running FITS" do
+      before {
+        allow(subject).to receive(:run_fits).and_raise(FileCharacterization::FITSError)
+      }
+      specify {
+        begin
+          subject.call
+        rescue FileCharacterization::FITSError
+        ensure
+          expect(subject.fits).not_to have_content
+        end
+      }
+    end
+
+    describe "when FITS runs successfully" do
+      before {
+        allow(subject).to receive(:run_fits) { fits_output }
+      }
+      specify {
+        subject.call
+        expect(subject.fits.content).to eq(fits_output)
+      }
+    end
+
+  end
+end

data/spec/models/has_admin_metadata_spec.rb

@@ -178,14 +178,17 @@ module Ddr::Models
 
       describe "contacts" do
         before do
-          allow(YAML).to receive(:load_file) { { 'a' => { 'name' => 'Contact A', 'short_name' => 'A' },
-                                                 'b' => { 'name' => 'Contact B', 'short_name' => 'B' } } }
-          Ddr::Contacts.load_contacts
+          allow(Ddr::Models::Contact).to receive(:get).with(:find, slug: 'xa') do
+            Ddr::Models::Contact.new('id'=>1, 'slug'=>'xa', 'name'=>'Contact A', 'short_name'=>'A')
+          end
+          allow(Ddr::Models::Contact).to receive(:get).with(:find, slug: 'yb') do
+            Ddr::Models::Contact.new('id'=>1, 'slug'=>'yb', 'name'=>'Contact B', 'short_name'=>'B')
+          end
         end
         describe "#research_help" do
-          before { subject.research_help_contact = 'b' }
+          before { subject.research_help_contact = 'yb' }
           it "should return the appropriate contact" do
-            expect(subject.research_help.slug).to eq('b')
+            expect(subject.research_help.slug).to eq('yb')
           end
         end
       end

data/spec/models/indexing_spec.rb

@@ -21,6 +21,7 @@ module Ddr::Models
       obj.permanent_url = "http://id.library.duke.edu/ark:/99999/fk4zzz"
       obj.display_format = "Image"
       obj.roles.grant role1, role2, role3, role4
+      obj.aspace_id = "aspace_dccea43034e1b8261e14cf999e86449d"
     end
 
     its([Indexing::LICENSE]) { is_expected.to eq("cc-by-nc-nd-40") }
@@ -35,6 +36,7 @@ module Ddr::Models
     its([Indexing::ACCESS_ROLE]) { is_expected.to eq(obj.roles.to_json) }
     its([Indexing::POLICY_ROLE]) { is_expected.to contain_exactly(role2.agent.first, role3.agent.first, role4.agent.first) }
     its([Indexing::RESOURCE_ROLE]) { is_expected.to contain_exactly(role1.agent.first) }
+    its([Indexing::ASPACE_ID]) { is_expected.to eq("aspace_dccea43034e1b8261e14cf999e86449d") }
 
   end
 end

data/spec/models/license_spec.rb

@@ -8,7 +8,9 @@ module Ddr::Models
         let(:url) { "http://example.com" }
         let(:obj) { double(pid: "test:1", license: url) }
         before do
-          allow(described_class).to receive(:find).with(url: url) { described_class.new(url: url, title: "A License") }
+          allow(described_class).to receive(:get).with(:find, url: url) do
+            described_class.new(url: url, title: "A License")
+          end
         end
         its(:pid) { is_expected.to eq("test:1") }
         its(:to_s) { is_expected.to eq("A License") }

data/spec/models/solr_document_spec.rb

@@ -128,9 +128,12 @@ RSpec.describe SolrDocument, type: :model, contacts: true do
 
   describe "contacts" do
     before do
-      allow(YAML).to receive(:load_file) { { 'xa' => { 'name' => 'Contact A', 'short_name' => 'A' },
-                                             'yb' => { 'name' => 'Contact B', 'short_name' => 'B' } } }
-      Ddr::Contacts.load_contacts
+      allow(Ddr::Models::Contact).to receive(:get).with(:find, slug: 'xa') do
+        Ddr::Models::Contact.new('id'=>1, 'slug'=>'xa', 'name'=>'Contact A', 'short_name'=>'A')
+      end
+      allow(Ddr::Models::Contact).to receive(:get).with(:find, slug: 'yb') do
+        Ddr::Models::Contact.new('id'=>1, 'slug'=>'yb', 'name'=>'Contact B', 'short_name'=>'B')
+      end
     end
     describe "#research_help" do
       context "object has research help contact" do

data/spec/models/with_content_file_spec.rb

@@ -0,0 +1,37 @@
+module Ddr::Models
+  RSpec.describe WithContentFile do
+
+    let(:obj) { FactoryGirl.create(:component) }
+
+    before {
+      obj.content.checksumType = "SHA-1"
+      obj.save!
+    }
+
+    it "yields a temp file path to the block and deletes the temp file afterwards" do
+      WithContentFile.new(obj) do |path|
+        @path = path
+        expect(File.exist?(path)).to be true
+      end
+      expect(File.exist?(@path)).to be false
+    end
+
+    it "deletes the temp file even when an exception is raised in the block" do
+      begin
+        WithContentFile.new(obj) do |path|
+          @path = path
+          expect(File.exist?(path)).to be true
+          raise Error, "error"
+        end
+      rescue Error
+        expect(File.exist?(@path)).to be false
+      end
+    end
+
+    it "raises an exception when the checksum verification fails" do
+      allow(obj.content).to receive(:checksum) { "foo" }
+      expect { WithContentFile.new(obj) { |p| nil } }.to raise_error(ChecksumInvalid)
+    end
+
+  end
+end

data/spec/spec_helper.rb

@@ -1,5 +1,7 @@
 ENV['RAILS_ENV'] ||= "test"
 
+ENV['DDR_AUX_API_URL'] ||= "http://localhost:3000/api"
+
 require File.expand_path("../dummy/config/environment.rb",  __FILE__)
 
 require "ddr-models"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ddr-models
 version: !ruby/object:Gem::Version
-  version: 2.3.2
+  version: 2.4.0.rc1
 platform: ruby
 authors:
 - Jim Coble
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-01 00:00:00.000000000 Z
+date: 2015-12-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -25,6 +25,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 4.1.6
+- !ruby/object:Gem::Dependency
+  name: activeresource
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: active-fedora
   requirement: !ruby/object:Gem::Requirement
@@ -185,26 +199,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.12'
-- !ruby/object:Gem::Dependency
-  name: ddr-aux-client
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.2'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.2.2
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.2'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.2.2
 - !ruby/object:Gem::Dependency
   name: ddr-antivirus
   requirement: !ruby/object:Gem::Requirement
@@ -428,7 +422,6 @@ files:
 - lib/ddr/auth/test_helpers.rb
 - lib/ddr/auth/user.rb
 - lib/ddr/auth/web_auth_context.rb
-- lib/ddr/contacts.rb
 - lib/ddr/datastreams.rb
 - lib/ddr/datastreams/administrative_metadata_datastream.rb
 - lib/ddr/datastreams/datastream_behavior.rb
@@ -469,10 +462,13 @@ files:
 - lib/ddr/index/query_value.rb
 - lib/ddr/index/response.rb
 - lib/ddr/index/unique_key_field.rb
-- lib/ddr/index_fields.rb
 - lib/ddr/jobs.rb
 - lib/ddr/jobs/fits_file_characterization.rb
+- lib/ddr/jobs/fixity_check.rb
+- lib/ddr/jobs/job.rb
 - lib/ddr/jobs/permanent_id.rb
+- lib/ddr/jobs/queue.rb
+- lib/ddr/jobs/update_index.rb
 - lib/ddr/managers.rb
 - lib/ddr/managers/derivatives_manager.rb
 - lib/ddr/managers/manager.rb
@@ -485,10 +481,12 @@ files:
 - lib/ddr/models/access_controllable.rb
 - lib/ddr/models/admin_set.rb
 - lib/ddr/models/base.rb
+- lib/ddr/models/contact.rb
 - lib/ddr/models/describable.rb
 - lib/ddr/models/engine.rb
 - lib/ddr/models/error.rb
 - lib/ddr/models/event_loggable.rb
+- lib/ddr/models/file_characterization.rb
 - lib/ddr/models/file_management.rb
 - lib/ddr/models/finding_aid.rb
 - lib/ddr/models/fixity_checkable.rb
@@ -510,6 +508,7 @@ files:
 - lib/ddr/models/struct_div.rb
 - lib/ddr/models/structure.rb
 - lib/ddr/models/version.rb
+- lib/ddr/models/with_content_file.rb
 - lib/ddr/models/year_facet.rb
 - lib/ddr/notifications.rb
 - lib/ddr/utils.rb
@@ -531,7 +530,6 @@ files:
 - spec/auth/effective_roles_spec.rb
 - spec/auth/group_spec.rb
 - spec/auth/groups_spec.rb
-- spec/auth/ldap_gateway_spec.rb
 - spec/auth/legacy_default_permissions_spec.rb
 - spec/auth/legacy_permissions_spec.rb
 - spec/auth/roles/detached_role_set_spec.rb
@@ -542,7 +540,6 @@ files:
 - spec/auth/superuser_ability_spec.rb
 - spec/auth/user_spec.rb
 - spec/auth/web_auth_context_spec.rb
-- spec/contacts/contacts_spec.rb
 - spec/controllers/application_controller_spec.rb
 - spec/controllers/including_role_based_access_controls_enforcement_spec.rb
 - spec/controllers/users/sessions_controller_spec.rb
@@ -607,6 +604,8 @@ files:
 - spec/factories/user_factories.rb
 - spec/fixtures/16bit.tif
 - spec/fixtures/8bit.tif
+- spec/fixtures/arrow1rightred_e0.gif
+- spec/fixtures/bird.jpg
 - spec/fixtures/fits/document.xml
 - spec/fixtures/fits/image.xml
 - spec/fixtures/imageA.tif
@@ -619,7 +618,11 @@ files:
 - spec/index/filter_spec.rb
 - spec/index/filters_spec.rb
 - spec/index/query_spec.rb
+- spec/index/query_value_spec.rb
 - spec/jobs/fits_file_characterization_spec.rb
+- spec/jobs/fixity_check_spec.rb
+- spec/jobs/job_spec.rb
+- spec/jobs/update_index_spec.rb
 - spec/managers/derivatives_manager_spec.rb
 - spec/managers/technical_metadata_manager_spec.rb
 - spec/models/active_fedora_base_spec.rb
@@ -628,9 +631,11 @@ files:
 - spec/models/attachment_spec.rb
 - spec/models/collection_spec.rb
 - spec/models/component_spec.rb
+- spec/models/contact_spec.rb
 - spec/models/descriptive_metadata_datastream_spec.rb
 - spec/models/effective_license_spec.rb
 - spec/models/events_spec.rb
+- spec/models/file_characterization_spec.rb
 - spec/models/file_management_spec.rb
 - spec/models/finding_aid_spec.rb
 - spec/models/has_admin_metadata_spec.rb
@@ -643,6 +648,7 @@ files:
 - spec/models/struct_div_spec.rb
 - spec/models/structure_spec.rb
 - spec/models/target_spec.rb
+- spec/models/with_content_file_spec.rb
 - spec/models/year_facet_spec.rb
 - spec/routing/user_routing_spec.rb
 - spec/spec_helper.rb
@@ -676,9 +682,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.2.2
@@ -695,7 +701,6 @@ test_files:
 - spec/auth/effective_roles_spec.rb
 - spec/auth/group_spec.rb
 - spec/auth/groups_spec.rb
-- spec/auth/ldap_gateway_spec.rb
 - spec/auth/legacy_default_permissions_spec.rb
 - spec/auth/legacy_permissions_spec.rb
 - spec/auth/roles/detached_role_set_spec.rb
@@ -706,7 +711,6 @@ test_files:
 - spec/auth/superuser_ability_spec.rb
 - spec/auth/user_spec.rb
 - spec/auth/web_auth_context_spec.rb
-- spec/contacts/contacts_spec.rb
 - spec/controllers/application_controller_spec.rb
 - spec/controllers/including_role_based_access_controls_enforcement_spec.rb
 - spec/controllers/users/sessions_controller_spec.rb
@@ -771,6 +775,8 @@ test_files:
 - spec/factories/user_factories.rb
 - spec/fixtures/16bit.tif
 - spec/fixtures/8bit.tif
+- spec/fixtures/arrow1rightred_e0.gif
+- spec/fixtures/bird.jpg
 - spec/fixtures/fits/document.xml
 - spec/fixtures/fits/image.xml
 - spec/fixtures/imageA.tif
@@ -783,7 +789,11 @@ test_files:
 - spec/index/filter_spec.rb
 - spec/index/filters_spec.rb
 - spec/index/query_spec.rb
+- spec/index/query_value_spec.rb
 - spec/jobs/fits_file_characterization_spec.rb
+- spec/jobs/fixity_check_spec.rb
+- spec/jobs/job_spec.rb
+- spec/jobs/update_index_spec.rb
 - spec/managers/derivatives_manager_spec.rb
 - spec/managers/technical_metadata_manager_spec.rb
 - spec/models/active_fedora_base_spec.rb
@@ -792,9 +802,11 @@ test_files:
 - spec/models/attachment_spec.rb
 - spec/models/collection_spec.rb
 - spec/models/component_spec.rb
+- spec/models/contact_spec.rb
 - spec/models/descriptive_metadata_datastream_spec.rb
 - spec/models/effective_license_spec.rb
 - spec/models/events_spec.rb
+- spec/models/file_characterization_spec.rb
 - spec/models/file_management_spec.rb
 - spec/models/finding_aid_spec.rb
 - spec/models/has_admin_metadata_spec.rb
@@ -807,6 +819,7 @@ test_files:
 - spec/models/struct_div_spec.rb
 - spec/models/structure_spec.rb
 - spec/models/target_spec.rb
+- spec/models/with_content_file_spec.rb
 - spec/models/year_facet_spec.rb
 - spec/routing/user_routing_spec.rb
 - spec/spec_helper.rb