blacklight-spotlight 3.0.0.alpha.8 → 3.0.0.rc3

data/spec/models/solr_document_spec.rb

@@ -4,14 +4,13 @@ describe SolrDocument, type: :model do
   subject { document }
 
   let(:document) { described_class.new(id: 'abcd123') }
+  let(:exhibit_alt) { FactoryBot.create(:exhibit) }
+  let(:exhibit) { FactoryBot.create(:exhibit) }
 
   before do
     allow(subject).to receive_messages(reindex: nil)
   end
 
-  let(:exhibit_alt) { FactoryBot.create(:exhibit) }
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-
   its(:to_key) { is_expected.to eq ['abcd123'] }
   its(:persisted?) { is_expected.to be_truthy }
 

data/spec/models/spotlight/access_controls_enforcement_search_builder_spec.rb

@@ -3,6 +3,7 @@
 describe Spotlight::AccessControlsEnforcementSearchBuilder do
   class MockSearchBuilder < Blacklight::SearchBuilder
     attr_reader :blacklight_params, :scope
+
     def initialize(blacklight_params, scope)
       @blacklight_params = blacklight_params
       @scope = scope

data/spec/models/spotlight/exhibit_spec.rb

@@ -19,6 +19,14 @@ describe Spotlight::Exhibit, type: :model do
     expect(subject.description).to eq 'Test description'
   end
 
+  it 'has reserved slugs' do
+    subject.slug = 'site'
+
+    expect do
+      subject.save!
+    end.to raise_error(ActiveRecord::RecordInvalid, 'Validation failed: Slug is reserved')
+  end
+
   describe 'validations' do
     it 'validates the presence of the title' do
       exhibit.title = ''
@@ -28,7 +36,7 @@ describe Spotlight::Exhibit, type: :model do
     end
 
     it 'does not validate the presence of the title under a non-default locale' do
-      expect(I18n).to receive(:locale).and_return(:fr)
+      allow(I18n).to receive(:locale).and_return(:fr)
       exhibit.title = ''
       expect do
         exhibit.save
@@ -147,7 +155,7 @@ describe Spotlight::Exhibit, type: :model do
     it 'removes the default browse category' do
       subject.save
       expect { subject.import({}) }.to change { subject.searches.count }.by(0)
-      expect { subject.import('searches' => [{ 'title' => 'All Exhibit Items', 'slug' => 'all-exhibit-items' }]) }.to change { subject.searches.count }.by(0)
+      expect { subject.import('searches' => [{ 'title' => 'All exhibit items', 'slug' => 'all-exhibit-items' }]) }.to change { subject.searches.count }.by(0)
     end
 
     it 'imports nested attributes from the hash' do
@@ -402,6 +410,7 @@ describe Spotlight::Exhibit, type: :model do
       I18n.locale = 'fr'
       persisted_exhibit.reload
       expect(persisted_exhibit.title).to eq 'Titre français'
+      expect(persisted_exhibit.translated_title).to eq 'Titre français'
     end
 
     it 'has a translatable subtitle' do
@@ -417,5 +426,12 @@ describe Spotlight::Exhibit, type: :model do
       persisted_exhibit.reload
       expect(persisted_exhibit.description).to eq 'Description français'
     end
+
+    it 'is nil if no translation has been prrovided' do
+      expect(persisted_exhibit.title).to eq 'Sample'
+      I18n.locale = 'es'
+      persisted_exhibit.reload
+      expect(persisted_exhibit.translated_title(default: '')).to eq nil
+    end
   end
 end

data/spec/models/spotlight/featured_image_spec.rb

@@ -3,6 +3,22 @@
 describe Spotlight::FeaturedImage do
   subject(:featured_image) { described_class.new }
 
