geo_combine 0.5.1 → 0.6.0

data/spec/helpers.rb

@@ -1,17 +1,17 @@
+# frozen_string_literal: true
+
 require 'json'
 
 module Helpers
   # From https://gist.github.com/ascendbruce/7070951
   def valid_json?(json)
-    begin
-      JSON.parse(json)
-      return true
-    rescue Exception => e
-      return false
-    end
+    JSON.parse(json)
+    true
+  rescue Exception => e
+    false
   end
 
   def trim(text)
-    %r/\A\s+#{text}\s+\Z/
+    /\A\s+#{text}\s+\Z/
   end
 end

data/spec/lib/geo_combine/bounding_box_spec.rb

@@ -1,58 +1,76 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GeoCombine::BoundingBox do
   subject(:valid) do
     described_class.new(west: -180, east: 180, north: 90, south: -90)
   end
+
   describe '#to_envelope' do
     it 'returns a valid envelope syntax' do
       expect(valid.to_envelope).to eq 'ENVELOPE(-180.0, 180.0, 90.0, -90.0)'
     end
   end
+
   describe '#valid?' do
     context 'when valid' do
       it { valid.valid? }
     end
+
     context 'when south > north' do
       subject(:invalid) do
         described_class.new(west: -180, east: 180, north: 33, south: 34)
       end
+
       it { expect { invalid.valid? }.to raise_error GeoCombine::Exceptions::InvalidGeometry }
     end
+
     context 'when west > east' do
       subject(:invalid) do
         described_class.new(west: 10, east: -20, north: 90, south: -90)
       end
+
       it { expect { invalid.valid? }.to raise_error GeoCombine::Exceptions::InvalidGeometry }
     end
+
     context 'when west out of range' do
       subject(:invalid) do
         described_class.new(west: -181, east: 180, north: 90, south: -90)
       end
+
       it { expect { invalid.valid? }.to raise_error GeoCombine::Exceptions::InvalidGeometry }
     end
+
     context 'when east out of range' do
       subject(:invalid) do
         described_class.new(west: -180, east: 181, north: 90, south: -90)
       end
+
       it { expect { invalid.valid? }.to raise_error GeoCombine::Exceptions::InvalidGeometry }
     end
+
     context 'when north out of range' do
       subject(:invalid) do
         described_class.new(west: -180, east: 180, north: 91, south: -90)
       end
+
       it { expect { invalid.valid? }.to raise_error GeoCombine::Exceptions::InvalidGeometry }
     end
+
     context 'when south out of range' do
       subject(:invalid) do
         described_class.new(west: -180, east: 180, north: 90, south: -91)
       end
+
       it { expect { invalid.valid? }.to raise_error GeoCombine::Exceptions::InvalidGeometry }
     end
   end
+
   describe '.from_envelope' do
     it { expect(described_class.from_envelope(valid.to_envelope).to_envelope).to eq valid.to_envelope }
   end
+
   describe '.from_string_delimiter' do
     it { expect(described_class.from_string_delimiter('-180, -90, 180, 90').to_envelope).to eq valid.to_envelope }
   end

data/spec/lib/geo_combine/ckan_metadata_spec.rb

@@ -1,63 +1,79 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GeoCombine::CkanMetadata do
   include JsonDocs
-  let(:ckan_sample) { GeoCombine::CkanMetadata.new(ckan_metadata) }
+  let(:ckan_sample) { described_class.new(ckan_metadata) }
+
   describe '#to_geoblacklight' do
     it 'returns a GeoCombine::Geoblacklight' do
       expect(ckan_sample.to_geoblacklight).to be_an GeoCombine::Geoblacklight
     end
   end
+
   describe '#geoblacklight_terms' do
     describe 'builds a hash which maps metadata' do
       let(:metadata) { ckan_sample.instance_variable_get(:@metadata) }
+
       it 'with dc_identifier_s' do
         expect(ckan_sample.geoblacklight_terms).to include(dc_identifier_s: metadata['id'])
       end
