geoblacklight 4.0.0 → 4.1.0

data/spec/features/download_layer_spec.rb

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Download layer' do
+require "spec_helper"
+
+feature "Download layer" do
   let(:shapefile_download) { instance_double(Geoblacklight::ShapefileDownload) }
   let(:kmz_download) { instance_double(Geoblacklight::KmzDownload) }
 
@@ -10,102 +11,102 @@ feature 'Download layer' do
     allow(Geoblacklight::KmzDownload).to receive(:new).and_return(kmz_download)
   end
 
-  scenario 'clicking initial shapefile download button should trigger download', js: true do
-    expect(shapefile_download).to receive(:get).and_return('mit-f6rqs4ucovjk2-shapefile.zip')
-    visit solr_document_path('mit-f6rqs4ucovjk2')
-    find('#downloads-button').click
-    find('a[data-download-type="shapefile"]', text: 'Export Shapefile').click
+  scenario "clicking initial shapefile download button should trigger download", js: true do
+    expect(shapefile_download).to receive(:get).and_return("mit-f6rqs4ucovjk2-shapefile.zip")
+    visit solr_document_path("mit-f6rqs4ucovjk2")
+    find("#downloads-button").click
+    find('a[data-download-type="shapefile"]', text: "Export Shapefile").click
     expect(page).to have_css(
       'a[href="/download/file/mit-f6rqs4ucovjk2-shapefile.zip"]',
-      text: 'Your file mit-f6rqs4ucovjk2-shapefile.zip is ready for download'
+      text: "Your file mit-f6rqs4ucovjk2-shapefile.zip is ready for download"
     )
   end
 
-  scenario 'failed download should return message with link to layer', js: true do
-    expect(shapefile_download).to receive(:get).and_raise(Geoblacklight::Exceptions::ExternalDownloadFailed.new(message: 'Failed', url: 'http://www.example.com/failed'))
-    visit solr_document_path('mit-f6rqs4ucovjk2')
-    find('#downloads-button').click
-    find('#downloads-collapse a[data-download-type="shapefile"]', text: 'Export Shapefile').click
-    expect(page).to have_css 'div.alert.alert-danger', text: 'Sorry, the requested file could not be downloaded. Try downloading it directly from:'
-    expect(page).to have_css 'a', text: 'http://www.example.com/failed'
+  scenario "failed download should return message with link to layer", js: true do
+    expect(shapefile_download).to receive(:get).and_raise(Geoblacklight::Exceptions::ExternalDownloadFailed.new(message: "Failed", url: "http://www.example.com/failed"))
+    visit solr_document_path("mit-f6rqs4ucovjk2")
+    find("#downloads-button").click
+    find('#downloads-collapse a[data-download-type="shapefile"]', text: "Export Shapefile").click
+    expect(page).to have_css "div.alert.alert-danger", text: "Sorry, the requested file could not be downloaded. Try downloading it directly from:"
+    expect(page).to have_css "a", text: "http://www.example.com/failed"
   end
 
-  scenario 'clicking kmz download button should trigger download', js: true do
-    expect(kmz_download).to receive(:get).and_return('mit-f6rqs4ucovjk2-kmz.kmz')
-    visit solr_document_path('mit-f6rqs4ucovjk2')
-    find('#downloads-button').click
-    find('#downloads-collapse a[data-download-type="kmz"]', text: 'Export KMZ').click
+  scenario "clicking kmz download button should trigger download", js: true do
+    expect(kmz_download).to receive(:get).and_return("mit-f6rqs4ucovjk2-kmz.kmz")
+    visit solr_document_path("mit-f6rqs4ucovjk2")
+    find("#downloads-button").click
+    find('#downloads-collapse a[data-download-type="kmz"]', text: "Export KMZ").click
     expect(page).to have_css(
       'a[href="/download/file/mit-f6rqs4ucovjk2-kmz.kmz"]',
-      text: 'Your file mit-f6rqs4ucovjk2-kmz.kmz is ready for download'
+      text: "Your file mit-f6rqs4ucovjk2-kmz.kmz is ready for download"
     )
   end
 
