geoblacklight 4.0.0.pre.rc3 → 4.1.0

data/spec/features/linkified_attribute_table_spec.rb

@@ -1,12 +1,13 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Attribute table values', js: true do
-  xscenario 'are linkified' do
+require "spec_helper"
+
+feature "Attribute table values", js: true do
+  xscenario "are linkified" do
     # ArcGIS server returning an error. Wait until SVG elements are added
-    visit solr_document_path 'f406332e63eb4478a9560ad86ae90327_18'
-    expect(page).to have_css '.leaflet-overlay-pane svg'
-    page.first('svg g path').click
-    expect(page).to have_css 'td a[href="http://www.minneapolismn.gov/fire/stations/fire_station28"]', text: 'http://www.minneapolismn.gov/fire/stations/fire_station28'
+    visit solr_document_path "f406332e63eb4478a9560ad86ae90327_18"
+    expect(page).to have_css ".leaflet-overlay-pane svg"
+    page.first("svg g path").click
+    expect(page).to have_css 'td a[href="http://www.minneapolismn.gov/fire/stations/fire_station28"]', text: "http://www.minneapolismn.gov/fire/stations/fire_station28"
   end
 end

data/spec/features/metadata_panel_spec.rb

@@ -1,11 +1,12 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Metadata tools' do
-  feature 'when metadata references are available', js: :true do
-    let(:iso19139) { File.read(Rails.root.join('..', 'spec', 'fixtures', 'iso19139', 'stanford-cg357zz0321.xml')) }
-    let(:fgdc) { File.read(Rails.root.join('..', 'spec', 'fixtures', 'fgdc', 'harvard-g7064-s2-1834-k3.xml')) }
-    let(:mods) { File.read(Rails.root.join('..', 'spec', 'fixtures', 'mods', 'stanford-cg357zz0321.mods')) }
+require "spec_helper"
+
+feature "Metadata tools" do
+  feature "when metadata references are available", js: true do
+    let(:iso19139) { File.read(Rails.root.join("..", "spec", "fixtures", "iso19139", "stanford-cg357zz0321.xml")) }
+    let(:fgdc) { File.read(Rails.root.join("..", "spec", "fixtures", "fgdc", "harvard-g7064-s2-1834-k3.xml")) }
+    let(:mods) { File.read(Rails.root.join("..", "spec", "fixtures", "mods", "stanford-cg357zz0321.mods")) }
     let(:fgdc_response) { instance_double(Faraday::Response) }
     let(:fgdc_connection) { instance_double(Faraday::Connection) }
     let(:iso19139_response) { instance_double(Faraday::Response) }
@@ -17,35 +18,35 @@ feature 'Metadata tools' do
       allow(iso19139_response).to receive(:body).and_return(iso19139)
       allow(iso19139_response).to receive(:status).and_return(200)
       allow(iso19139_connection).to receive(:get).and_return(iso19139_response)
-      allow(Faraday).to receive(:new).with(url: 'https://raw.githubusercontent.com/OpenGeoMetadata/edu.stanford.purl/master/cg/357/zz/0321/iso19139.xml').and_return(iso19139_connection)
+      allow(Faraday).to receive(:new).with(url: "https://raw.githubusercontent.com/OpenGeoMetadata/edu.stanford.purl/master/cg/357/zz/0321/iso19139.xml").and_return(iso19139_connection)
 
       allow(fgdc_response).to receive(:body).and_return(fgdc)
       allow(fgdc_response).to receive(:status).and_return(200)
       allow(fgdc_connection).to receive(:get).and_return(fgdc_response)
-      allow(Faraday).to receive(:new).with(url: 'https://raw.githubusercontent.com/OpenGeoMetadata/edu.harvard/master/217/121/227/77/fgdc.xml').and_return(fgdc_connection)
+      allow(Faraday).to receive(:new).with(url: "https://raw.githubusercontent.com/OpenGeoMetadata/edu.harvard/master/217/121/227/77/fgdc.xml").and_return(fgdc_connection)
 
       allow(response).to receive(:body).and_return(mods)
       allow(response).to receive(:status).and_return(200)
       allow(connection).to receive(:get).and_return(response)