+
       it 'with dc_title_s' do
         expect(ckan_sample.geoblacklight_terms).to include(dc_title_s: metadata['title'])
       end
+
       it 'with dc_rights_s' do
         expect(ckan_sample.geoblacklight_terms).to include(dc_rights_s: 'Public')
       end
+
       it 'with dct_provenance_s' do
         expect(ckan_sample.geoblacklight_terms).to include(dct_provenance_s: metadata['organization']['title'])
       end
+
       it 'with layer_slug_s' do
         expect(ckan_sample.geoblacklight_terms).to include(layer_slug_s: metadata['name'])
       end
+
       it 'with solr_geom' do
         expect(ckan_sample.geoblacklight_terms).to include(solr_geom: 'ENVELOPE(-158.2, -105.7, 59.2, 8.9)')
       end
+
       it 'with dc_subject_sm' do
         expect(ckan_sample.geoblacklight_terms[:dc_subject_sm].length).to eq 63
       end
+
       context 'with no information resources' do
         let(:ckan_sample) do
-          ckan = GeoCombine::CkanMetadata.new(ckan_metadata)
+          ckan = described_class.new(ckan_metadata)
           metadata = ckan.instance_variable_get('@metadata')
           metadata['resources'].delete_if { |resource| resource['resource_locator_function'] == 'information' }
           ckan
         end
+
         it 'has no url (home page) in dct_references_s' do
           expect(JSON.parse(ckan_sample.geoblacklight_terms[:dct_references_s])).not_to include('http://schema.org/url')
         end
       end
+
       context 'with no download resources' do
         let(:ckan_sample) do
-          ckan = GeoCombine::CkanMetadata.new(ckan_metadata)
+          ckan = described_class.new(ckan_metadata)
           metadata = ckan.instance_variable_get('@metadata')
           metadata['resources'].delete_if { |resource| resource['resource_locator_function'] == 'download' }
           ckan
         end
+
         it 'has no downloadUrl in dct_references_s' do
-          expect(ckan_sample.downloadable?).to be_falsey
+          expect(ckan_sample).not_to be_downloadable
           expect(JSON.parse(ckan_sample.geoblacklight_terms[:dct_references_s])).not_to include('http://schema.org/downloadUrl')
         end
       end
+
       context 'with a ZIP download' do
         let(:ckan_sample) do