-  scenario 'jpg download option should be present under toggle' do
-    visit solr_document_path('princeton-02870w62c')
-    find('#downloads-button').click
-    expect(page).to have_css('#downloads-collapse a', text: 'Original JPG')
+  scenario "jpg download option should be present under toggle" do
+    visit solr_document_path("princeton-02870w62c")
+    find("#downloads-button").click
+    expect(page).to have_css("#downloads-collapse a", text: "Original JPG")
   end
 
-  scenario 'clicking jpg download button should redirect to external image' do
-    visit solr_document_path('princeton-02870w62c')
-    find('#downloads-button').click
-    expect(page).to have_css("#downloads-collapse a[href='https://iiif-cloud.princeton.edu/iiif/2/6c%2F52%2F12%2F6c5212e81bc845f59bb1cdc740a88bad%2Fintermediate_file/full/full/0/default.jpg']", text: 'Original JPG')
+  scenario "clicking jpg download button should redirect to external image" do
+    visit solr_document_path("princeton-02870w62c")
+    find("#downloads-button").click
+    expect(page).to have_css("#downloads-collapse a[href='https://iiif-cloud.princeton.edu/iiif/2/6c%2F52%2F12%2F6c5212e81bc845f59bb1cdc740a88bad%2Fintermediate_file/full/full/0/default.jpg']", text: "Original JPG")
   end
 
-  scenario 'options should be available under toggle' do
-    visit solr_document_path('mit-f6rqs4ucovjk2')
-    find('#downloads-button').click
-    expect(page).to have_css('#downloads-collapse a[data-download-type="shapefile"]', text: 'Export Shapefile')
-    expect(page).to have_css('#downloads-collapse a[data-download-type="kmz"]', text: 'Export KMZ')
+  scenario "options should be available under toggle" do
+    visit solr_document_path("mit-f6rqs4ucovjk2")
+    find("#downloads-button").click
+    expect(page).to have_css('#downloads-collapse a[data-download-type="shapefile"]', text: "Export Shapefile")
+    expect(page).to have_css('#downloads-collapse a[data-download-type="kmz"]', text: "Export KMZ")
   end
 
-  scenario 'restricted layer should not have download available to non logged in user' do
-    visit solr_document_path('stanford-cg357zz0321')
-    expect(page).to have_css 'a', text: 'Login to View and Download'
-    expect(page).not_to have_css 'button', text: 'Download'
+  scenario "restricted layer should not have download available to non logged in user" do
+    visit solr_document_path("stanford-cg357zz0321")
+    expect(page).to have_css "a", text: "Login to View and Download"
+    expect(page).not_to have_css "button", text: "Download"
   end
 
-  scenario 'restricted layer should have download available to logged in user' do
+  scenario "restricted layer should have download available to logged in user" do
     sign_in
-    visit solr_document_path('stanford-cg357zz0321')
-    expect(page).not_to have_css 'a', text: 'Login to view and download'
-    expect(page).to have_css 'button', text: 'Download'
+    visit solr_document_path("stanford-cg357zz0321")
+    expect(page).not_to have_css "a", text: "Login to view and download"
+    expect(page).to have_css "button", text: "Download"
   end
 
-  scenario 'layer with direct download and wms/wfs should include all download types' do
+  scenario "layer with direct download and wms/wfs should include all download types" do
     sign_in
-    visit solr_document_path('stanford-cg357zz0321')
-    find('#downloads-button').click
-    expect(page).to have_css('#downloads-collapse a', text: 'Original')
-    expect(page).to have_css('#downloads-collapse a', text: 'Export')
+    visit solr_document_path("stanford-cg357zz0321")
+    find("#downloads-button").click
+    expect(page).to have_css("#downloads-collapse a", text: "Original")
+    expect(page).to have_css("#downloads-collapse a", text: "Export")
   end
 