-      allow(Faraday).to receive(:new).with(url: 'http://purl.stanford.edu/cg357zz0321.mods').and_return(connection)
+      allow(Faraday).to receive(:new).with(url: "http://purl.stanford.edu/cg357zz0321.mods").and_return(connection)
 
       allow(Faraday).to receive(:new).with(hash_including(url: %r{solr/blacklight-core})).and_call_original
     end
 
-    scenario 'shows up as HTML' do
-      visit solr_document_path 'harvard-g7064-s2-1834-k3'
-      expect(page).to have_css 'li.metadata a', text: 'Metadata'
-      click_link 'Metadata'
+    scenario "shows up as HTML" do
+      visit solr_document_path "harvard-g7064-s2-1834-k3"
+      expect(page).to have_css "li.metadata a", text: "Metadata"
+      click_link "Metadata"
       using_wait_time 15 do
-        within '.metadata-view' do
-          expect(page).to have_css '.pill-metadata', text: 'FGDC'
-          expect(page).to have_css 'dt', text: 'Identification Information'
-          expect(page).to have_css 'dt', text: 'Metadata Reference Information'
+        within ".metadata-view" do
+          expect(page).to have_css ".pill-metadata", text: "FGDC"
+          expect(page).to have_css "dt", text: "Identification Information"
+          expect(page).to have_css "dt", text: "Metadata Reference Information"
         end
       end
     end
-    context 'when the metadata is in the XML' do
-      let(:mods) { File.read(Rails.root.join('..', 'spec', 'fixtures', 'mods', 'fb897vt9938.mods')) }
+    context "when the metadata is in the XML" do
+      let(:mods) { File.read(Rails.root.join("..", "spec", "fixtures", "mods", "fb897vt9938.mods")) }
       let(:response) { instance_double(Faraday::Response) }
       let(:connection) { instance_double(Faraday::Connection) }
 
@@ -53,27 +54,27 @@ feature 'Metadata tools' do
         allow(response).to receive(:body).and_return(mods)
         allow(response).to receive(:status).and_return(200)
         allow(connection).to receive(:get).and_return(response)
-        allow(Faraday).to receive(:new).with(url: 'http://purl.stanford.edu/fb897vt9938.mods').and_return(connection)
+        allow(Faraday).to receive(:new).with(url: "http://purl.stanford.edu/fb897vt9938.mods").and_return(connection)
         allow(Faraday).to receive(:new).and_call_original
       end
 
-      scenario 'shows up as XML' do
-        visit solr_document_path 'stanford-cg357zz0321'
-        expect(page).to have_css 'li.metadata a', text: 'Metadata'
-        click_link 'Metadata'
+      scenario "shows up as XML" do
+        visit solr_document_path "stanford-cg357zz0321"
+        expect(page).to have_css "li.metadata a", text: "Metadata"
+        click_link "Metadata"
         using_wait_time 15 do
-          within '.metadata-view' do
-            expect(page).to have_css '.pill-metadata', text: 'MODS'
-            expect(page).to have_css '.CodeRay'
+          within ".metadata-view" do
+            expect(page).to have_css ".pill-metadata", text: "MODS"
+            expect(page).to have_css ".CodeRay"
           end
         end
       end
     end
   end
-  feature 'when metadata references are not available' do
-    scenario 'is not in tools' do
-      visit solr_document_path 'mit-f6rqs4ucovjk2'
-      expect(page).not_to have_css 'li.metadata a', text: 'Metadata'
+  feature "when metadata references are not available" do
+    scenario "is not in tools" do
+      visit solr_document_path "mit-f6rqs4ucovjk2"
+      expect(page).not_to have_css "li.metadata a", text: "Metadata"
     end
   end
 end