-          ckan = GeoCombine::CkanMetadata.new(ckan_metadata)
+          ckan = described_class.new(ckan_metadata)
           metadata = ckan.instance_variable_get('@metadata')
           metadata['resources'] = [
             {
@@ -68,43 +84,50 @@ RSpec.describe GeoCombine::CkanMetadata do
           ]
           ckan
         end
+
         it 'has a format and a download URL' do
-          expect(ckan_sample.downloadable?).to be_truthy
+          expect(ckan_sample).to be_downloadable
           expect(ckan_sample.geoblacklight_terms).to include(dc_format_s: 'ZIP')
           expect(JSON.parse(ckan_sample.geoblacklight_terms[:dct_references_s])).to include('http://schema.org/downloadUrl' => 'https://example.com/layer.zip')
         end
       end
+
       context 'without any resources' do
         let(:ckan_sample) do
-          ckan = GeoCombine::CkanMetadata.new(ckan_metadata)
+          ckan = described_class.new(ckan_metadata)
           metadata = ckan.instance_variable_get('@metadata')
           metadata.delete('resources')
           ckan
         end
+
         it 'has no urls in dct_references_s' do
-          expect(ckan_sample.downloadable?).to be_falsey
+          expect(ckan_sample).not_to be_downloadable
           expect(JSON.parse(ckan_sample.geoblacklight_terms[:dct_references_s])).not_to include('http://schema.org/url')
           expect(JSON.parse(ckan_sample.geoblacklight_terms[:dct_references_s])).not_to include('http://schema.org/downloadUrl')
         end
       end
+
       context 'with very long descriptions' do
         let(:ckan_sample) do
-          ckan = GeoCombine::CkanMetadata.new(ckan_metadata)
+          ckan = described_class.new(ckan_metadata)
           metadata = ckan.instance_variable_get('@metadata')
-          metadata['notes'] = 'x' * 50000
+          metadata['notes'] = 'x' * 50_000
           ckan
         end
+
         it 'restricts the size' do
           expect(ckan_sample.geoblacklight_terms[:dc_description_s].length).to eq GeoCombine::CkanMetadata::MAX_STRING_LENGTH + 1
         end
       end
+
       context 'with no descriptions' do
         let(:ckan_sample) do
-          ckan = GeoCombine::CkanMetadata.new(ckan_metadata)
+          ckan = described_class.new(ckan_metadata)
           metadata = ckan.instance_variable_get('@metadata')
           metadata['notes'] = nil
           ckan
         end
+
         it 'omits the description' do
           expect(ckan_sample.geoblacklight_terms).not_to include(:dc_description_s)
         end

data/spec/lib/geo_combine/esri_open_data_spec.rb

@@ -1,83 +1,104 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GeoCombine::EsriOpenData do
   include JsonDocs
-  let(:esri_sample) { GeoCombine::EsriOpenData.new(esri_opendata_metadata) }
+  let(:esri_sample) { described_class.new(esri_opendata_metadata) }
   let(:metadata) { esri_sample.instance_variable_get(:@metadata) }
+
   describe '#initialize' do
     it 'parses metadata argument JSON to Hash' do
       expect(esri_sample.instance_variable_get(:@metadata)).to be_an Hash
     end
+
     it 'puts extend data into @geometry' do
       expect(esri_sample.instance_variable_get(:@geometry)).to be_an Array
     end
   end
+
   describe '#to_geoblacklight' do
     it 'calls geoblacklight_terms to create a GeoBlacklight object' do
       expect(esri_sample).to receive(:geoblacklight_terms).and_return({})
       expect(esri_sample.to_geoblacklight).to be_an GeoCombine::Geoblacklight
     end
   end
+
   describe '#geoblacklight_terms' do
     describe 'builds a hash which maps metadata' do
       let(:metadata) { esri_sample.instance_variable_get(:@metadata) }
+
       it 'with dc_identifier_s' do
         expect(esri_sample.geoblacklight_terms).to include(dc_identifier_s: metadata['id'])
       end
+
       it 'with dc_title_s' do
         expect(esri_sample.geoblacklight_terms).to include(dc_title_s: metadata['name'])
       end
+
       it 'with dc_description_s sanitized' do
         expect(esri_sample).to receive(:sanitize_and_remove_lines).and_return 'sanitized'
         expect(esri_sample.geoblacklight_terms).to include(dc_description_s: 'sanitized')
       end
+
       it 'with dc_rights_s' do
         expect(esri_sample.geoblacklight_terms).to include(dc_rights_s: 'Public')
       end
+
       it 'with dct_provenance_s' do
         expect(esri_sample.geoblacklight_terms).to include(dct_provenance_s: metadata['owner'])
       end
+
       it 'with dct_references_s' do
         expect(esri_sample).to receive(:references).and_return ''
         expect(esri_sample.geoblacklight_terms).to include(:dct_references_s)
       end
+
       it 'with layer_id_s that is blank' do
         expect(esri_sample.geoblacklight_terms).to include(layer_id_s: '')
       end
+
       it 'with layer_geom_type_s' do
         expect(esri_sample.geoblacklight_terms).to include(layer_geom_type_s: metadata['geometry_type'])
       end
+
       it 'with layer_modified_dt' do
         expect(esri_sample.geoblacklight_terms).to include(layer_modified_dt: metadata['updated_at'])
       end
+
       it 'with layer_slug_s' do
         expect(esri_sample.geoblacklight_terms).to include(layer_slug_s: metadata['id'])
       end
+
       it 'with solr_geom' do
         expect(esri_sample).to receive(:envelope).and_return ''
         expect(esri_sample.geoblacklight_terms).to include(solr_geom: '')
       end
+
       it 'with dc_subject_sm' do
         expect(esri_sample.geoblacklight_terms).to include(dc_subject_sm: metadata['tags'])
       end
     end
   end
+
   describe '#references' do
     it 'converts references to a JSON string' do
       expect(esri_sample).to receive(:references_hash).and_return({})
       expect(esri_sample.references).to be_an String
     end
   end
+
   describe '#references_hash' do
     it 'builds out references' do
       expect(esri_sample.references_hash).to include('http://schema.org/url' => metadata['landing_page'])
       expect(esri_sample.references_hash).to include('http://resources.arcgis.com/en/help/arcgis-rest-api' => metadata['url'])
     end
   end
+
   describe '#envelope' do
     it 'creates an envelope for use in Solr' do
       expect(esri_sample.envelope).to be_an String
-      expect(esri_sample.envelope).to match /ENVELOPE\(.+,.+,.+,.+\)/
+      expect(esri_sample.envelope).to match(/ENVELOPE\(.+,.+,.+,.+\)/)
     end
   end
 end

data/spec/lib/geo_combine/fgdc_spec.rb

@@ -1,103 +1,134 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GeoCombine::Fgdc do
   include XmlDocs
-  let(:fgdc_object){ GeoCombine::Fgdc.new(tufts_fgdc) }
+  let(:fgdc_object) { described_class.new(tufts_fgdc) }
+
   describe '#initialize' do
     it 'returns an instantiated Fgdc object' do
-      expect(fgdc_object).to be_an GeoCombine::Fgdc
+      expect(fgdc_object).to be_an described_class
     end
   end
+
   describe '#xsl_geoblacklight' do
-    it 'should be defined' do
+    it 'is defined' do
       expect(fgdc_object.xsl_geoblacklight).to be_an Nokogiri::XSLT::Stylesheet
     end
   end
-  describe '#xsl_html'do
-    it 'should be defined' do
+
+  describe '#xsl_html' do
+    it 'is defined' do
       expect(fgdc_object.xsl_html).to be_an Nokogiri::XSLT::Stylesheet
     end
   end
+
   describe '#to_geoblacklight' do
     let(:fgdc_geobl) { fgdc_object.to_geoblacklight }
+
     it 'returns a GeoCombine::Geoblacklight object' do
       expect(fgdc_geobl).to be_an GeoCombine::Geoblacklight
     end
+
     it 'is not valid due to bad modification date but valid otherwise' do
       expect { fgdc_geobl.valid? }.to raise_error(JSON::Schema::ValidationError, /layer_modified_dt/)
       fgdc_geobl.metadata.delete 'layer_modified_dt'
-      expect(fgdc_geobl.valid?).to be_truthy
+      expect(fgdc_geobl).to be_valid
     end
+
     describe 'with GeoBlacklight-Schema fields' do
       it 'geoblacklight_version' do
         expect(fgdc_geobl.metadata['geoblacklight_version']).to eq '1.0'
       end
+
       it 'dc_identifier_s' do
         expect(fgdc_geobl.metadata['dc_identifier_s']).to eq 'http://www.geoportaligm.gob.ec/portal/'
       end
+
       it 'dc_title_s' do
         expect(fgdc_geobl.metadata['dc_title_s']).to eq 'Drilling Towers 50k Scale Ecuador 2011'
       end
+
       it 'dc_description_s' do
-        expect(fgdc_geobl.metadata['dc_description_s']).to match /Ecuador created from/
+        expect(fgdc_geobl.metadata['dc_description_s']).to match(/Ecuador created from/)
       end
+
       it 'dc_rights_s' do
         expect(fgdc_geobl.metadata['dc_rights_s']).to eq 'Public'
       end
+
       it 'dct_provenance_s' do
         expect(fgdc_geobl.metadata['dct_provenance_s']).to eq 'Tufts'
       end
+
       it 'layer_id_s' do
         expect(fgdc_geobl.metadata['layer_id_s']).to eq 'urn:Ecuador50KDrillingTower11'
       end
+
       it 'layer_slug_s' do
         expect(fgdc_geobl.metadata['layer_slug_s']).to eq 'Tufts-Ecuador50KDrillingTower11'
       end
+
       it 'layer_modified_dt' do
         expect(fgdc_geobl.metadata['layer_modified_dt']).to eq '2013-08-13'
       end
+
       it 'dc_creator_sm' do
         expect(fgdc_geobl.metadata['dc_creator_sm']).to be_an Array
         expect(fgdc_geobl.metadata['dc_creator_sm']).to include 'Instituto Geografico Militar (Ecuador)'
       end
+
       it 'dc_format_s' do
         expect(fgdc_geobl.metadata['dc_format_s']).to eq 'Shapefile'
       end
+
       it 'dc_language_s' do
         expect(fgdc_geobl.metadata['dc_language_s']).to eq 'English'
       end
+
       it 'dc_type_s' do
         expect(fgdc_geobl.metadata['dc_type_s']).to eq 'Dataset'
       end
+
       it 'dc_subject_sm' do
         expect(fgdc_geobl.metadata['dc_subject_sm']).to be_an Array
-        expect(fgdc_geobl.metadata['dc_subject_sm']).to include 'point', 'structure', 'economy', 'Drilling platforms', 'Oil well drilling'
+        expect(fgdc_geobl.metadata['dc_subject_sm']).to include 'point', 'structure', 'economy', 'Drilling platforms',
+                                                                'Oil well drilling'
       end
+
       it 'dc_spatial_sm' do
         expect(fgdc_geobl.metadata['dc_spatial_sm']).to be_an Array
-        expect(fgdc_geobl.metadata['dc_spatial_sm']).to include 'Ecuador', 'República del Ecuador', 'Northern Hemisphere', 'Southern Hemisphere', 'Western Hemisphere', 'South America'
+        expect(fgdc_geobl.metadata['dc_spatial_sm']).to include 'Ecuador', 'República del Ecuador',
+                                                                'Northern Hemisphere', 'Southern Hemisphere', 'Western Hemisphere', 'South America'
       end
+
       it 'dct_issued_s' do
         expect(fgdc_geobl.metadata['dct_issued_s']).to eq '2011'
       end
+
       it 'dct_temporal_sm' do
         expect(fgdc_geobl.metadata['dct_temporal_sm']).to be_an Array
         expect(fgdc_geobl.metadata['dct_temporal_sm']).to include '2011'
       end
+
       it 'dct_isPartOf_sm' do
         expect(fgdc_geobl.metadata['dct_isPartOf_sm']).to be_an Array
         expect(fgdc_geobl.metadata['dct_isPartOf_sm']).to include 'Ecuador', 'Instituto Geografico Militar Data'
       end
+
       it 'solr_geom' do
         expect(fgdc_geobl.metadata['solr_geom']).to eq 'ENVELOPE(-79.904768, -79.904768, -1.377743, -1.377743)'
       end
+
       it 'solr_year_i' do
         expect(fgdc_geobl.metadata['solr_year_i']).to eq 2011
       end
     end
   end
+
   describe '#to_html' do
-    it 'should create a transformation of the metadata as a String' do
+    it 'creates a transformation of the metadata as a String' do
       expect(fgdc_object.to_html).to be_an String
     end
   end

data/spec/lib/geo_combine/formatting_spec.rb

@@ -1,26 +1,34 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GeoCombine::Formatting do
+  subject { Object.new.extend(described_class) }
+
   let(:dirty) { "<p>paragraph</p> \n on a new line" }
-  subject { Object.new.extend(GeoCombine::Formatting) }
+
   describe '#sanitize' do
     it 'sanitizes a fragment' do
-      expect(subject.sanitize(dirty)).to_not match('<p>')
+      expect(subject.sanitize(dirty)).not_to match('<p>')
     end
   end
+
   describe '#remove_lines' do
     it 'removes new lines' do
-      expect(subject.remove_lines(dirty)).to_not match(/\n/)
+      expect(subject.remove_lines(dirty)).not_to match(/\n/)
     end
   end
+
   describe '#sanitize_and_remove_lines' do
     it 'returns a corrected string' do
-      expect(subject.sanitize_and_remove_lines(dirty)).to_not match('<p>')
-      expect(subject.sanitize_and_remove_lines(dirty)).to_not match(/\n/)
+      expect(subject.sanitize_and_remove_lines(dirty)).not_to match('<p>')
+      expect(subject.sanitize_and_remove_lines(dirty)).not_to match(/\n/)
     end
   end
+
   describe '#sluggify' do
     let(:preslug) { 'HARVARD...Co_0l' }
+
     it 'handles multiple . and _ and uppercase' do
       expect(subject.sluggify(preslug)).to eq 'harvard-co-0l'
     end

data/spec/lib/geo_combine/geo_blacklight_harvester_spec.rb

@@ -1,23 +1,25 @@
 # frozen_string_literal: true
 
+require 'net/http'
 require 'spec_helper'
 require 'rsolr'
 
 RSpec.describe GeoCombine::GeoBlacklightHarvester do
+  subject(:harvester) { described_class.new(site_key) }
+
   let(:site_key) { :INSTITUTION }
   let(:stub_json_response) { '{}' }
   let(:stub_solr_connection) { double('RSolr::Connection') }
-  subject(:harvester) { described_class.new(site_key) }
 
   before do
     allow(described_class).to receive(:config).and_return({
-      INSTITUTION: {
-        host: 'https://example.com/',
-        params: {
-          f: { dct_provenance_s: ['INSTITUTION'] }
-        }
-      }
-    })
+                                                            INSTITUTION: {
+                                                              host: 'https://example.com/',
+                                                              params: {
+                                                                f: { dct_provenance_s: ['INSTITUTION'] }
+                                                              }
+                                                            }
+                                                          })
   end
 
   describe 'initialization' do
