blacklight-spotlight 3.0.2 → 3.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2ef4a4bcb8283379d4091d374feb6d19075868ab8511a6f0727cefea71dda5ab
-  data.tar.gz: cb49bd3be13930c34cbab5320070659a0301f53936b6e0eaf68b0691d0f74387
+  metadata.gz: 931e772ae7165aea85c6664c6c1fcbb615d0397d42d2000b2bc4f7038288ff91
+  data.tar.gz: 5eac236663178ff5476f8b58213f8da5757bd7120108b801792247cc194392b4
 SHA512:
-  metadata.gz: 0252bbc23f096a58da23ecd47d8b17e1bbfa96dd2ab78e3e677f81d97af948e8ca9665ecd8344ecc18170e13dbd496d2da4d11e928a6f40f6e91e723d711e504
-  data.tar.gz: 793bbd141a4fe2fb01151b8a3eb0366cf8d32f55bf2dacb8dc011e0a743e0fba7cbc3e63ac9268106ed06b41ba059a25536de62d655c3af6f29c8e87a3655c87
+  metadata.gz: 528286f83609189237af8feef3e40048357b37de3770fdf772bc2672c3b0a52230cb1ab9f2ddfb31b59932aba9a0fafdc44ef2d2f79a333db9618ccf34746acf
+  data.tar.gz: 16900b7cbc3278b6be875cb0e91441f33860dc13200c36898e36f545d236557eddada90fe99536ed2fecd72538c076f6dd0edadeea9e481b5d7d025b470b93b6

data/app/jobs/spotlight/process_bulk_updates_csv_job.rb

@@ -43,7 +43,7 @@ module Spotlight
       needs_reindex = false
 
       begin
-        if sidecar.public != to_bool(row[config.csv_visibility])
+        if row.headers.include?(config.csv_visibility) && sidecar.public != to_bool(row[config.csv_visibility])
           sidecar.update(public: to_bool(row[config.csv_visibility]))
           needs_reindex = true
         end

data/app/models/spotlight/search.rb

@@ -55,7 +55,9 @@ module Spotlight
     end
 
     def count
-      documents.size
+      @count ||= Rails.cache.fetch([exhibit, self, 'count']) do
+        documents.size
+      end
     end
 
     delegate :blacklight_config, to: :exhibit

data/lib/spotlight/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Spotlight
-  VERSION = '3.0.2'
+  VERSION = '3.0.3'
 end

data/spec/examples.txt

@@ -650,7 +650,8 @@ example_id
 ./spec/jobs/spotlight/change_visibility_job_spec.rb[1:1]                                        | passed  | 2.86 seconds    |
 ./spec/jobs/spotlight/default_thumbnail_job_spec.rb[1:1]                                        | passed  | 0.00146 seconds |
 ./spec/jobs/spotlight/process_bulk_updates_csv_job_spec.rb[1:1:1]                               | passed  | 0.2189 seconds  |
-./spec/jobs/spotlight/process_bulk_updates_csv_job_spec.rb[1:1:2:1]                             | passed  | 0.08658 seconds |
+./spec/jobs/spotlight/process_bulk_updates_csv_job_spec.rb[1:1:2:1]                             | failed  | 0.62594 seconds |
+./spec/jobs/spotlight/process_bulk_updates_csv_job_spec.rb[1:1:3:1]                             | passed  | 0.57342 seconds |
 ./spec/jobs/spotlight/process_bulk_updates_csv_job_spec.rb[1:2:1]                               | passed  | 0.40446 seconds |
 ./spec/jobs/spotlight/process_bulk_updates_csv_job_spec.rb[1:2:2:1]                             | passed  | 0.49212 seconds |
 ./spec/jobs/spotlight/reindex_exhibit_job_spec.rb[1:1:1]                                        | passed  | 0.05834 seconds |

data/spec/factories/bulk_updates.rb

@@ -7,6 +7,12 @@ FactoryBot.define do
     exhibit
   end
 
+  factory :bulk_update_no_cols, class: 'Spotlight::BulkUpdate' do
+    file { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'updated-bulk-update-template-no-cols.csv'), __dir__)) }
+
+    exhibit
+  end
+
   factory :tagged_bulk_update, class: 'Spotlight::BulkUpdate' do
     file { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'updated-bulk-update-template-w-tags.csv'), __dir__)) }
 

data/spec/features/javascript/blocks/search_result_block_spec.rb

@@ -42,6 +42,6 @@ describe 'Search Result Block', type: :feature, js: true do
     end
 
     # Documents should exist
-    expect(page).to have_css('.documents .document')
+    expect(page).to have_css('.documents-gallery .document')
   end
 end

data/spec/fixtures/updated-bulk-update-template-no-cols.csv

@@ -0,0 +1,4 @@
+Item ID,Item Title
+bm387cy2596,L'AMERIQVE
+cz507zk0531,NEW MEXICO
+dq287tq6352,L'AMERIQUE

data/spec/jobs/spotlight/process_bulk_updates_csv_job_spec.rb

@@ -36,6 +36,14 @@ describe Spotlight::ProcessBulkUpdatesCsvJob do
         expect(SolrDocument.index.connection).not_to have_received(:update)
       end
     end
+
+    context 'without a visibility column' do
+      let(:bulk_update) { FactoryBot.create(:bulk_update_no_cols, exhibit: exhibit) }
+
+      it 'does nothing with the data' do
+        expect { subject.perform_now }.not_to(change { exhibit.reload.solr_document_sidecars.where(public: false).count })
+      end
+    end
   end
 
   describe 'tags' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: blacklight-spotlight
 version: !ruby/object:Gem::Version
-  version: 3.0.2
+  version: 3.0.3
 platform: ruby
 authors:
 - Chris Beer
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-14 00:00:00.000000000 Z
+date: 2021-05-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activejob-status
@@ -1609,6 +1609,7 @@ files:
 - spec/fixtures/iiif_responses.rb
 - spec/fixtures/json-upload-fixture.json
 - spec/fixtures/sample_solr_documents.yml
+- spec/fixtures/updated-bulk-update-template-no-cols.csv
 - spec/fixtures/updated-bulk-update-template-w-tags.csv
 - spec/fixtures/updated-bulk-update-template.csv
 - spec/helpers/spotlight/application_helper_spec.rb
@@ -1968,6 +1969,7 @@ test_files:
 - spec/fixtures/iiif_responses.rb
 - spec/fixtures/json-upload-fixture.json
 - spec/fixtures/sample_solr_documents.yml
+- spec/fixtures/updated-bulk-update-template-no-cols.csv
 - spec/fixtures/updated-bulk-update-template-w-tags.csv
 - spec/fixtures/updated-bulk-update-template.csv
 - spec/helpers/spotlight/application_helper_spec.rb