Merge pull request #1145 from geoblacklight/1142-arcgis

Disable ESRI Specs
geoblacklight · Feb 14, 2022 · 67d4eb4 · 67d4eb4
2 parents 53f4199 + 6d3fe06
commit 67d4eb4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/app/assets/javascripts/geoblacklight/viewers/esri.js b/app/assets/javascripts/geoblacklight/viewers/esri.js
@@ -20,6 +20,7 @@ GeoBlacklight.Viewer.Esri = GeoBlacklight.Viewer.Map.extend({
 
     // remove any trailing slash from endpoint url
     _this.data.url = _this.data.url.replace(/\/$/, '');
+
     L.esri.get(_this.data.url, {}, function(error, response){
       if(!error) {
         _this.layerInfo = response;

diff --git a/spec/features/esri_viewer_spec.rb b/spec/features/esri_viewer_spec.rb
@@ -3,17 +3,19 @@
 
 feature 'feature_layer reference', js: true do
   scenario 'displays image map layer' do
-    skip 'CORS error - Purdue web services are down'
+    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
   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
@@ -26,11 +28,13 @@
     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
   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