@@ -49,15 +51,16 @@ RSpec.describe GeoCombine::GeoBlacklightHarvester do
     describe 'document tranformations' do
       let(:docs) do
         [
-         { layer_slug_s: 'abc-123', _version_: '1', timestamp: '1999-12-31', score: 0.1 },
-         { layer_slug_s: 'abc-321', dc_source_s: 'abc-123' }
+          { layer_slug_s: 'abc-123', _version_: '1', timestamp: '1999-12-31', score: 0.1,
+            solr_bboxtype__minX: -87.324704, solr_bboxtype__minY: 40.233691, solr_bboxtype__maxX: -87.174404, solr_bboxtype__maxY: 40.310695 },
+          { layer_slug_s: 'abc-321', dc_source_s: 'abc-123' }
         ]
       end
 
       context 'when a tranformer is set' do
         before do
           expect(described_class).to receive(:document_transformer).at_least(:once).and_return(
-            ->(doc) {
+            lambda { |doc|
               doc.delete('_version_')
               doc
             }
@@ -68,7 +71,8 @@ RSpec.describe GeoCombine::GeoBlacklightHarvester do
           expect(stub_solr_connection).to receive(:update).with(
             hash_including(
               data: [
-                { layer_slug_s: 'abc-123', timestamp: '1999-12-31', score: 0.1 },
+                { layer_slug_s: 'abc-123', timestamp: '1999-12-31', score: 0.1, solr_bboxtype__minX: -87.324704,
+                  solr_bboxtype__minY: 40.233691, solr_bboxtype__maxX: -87.174404, solr_bboxtype__maxY: 40.310695 },
                 { layer_slug_s: 'abc-321', dc_source_s: 'abc-123' }
               ].to_json
             )
@@ -79,7 +83,7 @@ RSpec.describe GeoCombine::GeoBlacklightHarvester do
       end
 
       context 'when no transformer is set' do
-        it 'removes the _version_, timestamp, and score fields' do
+        it 'removes the _version_, timestamp, score, and solr_bboxtype__* fields' do
           expect(stub_solr_connection).to receive(:update).with(
             hash_including(
               data: [
@@ -125,7 +129,7 @@ RSpec.describe GeoCombine::GeoBlacklightHarvester do
     let(:first_docs) {  [{ 'layer_slug_s' => 'abc-123' }, { 'layer_slug_s' => 'abc-321' }] }
     let(:second_docs) { [{ 'layer_slug_s' => 'xyz-123' }, { 'layer_slug_s' => 'xyz-321' }] }
     let(:stub_first_response) do
-      { 'response' => { 'docs' => first_docs, 'pages' =>  { 'current_page' => 1, 'total_pages' => 2 } } }
+      { 'response' => { 'docs' => first_docs, 'pages' => { 'current_page' => 1, 'total_pages' => 2 } } }
     end
     let(:stub_second_response) do
       { 'response' => { 'docs' => second_docs, 'pages' => { 'current_page' => 2, 'total_pages' => 2 } } }
@@ -137,7 +141,8 @@ RSpec.describe GeoCombine::GeoBlacklightHarvester do
           URI('https://example.com?f%5Bdct_provenance_s%5D%5B%5D=INSTITUTION&format=json&per_page=100&page=2')
         ).and_return(stub_second_response.to_json)
         base_url = 'https://example.com?f%5Bdct_provenance_s%5D%5B%5D=INSTITUTION&format=json&per_page=100'
-        docs = described_class::LegacyBlacklightResponse.new(response: stub_first_response, base_url: base_url).documents
+        docs = described_class::LegacyBlacklightResponse.new(response: stub_first_response,
+                                                             base_url: base_url).documents
 
         expect(docs.to_a).to eq([first_docs, second_docs])
       end
@@ -154,19 +159,17 @@ RSpec.describe GeoCombine::GeoBlacklightHarvester do
 
     let(:first_results_response) do
       { 'data' => [
-          { 'links' => { 'self' => 'https://example.com/catalog/abc-123' } },
-          { 'links' => { 'self' => 'https://example.com/catalog/abc-321' } }
-        ],
-        'links' => { 'next' => 'https://example.com/catalog.json?f%5Bdct_provenance_s%5D%5B%5D=INSTITUTION&per_page=100&page=2' }
-      }
+        { 'links' => { 'self' => 'https://example.com/catalog/abc-123' } },
+        { 'links' => { 'self' => 'https://example.com/catalog/abc-321' } }
+      ],
+        'links' => { 'next' => 'https://example.com/catalog.json?f%5Bdct_provenance_s%5D%5B%5D=INSTITUTION&per_page=100&page=2' } }
     end
 
     let(:second_results_response) do
       { 'data' => [
-          { 'links' => { 'self' => 'https://example.com/catalog/xyz-123' } },
-          { 'links' => { 'self' => 'https://example.com/catalog/xyz-321' } }
-        ]
-      }
+        { 'links' => { 'self' => 'https://example.com/catalog/xyz-123' } },
+        { 'links' => { 'self' => 'https://example.com/catalog/xyz-321' } }
+      ] }
     end
 
     describe '#documents' do
@@ -178,12 +181,13 @@ RSpec.describe GeoCombine::GeoBlacklightHarvester do
         end
 
         base_url = 'https://example.com?f%5Bdct_provenance_s%5D%5B%5D=INSTITUTION&format=json&per_page=100'
-        docs = described_class::ModernBlacklightResponse.new(response: first_results_response, base_url: base_url).documents
+        docs = described_class::ModernBlacklightResponse.new(response: first_results_response,
+                                                             base_url: base_url).documents
 
         expect(docs.to_a).to eq([
-          [{ 'layer_slug_s' => 'abc-123' }, { 'layer_slug_s' => 'abc-321' }],
-          [{ 'layer_slug_s' => 'xyz-123' }, { 'layer_slug_s' => 'xyz-321' }],
-        ])
+                                  [{ 'layer_slug_s' => 'abc-123' }, { 'layer_slug_s' => 'abc-321' }],
+                                  [{ 'layer_slug_s' => 'xyz-123' }, { 'layer_slug_s' => 'xyz-321' }]
+                                ])
       end
     end
   end