-  scenario 'clicking download GeoTIFF for Harvard layer should show email form', js: true do
-    visit solr_document_path('harvard-g7064-s2-1834-k3')
-    find('#downloads-button').click
-    find('a[data-download-type="harvard-hgl"]', text: 'GeoTIFF').click
-    expect(page).to have_css('#hglRequest')
+  scenario "clicking download GeoTIFF for Harvard layer should show email form", js: true do
+    visit solr_document_path("harvard-g7064-s2-1834-k3")
+    find("#downloads-button").click
+    find('a[data-download-type="harvard-hgl"]', text: "GeoTIFF").click
+    expect(page).to have_css("#hglRequest")
   end
 
-  context 'with a successful request to the server' do
+  context "with a successful request to the server" do
     let(:hgl_download) { instance_double(Geoblacklight::HglDownload) }
 
-    xscenario 'submitting email form should trigger HGL request', js: true do
+    xscenario "submitting email form should trigger HGL request", js: true do
       # There are currently difficulties with testing the HGL downloader
-      visit solr_document_path('harvard-g7064-s2-1834-k3')
-      find('#downloads-button').click
-      find('a[data-download-type="harvard-hgl"]', text: 'Original GeoTIFF').click
+      visit solr_document_path("harvard-g7064-s2-1834-k3")
+      find("#downloads-button").click
+      find('a[data-download-type="harvard-hgl"]', text: "Original GeoTIFF").click
 
       allow_any_instance_of(Geoblacklight::HglDownload).to receive(:new).and_return(hgl_download)
-      allow(hgl_download).to receive(:get).and_return('success')
+      allow(hgl_download).to receive(:get).and_return("success")
 
-      within '#hglRequest' do
-        fill_in('Email', with: 'foo@example.com')
-        click_button('Request')
+      within "#hglRequest" do
+        fill_in("Email", with: "foo@example.com")
+        click_button("Request")
       end
-      expect(page).to have_css('.alert-success')
-      expect(page).to have_content('You should receive an email when your download is ready')
+      expect(page).to have_css(".alert-success")
+      expect(page).to have_content("You should receive an email when your download is ready")
     end
   end
 end

data/spec/features/empty_search_spec.rb

@@ -1,12 +1,13 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Empty search' do
+require "spec_helper"
+
+feature "Empty search" do
   before do
     visit root_path
   end
-  scenario 'Entering empty search returns results page' do
-    click_button 'search'
-    expect(page).to have_css '#documents'
+  scenario "Entering empty search returns results page" do
+    click_button "search"
+    expect(page).to have_css "#documents"
   end
 end

data/spec/features/esri_viewer_spec.rb

@@ -1,42 +1,43 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'feature_layer reference', js: true do
-  scenario 'displays image map layer' do
-    skip 'spec depend on potentially flaky external services'
-    visit solr_document_path '32653ed6-8d83-4692-8a06-bf13ffe2c018'
-    expect(page).to have_css '.leaflet-control-zoom', visible: true
-    expect(page).to have_css 'img.leaflet-image-layer', visible: true
+require "spec_helper"
+
+feature "feature_layer reference", js: true do
+  scenario "displays image map layer" do
+    skip "spec depend on potentially flaky external services"
+    visit solr_document_path "32653ed6-8d83-4692-8a06-bf13ffe2c018"
+    expect(page).to have_css ".leaflet-control-zoom", visible: true
+    expect(page).to have_css "img.leaflet-image-layer", visible: true
   end
-  scenario 'displays dynamic layer (all layers)' do
-    skip 'spec depend on potentially flaky external services'
-    visit solr_document_path '90f14ff4-1359-4beb-b931-5cb41d20ab90'
-    expect(page).to have_css '.leaflet-control-zoom', visible: true
-    expect(page).to have_css 'img.leaflet-image-layer', visible: true
+  scenario "displays dynamic layer (all layers)" do
+    skip "spec depend on potentially flaky external services"
+    visit solr_document_path "90f14ff4-1359-4beb-b931-5cb41d20ab90"
+    expect(page).to have_css ".leaflet-control-zoom", visible: true
+    expect(page).to have_css "img.leaflet-image-layer", visible: true
   end
