geoblacklight 2.1.2 → 3.0.0

data/spec/features/download_layer_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Download layer' do

data/spec/features/empty_search_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Empty search' do

data/spec/features/esri_viewer_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'feature_layer reference', js: true do
@@ -13,7 +14,6 @@ feature 'feature_layer reference', js: true do
     expect(page).to have_css 'img.leaflet-image-layer', visible: true
   end
   scenario 'displays dynamic layer (single layer)' do
-    pending 'external service disabled cors access'
     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
@@ -25,11 +25,6 @@ feature 'feature_layer reference', js: true do
     expect(Nokogiri::HTML.parse(page.body).css('g').length).to eq 23
     fail
   end
-  scenario 'displays image map layer' do
-    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 tiled map layer' do
     visit solr_document_path 'nyu-test-soil-survey-map'
     expect(page).to have_css '.leaflet-control-zoom', visible: true

data/spec/features/exports_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Export features' do
@@ -5,7 +6,7 @@ feature 'Export features' 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.exports a', text: 'Open in Carto'
+        expect(page).to have_css 'li.carto a', text: 'Open in Carto'
         click_link 'Open in Carto'
       end
     end
@@ -14,14 +15,14 @@ feature 'Export features' 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.exports a', text: 'Open in ArcGIS Online'
+        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.exports a', text: 'Open in Carto'
+      expect(page).not_to have_css 'li.carto a', text: 'Open in Carto'
     end
   end
 end

data/spec/features/help_text_spec.rb

@@ -0,0 +1,9 @@
+# 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
+  end
+end

data/spec/features/home_page_spec.rb

@@ -1,3 +1,4 @@
+# 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
@@ -37,4 +38,9 @@ feature 'Home page', js: true do # use js: true for tests which require js, but
     end
     expect(page).to have_css '#documents'
   end
+  scenario 'can search by placename' do
+    click_link 'Minnesota, United States'
+    results = page.all(:css, 'article.document')
+    expect(results.count).to equal(3)
+  end
 end

data/spec/features/iiif_viewer_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'iiif reference' do

data/spec/features/index_map_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Index map' do

data/spec/features/layer_inspection_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Layer inspection', js: true do

data/spec/features/layer_opacity_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Layer opacity', js: true do

data/spec/features/layer_preview_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Layer preview', js: true do

data/spec/features/layer_with_no_references_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'layer with no wms or iiif' do

data/spec/features/linkified_attribute_table_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Attribute table values', js: true do

data/spec/features/metadata_panel_spec.rb