data/spec/features/missing_metadata_spec.rb

@@ -1,25 +1,26 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Missing metadata', js: true do
-  scenario 'Yields error free results page for no spatial' do
-    visit search_catalog_path(q: 'ASTER Global Emissivity')
-    expect(page).to have_css('#map')
+require "spec_helper"
+
+feature "Missing metadata", js: true do
+  scenario "Yields error free results page for no spatial" do
+    visit search_catalog_path(q: "ASTER Global Emissivity")
+    expect(page).to have_css("#map")
   end
-  scenario 'Yields error free show page for no wxs_identifier or spatial' do
-    visit solr_document_path('aster-global-emissivity-dataset-1-kilometer-v003-ag1kmcad20')
-    expect(page).to have_css('#map')
+  scenario "Yields error free show page for no wxs_identifier or spatial" do
+    visit solr_document_path("aster-global-emissivity-dataset-1-kilometer-v003-ag1kmcad20")
+    expect(page).to have_css("#map")
   end
-  scenario 'Yields error free show page for no layer_geom_type_s' do
-    visit solr_document_path('05d-03-noGeomType')
-    expect(page).to have_css('#map')
+  scenario "Yields error free show page for no layer_geom_type_s" do
+    visit solr_document_path("05d-03-noGeomType")
+    expect(page).to have_css("#map")
   end
-  scenario 'Yields error free show page for no dct_provider_s' do
-    visit solr_document_path('99-0001-noprovider')
-    expect(page).to have_css('#map')
+  scenario "Yields error free show page for no dct_provider_s" do
+    visit solr_document_path("99-0001-noprovider")
+    expect(page).to have_css("#map")
   end
-  scenario 'Yields error free show page for no locn_geometry' do
-    visit solr_document_path('05d-p16022coll246-noGeo')
-    expect(page).to have_css('#map')
+  scenario "Yields error free show page for no locn_geometry" do
+    visit solr_document_path("05d-p16022coll246-noGeo")
+    expect(page).to have_css("#map")
   end
 end

data/spec/features/multiple_downloads_spec.rb

@@ -1,15 +1,16 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Multiple downloads' do
-  feature 'when item has multiple downloads in dct_references_s' do
-    scenario 'downloads are listed in download card' do
-      visit solr_document_path 'cugir-007950'
-      find('#downloads-button').click
-      within '#downloads-collapse' do
-        expect(page).to have_link 'Shapefile', href: 'https://cugir-data.s3.amazonaws.com/00/79/50/cugir-007950.zip'
-        expect(page).to have_link 'PDF', href: 'https://cugir-data.s3.amazonaws.com/00/79/50/agBROO.pdf'
-        expect(page).to have_link 'KMZ', href: 'https://cugir-data.s3.amazonaws.com/00/79/50/agBROO2011.kmz'
+require "spec_helper"
+
+feature "Multiple downloads" do
+  feature "when item has multiple downloads in dct_references_s" do
+    scenario "downloads are listed in download card" do
+      visit solr_document_path "cugir-007950"
+      find("#downloads-button").click
+      within "#downloads-collapse" do
+        expect(page).to have_link "Shapefile", href: "https://cugir-data.s3.amazonaws.com/00/79/50/cugir-007950.zip"
+        expect(page).to have_link "PDF", href: "https://cugir-data.s3.amazonaws.com/00/79/50/agBROO.pdf"
+        expect(page).to have_link "KMZ", href: "https://cugir-data.s3.amazonaws.com/00/79/50/agBROO2011.kmz"
       end
     end
   end

data/spec/features/oembed_spec.rb

@@ -1,10 +1,11 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Oembed layer' do
-  scenario 'Should show oembed content and map' do
-    visit solr_document_path('stanford-dc482zx1528')
-    expect(page).to have_css '#map'
+require "spec_helper"
+
+feature "Oembed layer" do
+  scenario "Should show oembed content and map" do
+    visit solr_document_path("stanford-dc482zx1528")
+    expect(page).to have_css "#map"
     expect(page).to have_css '[data-protocol="Oembed"]'
   end
 end