-  scenario 'displays dynamic layer (single layer)' do
-    skip 'spec depend on potentially flaky external services'
-    visit solr_document_path '4669301e-b4b2-4c8b-bf40-01b968a2865b'
-    expect(page).to have_css '.leaflet-control-zoom', visible: true
-    expect(page).to have_css 'img.leaflet-image-layer', visible: true
+  scenario "displays dynamic layer (single layer)" do
+    skip "spec depend on potentially flaky external services"
+    visit solr_document_path "4669301e-b4b2-4c8b-bf40-01b968a2865b"
+    expect(page).to have_css ".leaflet-control-zoom", visible: true
+    expect(page).to have_css "img.leaflet-image-layer", visible: true
   end
-  scenario 'displays feature layer' do
-    pending 'cannot currently test for svg feature'
-    visit solr_document_path 'f406332e63eb4478a9560ad86ae90327_18'
-    expect(page).to have_css '.leaflet-control-zoom', visible: true
-    expect(Nokogiri::HTML.parse(page.body).css('g').length).to eq 23
+  scenario "displays feature layer" do
+    pending "cannot currently test for svg feature"
+    visit solr_document_path "f406332e63eb4478a9560ad86ae90327_18"
+    expect(page).to have_css ".leaflet-control-zoom", visible: true
+    expect(Nokogiri::HTML.parse(page.body).css("g").length).to eq 23
     fail
   end
-  scenario 'displays tiled map layer' do
-    skip 'spec depend on potentially flaky external services'
-    visit solr_document_path 'nyu-test-soil-survey-map'
-    expect(page).to have_css '.leaflet-control-zoom', visible: true
-    expect(page).to have_css 'img.leaflet-tile.leaflet-tile-loaded', visible: true
+  scenario "displays tiled map layer" do
+    skip "spec depend on potentially flaky external services"
+    visit solr_document_path "nyu-test-soil-survey-map"
+    expect(page).to have_css ".leaflet-control-zoom", visible: true
+    expect(page).to have_css "img.leaflet-tile.leaflet-tile-loaded", visible: true
   end
-  scenario 'displays Esri WMS layer' do
-    skip 'spec depend on potentially flaky external services'
-    visit solr_document_path 'purdue-urn-f082acb1-b01e-4a08-9126-fd62a23fd9aa'
-    expect(page).to have_css '.leaflet-control-zoom', visible: true
-    expect(page).to have_css 'img.leaflet-tile.leaflet-tile-loaded', visible: true
+  scenario "displays Esri WMS layer" do
+    skip "spec depend on potentially flaky external services"
+    visit solr_document_path "purdue-urn-f082acb1-b01e-4a08-9126-fd62a23fd9aa"
+    expect(page).to have_css ".leaflet-control-zoom", visible: true
+    expect(page).to have_css "img.leaflet-tile.leaflet-tile-loaded", visible: true
   end
 end

data/spec/features/exports_spec.rb

@@ -1,28 +1,29 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Export features' do
-  feature 'when item is public and wfs is available' do
-    feature 'Open in Carto' do
-      scenario 'shows up in tools' do
-        visit solr_document_path 'tufts-cambridgegrid100-04'
-        expect(page).to have_css 'li.carto a', text: 'Open in Carto'
-        click_link 'Open in Carto'
+require "spec_helper"
+
+feature "Export features" do
+  feature "when item is public and wfs is available" do
+    feature "Open in Carto" do
+      scenario "shows up in tools" do
+        visit solr_document_path "tufts-cambridgegrid100-04"
+        expect(page).to have_css "li.carto a", text: "Open in Carto"
+        click_link "Open in Carto"
       end
     end
   end
-  feature 'when esri web services are available' do
-    feature 'Open in ArcGIS Online' do
-      scenario 'shows up in tools' do
-        visit solr_document_path '90f14ff4-1359-4beb-b931-5cb41d20ab90'
-        expect(page).to have_css 'li.arcgis a', text: 'Open in ArcGIS Online'
+  feature "when esri web services are available" do
+    feature "Open in ArcGIS Online" do
+      scenario "shows up in tools" do
+        visit solr_document_path "90f14ff4-1359-4beb-b931-5cb41d20ab90"
+        expect(page).to have_css "li.arcgis a", text: "Open in ArcGIS Online"
       end
     end
   end