+  let(:temp_image) { FactoryBot.create(:temporary_image) }
+
+  context 'with an uploaded image' do
+    it 'copies the temporary uploaded image to this model' do
+      featured_image.source = 'remote'
+      featured_image.upload_id = temp_image.id
+
+      featured_image.save
+
+      expect(featured_image.image.filename).to eq temp_image.image.filename
+      expect(featured_image.image.read).to eq temp_image.image.read
+
+      expect { temp_image.reload }.to raise_exception ActiveRecord::RecordNotFound
+    end
+  end
+
   describe '#iiif_url' do
     let(:iiif_tilesource) { 'http://example.com/iiif/abc123/info.json' }
     let(:iiif_region) { '0,0,400,300' }
@@ -17,6 +33,17 @@ describe Spotlight::FeaturedImage do
         expect(subject.iiif_url).to match(%r{^http://example.com/iiif/abc123/})
         expect(subject.iiif_url).not_to include('info.json')
       end
+
+      context 'with an uploaded image' do
+        before do
+          featured_image.image = temp_image.image
+          featured_image.save!
+        end
+
+        it 'points at the RIIIF endpoint' do
+          expect(subject.iiif_url).to match(%r{^/images/\d+/})
+        end
+      end
     end
 
     describe 'region' do

data/spec/models/spotlight/group_spec.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+describe Spotlight::Group, type: :model do
+  let(:exhibit) { FactoryBot.create(:exhibit) }
+
+  describe '#searches' do
+    let(:group) { FactoryBot.create(:group_with_searches, exhibit: exhibit, searches_count: 4) }
+
+    it do
+      expect(group.searches.count).to eq 4
+    end
+
+    it do
+      group.searches.all do |search|
+        expect(search).to be_an Spotlight::Search
+      end
+    end
+  end
+end

data/spec/models/spotlight/main_navigation_spec.rb

@@ -7,7 +7,7 @@ describe Spotlight::MainNavigation, type: :model do
 
   it 'has a default_label' do
     subject.nav_type = :curated_features
-    expect(subject.default_label).to eq 'Curated Features'
+    expect(subject.default_label).to eq 'Curated features'
   end
 
   it 'returns the use the default label in the absence of a label' do

data/spec/models/spotlight/page_spec.rb

@@ -119,7 +119,7 @@ describe Spotlight::Page, type: :model do
   end
 
   describe '#slug' do
-    let(:page) { FactoryBot.create(:feature_page) }
+    let(:page) { FactoryBot.create(:about_page) }
 
     it 'gets a default slug' do
       expect(page.slug).not_to be_blank
@@ -130,6 +130,11 @@ describe Spotlight::Page, type: :model do
       expect(page.slug).to eq 'abc'
     end
 
+    it 'blacklists contacts' do
+      page.update(title: 'contacts')
+      expect(page.slug).not_to eq 'contacts'
+    end
+
     context 'with a custom slug' do
       let(:page) { FactoryBot.create(:feature_page, slug: 'xyz') }
 

data/spec/models/spotlight/resources/upload_spec.rb

@@ -1,107 +1,71 @@
 # frozen_string_literal: true
 
 describe Spotlight::Resources::Upload, type: :model do
-  let!(:exhibit) { FactoryBot.create :exhibit }
-  let!(:custom_field) { FactoryBot.create :custom_field, exhibit: exhibit }
-  let(:resource) { described_class.new(exhibit: exhibit) }
-  let(:doc_builder) { resource.document_builder }
+  subject(:upload) { described_class.new(id: 42, exhibit: exhibit) }
 
-  let(:configured_fields) { [title_field] + described_class.fields(exhibit) }
-  let(:title_field) { Spotlight::UploadFieldConfig.new(field_name: 'configured_title_field') }
-  let(:upload_data) do
-    {
-      title_field.field_name => 'Title Data',
-      'spotlight_upload_description_tesim' => 'Description Data',
-      'spotlight_upload_attribution_tesim' => 'Attribution Data',
-      'spotlight_upload_date_tesim' => 'Date Data',
-      custom_field.field => 'Custom Field Data'
-    }
-  end
-  let(:featured_image) { FactoryBot.create(:featured_image, image: File.open(File.join(FIXTURES_PATH, '800x600.png'))) }
-
-  before do
-    Rails.cache.clear # wipes out any cached image info.
-    allow(resource).to receive(:configured_fields).and_return configured_fields
-    allow(described_class).to receive(:fields).and_return configured_fields
-
-    allow(resource.send(:blacklight_solr)).to receive(:update)
-    allow(Spotlight::Engine.config).to receive(:upload_title_field).and_return(title_field)
-    resource.data = upload_data
-    resource.upload = featured_image
-    resource.save
-  end
+  let(:exhibit) { FactoryBot.create(:exhibit) }
 
-  context 'with a custom upload title field' do
-    subject { doc_builder.to_solr }
-
-    let(:title_field) { Spotlight::UploadFieldConfig.new(field_name: 'configured_title_field', solr_fields: [:some_other_field]) }
+  describe '.fields' do
+    it "includes the exhibit's uploaded resource fields" do
+      expect(described_class.fields(exhibit)).to include(*exhibit.uploaded_resource_fields)
+    end
 
-    describe '#to_solr' do
-      it 'stores the title field in the provided solr field' do
-        expect(subject[:some_other_field]).to eq 'Title Data'
+    context 'title field' do
+      it 'is an UploadFieldConfig object for the configured index title_field' do
+        upload_config = described_class.fields(exhibit).first
+        expect(upload_config).to be_a Spotlight::UploadFieldConfig
+        expect(upload_config.field_name).to eq exhibit.blacklight_config.index.title_field
       end
     end
   end
 
-  context 'multiple solr field mappings' do
-    let :configured_fields do
-      [
-        Spotlight::UploadFieldConfig.new(field_name: 'some_field', solr_fields: %w[a b])
-      ]
-    end
-
-    let :upload_data do
-      { 'some_field' => 'value' }
+  describe '#compound_id' do
+    it 'appends the object ID w/ the exhibit ID' do
+      expect(upload.compound_id).to eq "#{exhibit.id}-42"
     end
+  end
 
-    describe '#to_solr' do
-      subject { doc_builder.to_solr }
+  describe '#sidecar' do
+    it 'is a SolrDocumentSidecar with the correct relationships' do
+      sidecar = upload.sidecar
 
-      it 'maps a single uploaded field to multiple solr fields' do
-        expect(subject).not_to include 'some_field'
-        expect(subject['a']).to eq 'value'
-        expect(subject['b']).to eq 'value'
-      end
+      expect(sidecar).to be_a Spotlight::SolrDocumentSidecar
+      expect(sidecar.exhibit_id).to eq exhibit.id
+      expect(sidecar.document_id).to eq "#{exhibit.id}-42"
     end
   end
 
-  describe '#to_solr' do
-    subject { doc_builder.to_solr }
-
-    it 'has the exhibit id and the upload id as the solr id' do
-      expect(subject[:id]).to eq "#{resource.exhibit.id}-#{resource.id}"
+  context 'when creating' do
+    before do
+      allow(upload).to receive(:write?).and_return(false)
     end
 
-    it 'has a title field using the exhibit specific blacklight_config' do
-      expect(subject['configured_title_field']).to eq 'Title Data'
-    end
+    it 'the sidecar is updated with the apporpriate data from configured fields' do
+      expect(upload.sidecar).to receive(:update).with(
+        data: {
+          'configured_fields' => {
+            'full_title_tesim' => 'My Upload Title',
+            'spotlight_upload_date_tesim' => 'My Upload Date'
+          }
+        }
+      )
 
-    it 'has the other additional configured fields' do
-      expect(subject[:spotlight_upload_description_tesim]).to eq 'Description Data'
-      expect(subject[:spotlight_upload_attribution_tesim]).to eq 'Attribution Data'
-      expect(subject[:spotlight_upload_date_tesim]).to eq 'Date Data'
-    end
+      upload.data = { 'full_title_tesim' => 'My Upload Title', 'spotlight_upload_date_tesim' => 'My Upload Date' }
 
-    it 'has a spotlight_resource_type field' do
-      expect(subject[:spotlight_resource_type_ssim]).to eq 'spotlight/resources/uploads'
+      upload.save
     end
 
-    it 'has the various image fields' do
-      expect(subject).to have_key Spotlight::Engine.config.thumbnail_field
-    end
+    it 'the sidecar is updated with the appropriate data from custom fields' do
+      FactoryBot.create(:custom_field, exhibit: exhibit, slug: 'custom_field_1')
+      FactoryBot.create(:custom_field, exhibit: exhibit, slug: 'custom_field_2')
 
-    it 'has the full image dimensions fields' do
-      expect(subject[:spotlight_full_image_height_ssm]).to eq 600
-      expect(subject[:spotlight_full_image_width_ssm]).to eq 800
-    end
+      expect(upload.sidecar).to receive(:update).with(
+        data: hash_including('custom_field_1' => 'Custom Field 1 Data', 'custom_field_2' => 'Custom Field 2 Data')
+      )
 
-    it 'has fields representing exhibit specific custom fields' do
-      expect(subject[custom_field.solr_field]).to eq 'Custom Field Data'
-    end
+      upload.data = { 'custom_field_1' => 'Custom Field 1 Data', 'custom_field_2' => 'Custom Field 2 Data' }
 
-    it 'has a field for the iiif manifest url' do
-      manifest_path = Spotlight::Engine.routes.url_helpers.manifest_exhibit_solr_document_path(exhibit, resource.compound_id)
-      expect(subject[Spotlight::Engine.config.iiif_manifest_field]).to eq(manifest_path)
+      upload.save
     end
   end
 end

data/spec/models/spotlight/role_spec.rb

@@ -9,7 +9,7 @@ describe Spotlight::Role, type: :model do
 
       it 'does not be valid' do
         expect(subject).not_to be_valid
-        expect(subject.errors.messages).to include(role: ['is not included in the list'], 'user.email': ["can't be blank"])
+        expect(subject.errors.messages.transform_values(&:to_a)).to include(role: ['is not included in the list'], 'user.email': ["can't be blank"])
       end
     end
 
@@ -37,10 +37,10 @@ describe Spotlight::Role, type: :model do
 
       describe 'that points at a user with an existing role' do
         let(:user) { FactoryBot.create(:user) }
-        before { described_class.create!(role: 'curator', user: user) }
-
         let(:args) { { role: 'curator', user_key: user.email } }
 
+        before { described_class.create!(role: 'curator', user: user) }
+
         it 'is valid' do
           expect(subject).not_to be_valid
           expect(subject.errors.messages).to eq(user_key: ['already a member of this exhibit'])

data/spec/models/spotlight/search_spec.rb

@@ -5,7 +5,7 @@ describe Spotlight::Search, type: :model do
 
   let(:exhibit) { FactoryBot.create(:exhibit) }
 
-  let(:query_params) { { 'f' => { 'genre_sim' => ['map'] } } }
+  let(:query_params) { { 'f' => { 'genre_ssim' => ['map'] } } }
 
   let(:blacklight_config) { ::CatalogController.blacklight_config }
   let(:document) do
@@ -17,7 +17,10 @@ describe Spotlight::Search, type: :model do
     let(:query_params) { {} }
 
     it 'has items' do
-      expect(subject.documents.size).to eq 55
+      # Add some fuzziness to this. Running locally it should be 55 but in some
+      # CI environments, spec/features/add_items_spec.rb will end up adding new
+      # records and cannot ensure deletion of them.
+      expect(subject.documents.size).to be_within(1).of(55)
     end
   end
 
@@ -31,6 +34,16 @@ describe Spotlight::Search, type: :model do
     end
   end
 
+  describe '.unpublished' do
+    let!(:search1) { FactoryBot.create(:search, published: true) }
+    let!(:search2) { FactoryBot.create(:search, published: false) }
+    let!(:search3) { FactoryBot.create(:search, published: nil) }
+
+    it 'accounts for nil and false values' do
+      expect(described_class.unpublished.count).to eq 5
+    end
+  end
+
   describe '#slug' do
     let(:search) { FactoryBot.create(:search) }
 
@@ -54,7 +67,7 @@ describe Spotlight::Search, type: :model do
 
   describe '#search_params' do
     it 'maps the search to the appropriate facet values' do
-      expect(subject.search_params.to_hash).to include 'fq' => array_including('{!term f=genre_sim}map')
+      expect(subject.search_params.to_hash).to include 'fq' => array_including('{!term f=genre_ssim}map')
     end
 
     context 'with filter_resources_by_exhibit configured' do
@@ -95,4 +108,18 @@ describe Spotlight::Search, type: :model do
       expect(search_params).to include user_params
     end
   end
+
+  describe '#groups' do
+    let(:search) { FactoryBot.create(:search_with_groups, groups_count: 3) }
+
+    it do
+      expect(search.groups.count).to eq 3
+    end
+
+    it do
+      search.groups.all do |group|
+        expect(group).to be_an Spotlight::Group
+      end
+    end
+  end
 end

data/spec/{serializers/spotlight/exhibit_export_serializer_spec.rb → services/spotlight/exhibit_import_export_service_spec.rb}

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe Spotlight::ExhibitExportSerializer do
+describe Spotlight::ExhibitImportExportService do
   subject { JSON.parse(described_class.new(source_exhibit).to_json) }
 
   let!(:source_exhibit) { FactoryBot.create(:exhibit) }
@@ -24,6 +24,10 @@ describe Spotlight::ExhibitExportSerializer do
     expect(subject['searches']).to have(source_exhibit.searches.count).searches
   end
 
+  it 'has group attributes' do
+    expect(subject['groups']).to have(source_exhibit.groups.count).groups
+  end
+
   it 'has home page attributes' do
     expect(subject).to have_key 'home_page'
     expect(subject['home_page']).not_to have_key 'id'
@@ -83,29 +87,43 @@ describe Spotlight::ExhibitExportSerializer do
 
   describe 'should round-trip data' do
     subject do
-      e = FactoryBot.create(:exhibit)
-      e.import(export).tap(&:save)
+      destination_exhibit.import(export)
+      destination_exhibit
     end
 
-    before do
-      sidecar = source_exhibit.solr_document_sidecars.create! document: SolrDocument.new(id: 1), public: false
-      source_exhibit.tag(sidecar, with: 'xyz', on: :tags)
-    end
+    let(:destination_exhibit) { FactoryBot.create(:exhibit) }
 
     let :export do
       described_class.new(source_exhibit).as_json
     end
 
+    before do
+      sidecar = source_exhibit.solr_document_sidecars.create! document: SolrDocument.new(id: 1), public: false
+      source_exhibit.tag(sidecar, with: 'xyz', on: :tags)
+    end
+
     it 'has exhibit properties' do
       expect(subject.title).to eq source_exhibit.title
     end
 
+    context 'with a theme that does not exist in the destination' do
+      it 'ignores the invalid theme' do
+        source_exhibit.theme = 'xyz'
+        expect(subject.theme).to eq nil
+      end
+    end
+
     it 'does not duplicate saved searches' do
       expect(subject.searches).to have(1).item
     end
 
     it 'has blacklight configuration properties' do
+      subject.blacklight_configuration.index_fields_will_change!
+      subject.blacklight_configuration.index_fields['language_ssm'] = { some_value: true }
+      subject.blacklight_configuration.save
+
       expect(subject.blacklight_configuration).to be_persisted
+      expect(subject.blacklight_configuration.blacklight_config.index_fields['language_ssm'].to_h).to include some_value: true
     end
 
     context 'for an exhibit without search fields' do
@@ -221,7 +239,7 @@ describe Spotlight::ExhibitExportSerializer do
       before do
         feature_page.content = { data: [{ type: 'text', data: { text: 'xyz' } }] }.to_json
         feature_page.thumbnail = thumbnail
-        feature_page.save
+        feature_page.save!
       end
 
       it 'copies the masthead' do
@@ -239,6 +257,21 @@ describe Spotlight::ExhibitExportSerializer do
         expect(subject.feature_pages.first.content.length).to eq 1
         expect(subject.feature_pages.first.content.first).to be_a_kind_of SirTrevorRails::Blocks::TextBlock
       end
+
+      context 'with a translation' do
+        before do
+          localized_page = feature_page.clone_for_locale('ar')
+          localized_page.content = { data: [{ type: 'text', data: { text: 'xyz-in-ar' } }] }.to_json
+          localized_page.save!
+        end
+
+        it 'transfers the localized page' do
+          expect(subject.feature_pages.first.translated_pages.length).to eq 1
+          expect(JSON.parse(subject.feature_pages.first.translated_page_for('ar').read_attribute(:content))).to have_key 'data'
+          expect(subject.feature_pages.first.translated_page_for('ar').content.length).to eq 1
+          expect(subject.feature_pages.first.translated_page_for('ar').content.first).to be_a_kind_of SirTrevorRails::Blocks::TextBlock
+        end
+      end
     end
 
     it 'assigns STI resources the correct class' do
@@ -259,19 +292,31 @@ describe Spotlight::ExhibitExportSerializer do
       let(:masthead) { FactoryBot.create(:masthead) }
       let(:thumbnail) { FactoryBot.create(:featured_image) }
       let!(:search) { FactoryBot.create(:search, exhibit: source_exhibit, masthead: masthead, thumbnail: thumbnail) }
+      let!(:existing_search) { FactoryBot.create(:search, slug: search.slug, exhibit: destination_exhibit) }
 
       before do
         source_exhibit.reload
+        destination_exhibit.reload
+      end
+
+      it 'copies the title' do
+        subject
+        existing_search.reload
+        expect(existing_search.title).to eq search.title
       end
 
       it 'copies the masthead' do
-        expect(subject.searches.last.masthead).not_to be_blank
-        expect(subject.searches.last.masthead.image.file.path).not_to eq search.masthead.image.file.path
+        subject
+        existing_search.reload
+        expect(existing_search.masthead).not_to be_blank
+        expect(existing_search.masthead.image.file.path).not_to eq search.masthead.image.file.path
       end
 
       it 'copies the thumbnail' do
-        expect(subject.searches.first.thumbnail).not_to be_blank
-        expect(subject.searches.first.thumbnail.image.file.path).not_to eq search.thumbnail.image.file.path
+        subject
+        existing_search.reload
+        expect(existing_search.thumbnail).not_to be_blank
+        expect(existing_search.thumbnail.image.file.path).not_to eq search.thumbnail.image.file.path
       end
 
       context 'without an attached image' do
@@ -281,8 +326,24 @@ describe Spotlight::ExhibitExportSerializer do
         end
 
         it 'copies the masthead without an image' do
-          expect(subject.searches.last.masthead).not_to be_blank
-          expect(subject.searches.last.masthead.image).to be_blank
+          subject
+          existing_search.reload
+          expect(existing_search.masthead).not_to be_blank
+          expect(existing_search.masthead.image).to be_blank
+        end
+      end
+
+      context 'remote thumbnail with existing tilesource' do
+        it do
+          search.thumbnail.iiif_tilesource
+          search.thumbnail.save
+          source_exhibit.reload
+        end
+
+        it 'unsets the iiif_tilesource' do
+          subject
+          existing_search.reload
+          expect(existing_search.thumbnail.iiif_tilesource).not_to eq search.thumbnail.iiif_tilesource
         end
       end
 
@@ -290,13 +351,44 @@ describe Spotlight::ExhibitExportSerializer do
         before do
           search.masthead.document_global_id = SolrDocument.new(id: 'xyz').to_global_id
           search.masthead.source = 'exhibit'
+          search.masthead.iiif_tilesource = 'foo'
+          search.masthead.image = nil # Setting to nil to mimick an exhibit source
           search.masthead.save
 
           source_exhibit.reload
         end
 
         it 'copies the resource' do
-          expect(subject.searches.last.masthead).not_to be_blank
+          subject
+          existing_search.reload
+          expect(existing_search.masthead).not_to be_blank
+        end
+
+        it 'does not unset the iiif_tilesource' do
+          subject
+          existing_search.reload
+          expect(existing_search.masthead.iiif_tilesource).to eq search.masthead.iiif_tilesource
+        end
+      end
+
+      context 'in a group' do
+        let!(:group) { FactoryBot.create(:group, title: 'blah', exhibit: source_exhibit) }
+
+        before do
+          search.groups << group
+          search.save!
+        end
+
+        it 'copies the group' do
+          subject
+          expect(destination_exhibit.groups.length).to eq 1
+          expect(destination_exhibit.groups.first.as_json).to include(group.as_json.slice('slug', 'title', 'weight', 'published'))
+        end
+
+        it 'copies the group membership' do
+          subject
+          expect(existing_search.reload.groups.length).to eq 1
+          expect(existing_search.groups.pluck(:slug)).to match_array [group.slug]
         end
       end
     end
@@ -328,18 +420,71 @@ describe Spotlight::ExhibitExportSerializer do
     end
   end
 
-  it 'is idempotent-ish' do
-    FactoryBot.create :feature_subpage, exhibit: source_exhibit
-    export = described_class.new(source_exhibit).as_json
-    e = FactoryBot.create(:exhibit)
-    e.import(export).tap(&:save)
-    e.import(export).tap(&:save)
+  context 'testing idempotency' do
+    let(:masthead) { FactoryBot.create(:masthead) }
+    let(:thumbnail) { FactoryBot.create(:exhibit_thumbnail) }
+    let!(:search) { FactoryBot.create(:search, exhibit: source_exhibit, masthead: FactoryBot.create(:masthead), thumbnail: FactoryBot.create(:featured_image)) }
+    let!(:feature_page_1) { FactoryBot.create(:feature_page, exhibit: source_exhibit) }
+    let!(:feature_page_2) { FactoryBot.create(:feature_page, exhibit: source_exhibit) }
+
+    before do
+      source_exhibit.masthead = masthead
+      source_exhibit.thumbnail = thumbnail
+    end
+
+    # From Rails 6:
+    def _deep_transform_values_in_object(object, &block)
+      case object
+      when Hash
+        object.transform_values { |value| _deep_transform_values_in_object(value, &block) }
+      when Array
+        object.map { |e| _deep_transform_values_in_object(e, &block) }
+      else
+        yield(object)
+      end
+    end
+
+    it 'is re-runnable' do
+      FactoryBot.create :feature_subpage, exhibit: source_exhibit
+      export = described_class.new(source_exhibit).as_json
+      e = FactoryBot.create(:exhibit)
+      e.import(export)
+      e.import(export)
+    end
+
+    it 'is idempotent-ish' do
+      export = described_class.new(source_exhibit).as_json
+      e = FactoryBot.create(:exhibit)
+      e.import(export)
+      new_export = described_class.new(e).as_json
+
+      actual = _deep_transform_values_in_object(new_export) do |v|
+        if DateTime.parse(v.to_s)
+          nil
+        else
+          v
+        end
+      rescue ArgumentError
+        nil
+      end
+      expected = _deep_transform_values_in_object(export) do |v|
+        if DateTime.parse(v.to_s)
+          nil
+        else
+          v
+        end
+      rescue ArgumentError
+        nil
+      end
+      expect(actual).to include expected
+    end
   end
 
   describe 'should export saved searches with query parameters that can be re-generated' do
     subject do
       e = FactoryBot.create(:exhibit)
-      e.import(export).tap(&:save)
+      e.import(export)
+      e
     end
 
     before do
@@ -418,7 +563,7 @@ describe Spotlight::ExhibitExportSerializer do
     end
 
     it 'includes only the page-related data' do
-      expect(subject.keys).to match_array %w[searches about_pages feature_pages home_page contacts]
+      expect(subject.keys).to match_array %w[searches groups about_pages feature_pages home_page contacts]
     end
   end
 end