data/spec/features/relations_spec.rb

@@ -1,44 +1,69 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Display related documents' do
-  scenario 'Record with dct_source_sm value(s) should have parent(s)' do
-    visit relations_solr_document_path('nyu_2451_34502')
-    expect(page).to have_css('.relations .card-header h2', text: 'Source Datasets')
+require "spec_helper"
+
+feature "Display related documents" do
+  scenario "Record with dct_source_sm value(s) should have parent(s)" do
+    visit relations_solr_document_path("nyu_2451_34502")
+    expect(page).to have_css(".relations .card-header h2", text: "Source records...")
   end
 
-  scenario 'Record that is pointed to by others should have children' do
-    visit relations_solr_document_path('nyu_2451_34635')
-    expect(page).to have_css('.relations .card-header h2', text: 'Derived Datasets')
+  scenario "Record that is pointed to by others should have children" do
+    visit relations_solr_document_path("nyu_2451_34635")
+    expect(page).to have_css(".relations .card-header h2", text: "Derived records...")
   end
 
-  scenario 'Relations should respond to json' do
-    visit relations_solr_document_path('nyu_2451_34635', format: 'json')
+  scenario "Relations should respond to json" do
+    visit relations_solr_document_path("nyu_2451_34635", format: "json")
     response = JSON.parse(page.body)
-    expect(response['relations']).not_to respond_to('source_ancestors')
-    expect(response['relations']['source_descendants']['docs'].first[Settings.FIELDS.ID]).to eq 'nyu_2451_34502'
-    expect(response['current_doc']).to eq 'nyu_2451_34635'
+    expect(response["relations"]).not_to respond_to("source_ancestors")
+    expect(response["relations"]["source_descendants"]["docs"].first[Settings.FIELDS.ID]).to eq "nyu_2451_34502"
+    expect(response["current_doc"]).to eq "nyu_2451_34635"
   end
 
-  scenario 'Record with relations should render widget in catalog#show', js: true do
-    visit solr_document_path('nyu_2451_34635')
-    expect(page).to have_css('.card.relations')
-    expect(page).to have_css('div.card-header', text: 'Derived Datasets')
+  scenario "Record with relations should render widget in catalog#show", js: true do
+    visit solr_document_path("nyu_2451_34635")
+    expect(page).to have_css(".card.relations")
+    expect(page).to have_css("div.card-header", text: "Derived records...")
   end
 
-  scenario 'Record without relations should not render widget in catalog#show', js: true do
-    visit solr_document_path('harvard-g7064-s2-1834-k3')
-    expect(page).to have_no_css('.card.relations')
+  scenario "Record without relations should not render widget in catalog#show", js: true do
+    visit solr_document_path("harvard-g7064-s2-1834-k3")
+    expect(page).to have_no_css(".card.relations")
   end
 
-  scenario 'Relationship browse link returns relationship-scoped results', js: true do
+  scenario "Relationship browse link returns relationship-scoped results", js: true do
     # Wabash Topo parent record
-    visit solr_document_path('88cc9b19-3294-4da9-9edd-775c81fb1c59')
+    visit solr_document_path("88cc9b19-3294-4da9-9edd-775c81fb1c59")
+
+    expect(page).to have_content("Has part...")
+    expect(page).to have_link("Browse all 4 records...")
+    click_link("Browse all 4 records...")
 
-    expect(page).to have_content('Has part...')
-    expect(page).to have_link('Browse all 4 records...')
-    click_link('Browse all 4 records...')
+    expect(page).not_to have_content("No results found for your search")
+  end
+
+  scenario "Record with dct_isPartOf_sm value(s) should link to relations", js: true do
+    # All Relationships
+    visit solr_document_path("all-relationships")
+    expect(page).to have_content("Is part of...")
+  end
+
+  scenario "Record pointed at by a parent with dct_isPartOf_sm value(s) should link back", js: true do
+    # The Related Record
+    visit solr_document_path("the-related-record")
+    expect(page).to have_content("Has part...")
+  end
+
+  scenario "Record with pcdm_memberOf_sm value(s) should link to relations", js: true do
+    # All Relationships
+    visit solr_document_path("all-relationships")
+    expect(page).to have_content("Belongs to collection...")
+  end
 