-  feature 'when restricted or no wfs' do
-    scenario 'is not in tools' do
-      visit solr_document_path 'princeton-02870w62c'
-      expect(page).not_to have_css 'li.carto a', text: 'Open in Carto'
+  feature "when restricted or no wfs" do
+    scenario "is not in tools" do
+      visit solr_document_path "princeton-02870w62c"
+      expect(page).not_to have_css "li.carto a", text: "Open in Carto"
     end
   end
 end

data/spec/features/full_screen_controll_spec.rb

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
-feature 'Leaflet fullscreen control', js: true do
-  scenario 'IIIF layer should have full screen control' do
-    visit solr_document_path('princeton-sx61dn82p')
-    expect(page).to have_css('.leaflet-control-fullscreen-button')
+feature "Leaflet fullscreen control", js: true do
+  scenario "IIIF layer should have full screen control" do
+    visit solr_document_path("princeton-sx61dn82p")
+    expect(page).to have_css(".leaflet-control-fullscreen-button")
   end
 
-  scenario 'WMS layer should have full screen control' do
-    visit solr_document_path('stanford-cz128vq0535')
-    expect(page).to have_css('.leaflet-control-fullscreen-button')
+  scenario "WMS layer should have full screen control" do
+    visit solr_document_path("stanford-cz128vq0535")
+    expect(page).to have_css(".leaflet-control-fullscreen-button")
   end
 end

data/spec/features/help_text_spec.rb

@@ -1,9 +1,10 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Help Text' do
-  scenario 'Displays help text entry' do
-    visit '/catalog/stanford-cg357zz0321'
-    expect(page).to have_css '.help-text', count: 1
+require "spec_helper"
+
+feature "Help Text" do
+  scenario "Displays help text entry" do
+    visit "/catalog/stanford-cg357zz0321"
+    expect(page).to have_css ".help-text", count: 1
   end
 end

data/spec/features/home_page_spec.rb

@@ -1,46 +1,47 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Home page', js: true do # use js: true for tests which require js, but it slows things down
+require "spec_helper"
+
+feature "Home page", js: true do # use js: true for tests which require js, but it slows things down
   before do
     visit root_path
   end
-  scenario 'navbar' do
-    expect(page).to have_css '#bookmarks_nav'
-    expect(page).to have_css 'a', text: 'History'
+  scenario "navbar" do
+    expect(page).to have_css "#bookmarks_nav"
+    expect(page).to have_css "a", text: "History"
   end
-  scenario 'search bar' do
-    expect(page).not_to have_css '#search-navbar'
-    within '.jumbotron' do
-      expect(page).to have_css 'h2', text: 'Explore and discover...'
-      expect(page).to have_css 'h3', text: 'Find the maps and data you need'
-      expect(page).to have_css 'form.search-query-form'
+  scenario "search bar" do
+    expect(page).not_to have_css "#search-navbar"
+    within ".jumbotron" do
+      expect(page).to have_css "h2", text: "Explore and discover..."
+      expect(page).to have_css "h3", text: "Find the maps and data you need"
+      expect(page).to have_css "form.search-query-form"
     end
   end
-  scenario 'find by category' do
-    expect(page).to have_css '.category-block', count: 4
-    expect(page).to have_css '.home-facet-link', count: 36
-    expect(page).to have_css 'a.more_facets_link', count: 4
-    click_link 'Counties'
-    expect(page).to have_css '.filter-name', text: 'Subject'
-    expect(page).to have_css '.filter-value', text: 'Counties'
+  scenario "find by category" do
+    expect(page).to have_css ".category-block", count: 4
+    expect(page).to have_css ".home-facet-link", count: 36
+    expect(page).to have_css "a.more_facets_link", count: 4
+    click_link "Counties"
+    expect(page).to have_css ".filter-name", text: "Subject"
+    expect(page).to have_css ".filter-value", text: "Counties"
   end