@@ -1,7 +1,37 @@
+# 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')) }
+    let(:fgdc_response) { instance_double(Faraday::Response) }
+    let(:fgdc_connection) { instance_double(Faraday::Connection) }
+    let(:iso19139_response) { instance_double(Faraday::Response) }
+    let(:iso19139_connection) { instance_double(Faraday::Connection) }
+    let(:response) { instance_double(Faraday::Response) }
+    let(:connection) { instance_double(Faraday::Connection) }
+
+    before 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(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(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(hash_including(url: 'http://127.0.0.1:8983/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'
@@ -14,14 +44,28 @@ feature 'Metadata tools' do
         end
       end
     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'
-      using_wait_time 15 do
-        within '.metadata-view' do
-          expect(page).to have_css '.pill-metadata', text: 'MODS'
-          expect(page).to have_css '.CodeRay'
+    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) }
+
+      before 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).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'
+        using_wait_time 15 do
+          within '.metadata-view' do
+            expect(page).to have_css '.pill-metadata', text: 'MODS'
+            expect(page).to have_css '.CodeRay'
+          end
         end
       end
     end

data/spec/features/missing_metadata_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Missing metadata', js: true do
@@ -9,4 +10,16 @@ feature 'Missing metadata', js: true 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')
+  end
+  scenario 'Yields error free show page for no dct_provenance_s' do
+    visit solr_document_path('99-0001-noProvenance')
+    expect(page).to have_css('#map')
+  end
+  scenario 'Yields error free show page for no solr_geom' do
+    visit solr_document_path('05d-p16022coll246-noGeo')
+    expect(page).to have_css('#map')
+  end
 end

data/spec/features/multiple_downloads_spec.rb

@@ -0,0 +1,15 @@
+# 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'
+      within '.card.downloads' 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
+end

data/spec/features/oembed_spec.rb

@@ -0,0 +1,10 @@
+# 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'
+    expect(page).to have_css '[data-protocol="Oembed"]'
+  end
+end

data/spec/features/relations_spec.rb

@@ -1,51 +1,33 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Display related documents' do
-  let(:expected_json_resp) do
-    {
-      'ancestors' =>
-        {
-          'numFound' => 0,
-          'start' => 0,
-          'docs' => []
-        },
-      'descendants' =>
-            {
-              'numFound' => 1,
-              'start' => 0,
-              'docs' =>
-                    [
-                      {
-                        'dc_title_s' => '2015 New York City Subway Complexes and Ridership',
-                        'layer_slug_s' => 'nyu_2451_34502'
-                      }
-                    ]
-            },
-      'current_doc' => 'nyu_2451_34635'
-    }
-  end
   scenario 'Record with dc_source_sm value(s) should have parent(s)' do
     visit relations_solr_document_path('nyu_2451_34502')
-    expect(page).to have_css('ul b', text: 'Source Datasets')
+    expect(page).to have_css('.relations .card-header h2', text: 'Source Datasets')
   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('ul b', text: 'Derived Datasets')
+    expect(page).to have_css('.relations .card-header h2', text: 'Derived Datasets')
   end
 
   scenario 'Relations should respond to json' do
     visit relations_solr_document_path('nyu_2451_34635', format: 'json')
-    expect(page.body).to eq(expected_json_resp.to_json)
+    response = JSON.parse(page.body)
+    expect(response['ancestors']['numFound']).to eq 0
+    expect(response['descendants']['docs'].first['layer_slug_s']).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('div.card-header', text: 'Data Relations')
+    expect(page).to have_css('.card.relations')
+    expect(page).to have_css('div.card-header', text: 'Derived Datasets')
   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('div.panel-heading', text: 'Data Relations')
+    expect(page).to have_no_css('.card.relations')
   end
 end

data/spec/features/saved_searches_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'saved searches' do

data/spec/features/search_bar_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'search bar' do

data/spec/features/search_results_map_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'search results map', js: true do
@@ -5,12 +6,6 @@ feature 'search results map', js: true do
     visit search_catalog_path(q: 'Minnesota')
     expect(page).to have_css '#map'
   end
-  scenario 'present on a search result page' do
-    visit root_path
-    click_link 'Minnesota, United States'
-    results = page.all(:css, 'article.document')
-    expect(results.count).to equal(3)
-  end
   scenario 'view is scoped to Minnesota' do
     visit root_path
     click_link 'Minnesota, United States'
@@ -21,7 +16,7 @@ feature 'search results map', js: true do
     # "-101.90917968749999,38.75408327579141,-83.27636718749999,53.27835301753182"
     left, bottom, right, top = bbox.split(',')
     expect(left.to_f).to be_within(2).of(-101)
-    expect(bottom.to_f).to be_within(2).of(38)
+    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

data/spec/features/search_results_overlap_ratio_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'spatial search results overlap ratio' do
@@ -34,15 +35,15 @@ feature 'spatial search results overlap ratio' do
 
     # NY State result
     # Bigger bbox / result bbox overlaps bbox param best
-    expect(position_in_result_page(page, 'cugir-008186')).to be < 3
+    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 < 3
+    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 3
+    expect(position_in_result_page(page, 'cugir-007741')).to eq 4
   end
 end
 

data/spec/features/search_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Search' do

data/spec/features/show_page_metadata_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Metadata display on show page' do

data/spec/features/sms_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'SMS tool', js: true do

data/spec/features/split_view.html.erb_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 feature 'Index view', js: true do
@@ -9,7 +10,7 @@ feature 'Index view', js: true do
   scenario 'should have documents and map on page' do
     visit search_catalog_path(f: { Settings.FIELDS.PROVENANCE => ['Stanford'] })
     expect(page).to have_css('#documents')
-    expect(page).to have_css('.document', count: 4)
+    expect(page).to have_css('.document', count: 5)
     expect(page).to have_css('#map')
   end
 
@@ -30,7 +31,7 @@ feature 'Index view', js: true do
       expect(page).to have_css('div.card.facet-limit', text: 'Data type')
       expect(page).to have_css('div.card.facet-limit', text: 'Format')
     end
-    click_link 'Institution'
+    click_button 'Institution'
     using_wait_time 120 do
       expect(page).to have_css('a.facet-select', text: 'Minnesota', visible: true)
       expect(page).to have_css('a.facet-select', text: 'MIT', visible: true)
@@ -51,9 +52,9 @@ feature 'Index view', js: true do
   end
 
   scenario 'click on a record area to expand collapse' do
-    within('.documentHeader', match: :first) do
+    within('article', match: :first) do
       expect(page).not_to have_css('.collapse')
-      find('.status-icons').click
+      find('button').click
       expect(page).to have_css('.collapse', visible: true)
     end
   end

data/spec/features/suppressed_records_spec.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+require 'spec_helper'
+
+feature 'Suppressed records' do
+  scenario 'are viewable' do
+    visit solr_document_path 'princeton-jq085m62x'
+    expect(page).to have_css('#document')
+  end
+
+  scenario 'are exportable' do
+    visit solr_document_path 'princeton-jq085m62x'
+    click_link 'Web services'
+    expect(page).to have_css 'h1.modal-title', text: 'Web services'
+  end
+end