-    expect(page).not_to have_content('No results found for your search')
+  scenario "Record pointed at by a parent with pcdm_memberOf_sm value(s) should link back", js: true do
+    # The Related Record
+    visit solr_document_path("the-related-record")
+    expect(page).to have_content("Collection records...")
   end
 end

data/spec/features/saved_searches_spec.rb

@@ -1,14 +1,15 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'saved searches' do
-  scenario 'list spatial search', js: true do
+require "spec_helper"
+
+feature "saved searches" do
+  scenario "list spatial search", js: true do
     visit root_path
-    within '#map' do
-      find('.search-control a').click
+    within "#map" do
+      find(".search-control a").click
       expect(page.current_url).to match(/bbox=/)
     end
     visit blacklight.search_history_path
-    expect(page).to have_css 'td.query a', text: /#{I18n.t('geoblacklight.bbox_label')}/
+    expect(page).to have_css "td.query a", text: /#{I18n.t('geoblacklight.bbox_label')}/
   end
 end

data/spec/features/search_bar_spec.rb

@@ -1,13 +1,14 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'search bar' do
-  scenario 'present on a spatial search' do
-    visit search_catalog_path(bbox: '25 3 75 35')
-    expect(page).to have_css '.navbar-search'
+require "spec_helper"
+
+feature "search bar" do
+  scenario "present on a spatial search" do
+    visit search_catalog_path(bbox: "25 3 75 35")
+    expect(page).to have_css ".navbar-search"
   end
-  scenario 'present on a text search' do
-    visit search_catalog_path(q: 'test')
-    expect(page).to have_css '.navbar-search'
+  scenario "present on a text search" do
+    visit search_catalog_path(q: "test")
+    expect(page).to have_css ".navbar-search"
   end
 end

data/spec/features/search_results_complex_geometry_spec.rb

@@ -1,24 +1,25 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'spatial search results with an WKT bounding box that crosses the anti-meridian' do
-  scenario 'when searching within the covered area' do
+require "spec_helper"
+
+feature "spatial search results with an WKT bounding box that crosses the anti-meridian" do
+  scenario "when searching within the covered area" do
     # BBox param is in China
     visit search_catalog_path(
-      q: 'A new & accurate map of Asia',
-      bbox: '98.4375 31.765537 117.202148 45.429299'
+      q: "A new & accurate map of Asia",
+      bbox: "98.4375 31.765537 117.202148 45.429299"
     )
-    result_id = page.all('div.documentHeader.row').first['data-layer-id']
-    expect(result_id).to eq '57f0f116-b64e-4773-8684-96ba09afb549'
+    result_id = page.all("div.documentHeader.row").first["data-layer-id"]
+    expect(result_id).to eq "57f0f116-b64e-4773-8684-96ba09afb549"
   end
 
-  scenario 'when searching outside the covered area' do
+  scenario "when searching outside the covered area" do
     # BBox param is in Oklahoma
     visit search_catalog_path(
-      q: 'A new & accurate map of Asia',
-      bbox: '-98.717651 34.40691 -96.37207 36.199958'
+      q: "A new & accurate map of Asia",
+      bbox: "-98.717651 34.40691 -96.37207 36.199958"
     )
-    results = page.all('div.documentHeader.row')
+    results = page.all("div.documentHeader.row")
     expect(results.count).to eq 0
   end
 end

data/spec/features/search_results_icons_spec.rb