-  scenario 'map should be visible' do
-    within '#main-container' do
-      expect(page).to have_css('#map')
-      expect(page).to have_css('img.leaflet-tile', minimum: 3)
+  scenario "map should be visible" do
+    within "#main-container" do
+      expect(page).to have_css("#map")
+      expect(page).to have_css("img.leaflet-tile", minimum: 3)
     end
   end
-  scenario 'clicking map search should create a spatial search' do
-    within '#map' do
-      find('.search-control a').click
+  scenario "clicking map search should create a spatial search" do
+    within "#map" do
+      find(".search-control a").click
       expect(page.current_url).to match(/bbox=/)
     end
-    expect(page).to have_css '#documents'
+    expect(page).to have_css "#documents"
   end
-  scenario 'can search by placename' do
-    click_link 'New York, New York'
-    results = page.all(:css, 'article.document')
+  scenario "can search by placename" do
+    click_link "New York, New York"
+    results = page.all(:css, "article.document")
     expect(results.count).to equal(4)
   end
 end

data/spec/features/iiif_viewer_spec.rb

@@ -1,9 +1,10 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'iiif reference' do
-  scenario 'displays leaflet viewer', js: true do
-    visit solr_document_path('princeton-02870w62c')
-    expect(page).to have_css '.leaflet-control-zoom', visible: true
+require "spec_helper"
+
+feature "iiif reference" do
+  scenario "displays leaflet viewer", js: true do
+    visit solr_document_path("princeton-02870w62c")
+    expect(page).to have_css ".leaflet-control-zoom", visible: true
   end
 end

data/spec/features/index_map_spec.rb

@@ -1,31 +1,32 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Index map' do
+require "spec_helper"
+
+feature "Index map" do
   # Colors
-  default_color = '#7FCDBB'
-  selected_color = '#2C7FB8'
-  scenario 'displays index map viewer (polygon)', js: true do
-    visit solr_document_path('stanford-fb897vt9938')
+  default_color = "#7FCDBB"
+  selected_color = "#2C7FB8"
+  scenario "displays index map viewer (polygon)", js: true do
+    visit solr_document_path("stanford-fb897vt9938")
     # Wait until SVG elements are added
-    expect(page).to have_css '.leaflet-overlay-pane svg'
-    within '#map' do
+    expect(page).to have_css ".leaflet-overlay-pane svg"
+    within "#map" do
       expect(page).to have_css "svg g path:nth-child(2)[fill='#{default_color}']"
-      find('svg g path:nth-child(2)').click
+      find("svg g path:nth-child(2)").click
       expect(page).to have_css "svg g path:nth-child(2)[fill='#{selected_color}']"
-      first('svg g path').click
+      first("svg g path").click
       expect(page).to have_css "svg g path:nth-child(2)[fill='#{default_color}']"
     end
   end
-  scenario 'displays index map viewer (points)', js: true do
-    visit solr_document_path('cornell-ny-aerial-photos-1960s')
+  scenario "displays index map viewer (points)", js: true do
+    visit solr_document_path("cornell-ny-aerial-photos-1960s")
     # Wait until SVG elements are added
-    expect(page).to have_css '.leaflet-overlay-pane svg'
-    within '#map' do
+    expect(page).to have_css ".leaflet-overlay-pane svg"
+    within "#map" do
       expect(page).to have_css "svg g path:nth-child(2)[fill='#{default_color}']"
-      find('svg g path:nth-child(2)').click
+      find("svg g path:nth-child(2)").click
       expect(page).to have_css "svg g path:nth-child(2)[fill='#{selected_color}']"
-      first('svg g path').click
+      first("svg g path").click
       expect(page).to have_css "svg g path:nth-child(2)[fill='#{default_color}']"
     end
   end

data/spec/features/layer_inspection_spec.rb