@@ -1,16 +1,17 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'search results display document iconography' do
-  scenario 'when viewing result row' do
+require "spec_helper"
+
+feature "search results display document iconography" do
+  scenario "when viewing result row" do
     # Search returns fixture stanford-cg357zz0321
     visit search_catalog_path(
-      search_field: 'all_fields',
-      q: 'stanford-cg357zz0321'
+      search_field: "all_fields",
+      q: "stanford-cg357zz0321"
     )
-    first_result = page.all('span.status-icons > span')
-    expect(first_result[0][:class]).to include 'blacklight-icon-dataset'
-    expect(first_result[1][:class]).to include 'blacklight-icon-stanford'
-    expect(first_result[2][:class]).to include 'blacklight-icon-restricted'
+    first_result = page.all("span.status-icons > span")
+    expect(first_result[0][:class]).to include "blacklight-icon-dataset"
+    expect(first_result[1][:class]).to include "blacklight-icon-stanford"
+    expect(first_result[2][:class]).to include "blacklight-icon-restricted"
   end
 end

data/spec/features/search_results_map_spec.rb

@@ -1,48 +1,49 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'search results map', js: true do
-  scenario 'present on a search result page' do
-    visit search_catalog_path(q: 'Minnesota')
-    expect(page).to have_css '#map'
+require "spec_helper"
+
+feature "search results map", js: true do
+  scenario "present on a search result page" do
+    visit search_catalog_path(q: "Minnesota")
+    expect(page).to have_css "#map"
   end
-  xscenario 'view is scoped to Minnesota' do
+  xscenario "view is scoped to Minnesota" do
     # pending 'Minnesota fixtures have changed.'
     visit root_path
-    click_link 'Minnesota, United States'
-    expect(page).to have_css '#map'
-    bbox = page.find('#map')['data-js-map-render-bbox']
+    click_link "Minnesota, United States"
+    expect(page).to have_css "#map"
+    bbox = page.find("#map")["data-js-map-render-bbox"]
 
     # Example bbox for Place > Minnesota, United States:
     # "-101.90917968749999,38.75408327579141,-83.27636718749999,53.27835301753182"
-    left, bottom, right, top = bbox.split(',')
+    left, bottom, right, top = bbox.split(",")
     expect(left.to_f).to be_within(2).of(-101)
     expect(bottom.to_f).to be_within(3).of(38)
     expect(right.to_f).to be_within(2).of(-83)
     expect(top.to_f).to be_within(2).of(53)
   end
-  scenario 'view is scoped to Twin Cities metro area' do
-    visit search_catalog_path(q: 'Minneapolis')
-    expect(page).to have_css '#map'
-    bbox = page.find('#map')['data-js-map-render-bbox']
+  scenario "view is scoped to Twin Cities metro area" do
+    visit search_catalog_path(q: "Minneapolis")
+    expect(page).to have_css "#map"
+    bbox = page.find("#map")["data-js-map-render-bbox"]
 
     # Example bbox for q: Minneapolis
     # "-94.537353515625,44.004669106432225,-92.208251953125,45.87088761346192"
-    left, bottom, right, top = bbox.split(',')
+    left, bottom, right, top = bbox.split(",")
     expect(left.to_f).to be_within(1).of(-94)
     expect(bottom.to_f).to be_within(1).of(44)
     expect(right.to_f).to be_within(1).of(-92)
     expect(top.to_f).to be_within(1).of(45)
   end
-  scenario 'view is scoped to NYC' do
+  scenario "view is scoped to NYC" do
     visit root_path
-    click_link 'New York, New York'
-    expect(page).to have_css '#map'
-    bbox = page.find('#map')['data-js-map-render-bbox']
+    click_link "New York, New York"
+    expect(page).to have_css "#map"
+    bbox = page.find("#map")["data-js-map-render-bbox"]
 
     # Example bbox for Place > New York, New York, United States
     # "-74.26895141601562,40.455307212131494,-73.68667602539061,40.95501133048621"
-    left, bottom, right, top = bbox.split(',')
+    left, bottom, right, top = bbox.split(",")
     expect(left.to_f).to be_within(2).of(-74)
     expect(bottom.to_f).to be_within(2).of(40)
     expect(right.to_f).to be_within(2).of(-73)

data/spec/features/search_results_overlap_ratio_spec.rb

@@ -1,56 +1,57 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'spatial search results overlap ratio' do
-  scenario 'result bboxes fully contained, overlap ratio applied to relevancy' do
+require "spec_helper"
+
+feature "spatial search results overlap ratio" do
+  scenario "result bboxes fully contained, overlap ratio applied to relevancy" do
     allow(Settings).to receive(:OVERLAP_RATIO_BOOST).and_return 200
 
     # BBox param is the US upper midwest and Canada, roughly centered on the state of Minnesota
     visit search_catalog_path(
-      bbox: '-103.196521 39.21962 -84.431873 53.63497',
-      "f[#{Settings.FIELDS.PROVIDER}][]": 'University of Minnesota'
+      bbox: "-103.196521 39.21962 -84.431873 53.63497",
+      "f[#{Settings.FIELDS.PROVIDER}][]": "University of Minnesota"
     )
 
     # MN State result
     # Slightly bigger bbox / result bbox fits bbox param best
-    expect(position_in_result_page(page, 'e9c71086-6b25-4950-8e1c-84c2794e3382')).to eq 1
+    expect(position_in_result_page(page, "e9c71086-6b25-4950-8e1c-84c2794e3382")).to eq 1
 
     # MN State result
     # Slightly smaller bbox / result bbox fits bbox param second best
-    expect(position_in_result_page(page, '2eddde2f-c222-41ca-bd07-2fd74a21f4de')).to eq 2
+    expect(position_in_result_page(page, "2eddde2f-c222-41ca-bd07-2fd74a21f4de")).to eq 2
 
     # TC Metro result
     # Smaller bbox / result bbox fits bbox param third
-    expect(position_in_result_page(page, '02236876-9c21-42f6-9870-d2562da8e44f')).to eq 3
+    expect(position_in_result_page(page, "02236876-9c21-42f6-9870-d2562da8e44f")).to eq 3
   end
 
-  scenario 'three bboxes overlap, but none are fully contained, overlap ratio should still impact relevancy' do
+  scenario "three bboxes overlap, but none are fully contained, overlap ratio should still impact relevancy" do
     allow(Settings).to receive(:OVERLAP_RATIO_BOOST).and_return 200
 
     # BBox param is the center to western edge of New York state and Canada, roughly centered on Lake Ontario
     visit search_catalog_path(
-      bbox: '-83.750499 40.41709 -74.368175 47.963663',
-      "f[#{Settings.FIELDS.PROVIDER}][]": 'Cornell'
+      bbox: "-83.750499 40.41709 -74.368175 47.963663",
+      "f[#{Settings.FIELDS.PROVIDER}][]": "Cornell"
     )
 
     # NY State result
     # Bigger bbox / result bbox overlaps bbox param best
-    expect(position_in_result_page(page, 'cugir-008186')).to be < 4
+    expect(position_in_result_page(page, "cugir-008186")).to be < 4
 
     # NY State result
     # Bigger bbox / result bbox overlaps bbox param best (score tie)
-    expect(position_in_result_page(page, 'cugir-008186-no-downloadurl')).to be < 4
+    expect(position_in_result_page(page, "cugir-008186-no-downloadurl")).to be < 4
 
     # NY Adirondak Region result
     # Smaller bbox / result bbox overlaps bbox param the least
-    expect(position_in_result_page(page, 'cugir-007741')).to eq 4
+    expect(position_in_result_page(page, "cugir-007741")).to eq 4
   end
 end
 
 def position_in_result_page(page, id)
   results = []
-  page.all('div.documentHeader.row').each do |div|
-    results << div['data-layer-id']
+  page.all("div.documentHeader.row").each do |div|
+    results << div["data-layer-id"]
   end
   results.index(id) + 1
 end