@@ -1,11 +1,12 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Layer inspection', js: true do
-  scenario 'clicking map should trigger inspection' do
-    visit solr_document_path('mit-f6rqs4ucovjk2')
-    expect(page).to have_css('th', text: 'Attribute')
-    find('#map').click
-    expect(page).not_to have_css('td.default-text')
+require "spec_helper"
+
+feature "Layer inspection", js: true do
+  scenario "clicking map should trigger inspection" do
+    visit solr_document_path("mit-f6rqs4ucovjk2")
+    expect(page).to have_css("th", text: "Attribute")
+    find("#map").click
+    expect(page).not_to have_css("td.default-text")
   end
 end

data/spec/features/layer_opacity_spec.rb

@@ -1,22 +1,23 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Layer opacity', js: true do
-  scenario 'WMS layer should have opacity control' do
-    visit solr_document_path('mit-f6rqs4ucovjk2')
-    expect(page).to have_css('div.opacity-text', text: '75%')
-    expect(page.all('div.leaflet-layer')[1][:style]).to match(/opacity: 0.75;/)
+require "spec_helper"
+
+feature "Layer opacity", js: true do
+  scenario "WMS layer should have opacity control" do
+    visit solr_document_path("mit-f6rqs4ucovjk2")
+    expect(page).to have_css("div.opacity-text", text: "75%")
+    expect(page.all("div.leaflet-layer")[1][:style]).to match(/opacity: 0.75;/)
   end
 
-  scenario 'ESRI image service layer should have opacity control' do
-    skip 'CORS error - Purdue web services are down'
-    visit solr_document_path('32653ed6-8d83-4692-8a06-bf13ffe2c018')
-    expect(page).to have_css('div.opacity-text', text: '75%')
-    expect(page.find('img.leaflet-image-layer', match: :first)[:style]).to match(/opacity: 0.75;/)
+  scenario "ESRI image service layer should have opacity control" do
+    skip "CORS error - Purdue web services are down"
+    visit solr_document_path("32653ed6-8d83-4692-8a06-bf13ffe2c018")
+    expect(page).to have_css("div.opacity-text", text: "75%")
+    expect(page.find("img.leaflet-image-layer", match: :first)[:style]).to match(/opacity: 0.75;/)
   end
 
-  scenario 'IIIF layer should not have opacity control' do
-    visit solr_document_path('princeton-02870w62c')
-    expect(page).not_to have_css('div.opacity-text', text: '75%')
+  scenario "IIIF layer should not have opacity control" do
+    visit solr_document_path("princeton-02870w62c")
+    expect(page).not_to have_css("div.opacity-text", text: "75%")
   end
 end

data/spec/features/layer_preview_spec.rb

@@ -1,18 +1,19 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'Layer preview', js: true do
-  scenario 'Restricted layer should show bounding box' do
-    visit solr_document_path('stanford-cg357zz0321')
-    within('#map') do
-      expect(page).to have_css('path')
+require "spec_helper"
+
+feature "Layer preview", js: true do
+  scenario "Restricted layer should show bounding box" do
+    visit solr_document_path("stanford-cg357zz0321")
+    within("#map") do
+      expect(page).to have_css("path")
     end
   end
 
-  scenario 'Public layer should show wms layer not bounding box' do
-    visit solr_document_path('mit-f6rqs4ucovjk2')
-    within '.leaflet-tile-pane' do
-      expect(page).to have_css('.leaflet-layer', count: 2)
+  scenario "Public layer should show wms layer not bounding box" do
+    visit solr_document_path("mit-f6rqs4ucovjk2")
+    within ".leaflet-tile-pane" do
+      expect(page).to have_css(".leaflet-layer", count: 2)
     end
   end
 end

data/spec/features/layer_with_no_references_spec.rb

@@ -1,9 +1,10 @@
 # frozen_string_literal: true
-require 'spec_helper'
 
-feature 'layer with no wms or iiif' do
-  scenario 'has map with bounding box' do
-    pending 'waiting on #201, need example metadata record'
+require "spec_helper"
+
+feature "layer with no wms or iiif" do
+  scenario "has map with bounding box" do
+    pending "waiting on #201, need example metadata record"
     fail
   end
 end