dont force IdentifyFeatures passed through to include a map

Esri · Dec 21, 2017 · 32e14fd · 32e14fd
1 parent 3cb108d
commit 32e14fd
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/karma.conf.js b/karma.conf.js
@@ -13,15 +13,20 @@ module.exports = function (config) {
 
     // list of files / patterns to load in the browser
     // not sure why tests are failing when files are loaded in bulk
-
     files: [
       'node_modules/leaflet/dist/leaflet.css',
       'node_modules/leaflet/dist/leaflet-src.js',
       'dist/esri-leaflet-debug.js',
       // these two are the tempermental ones
       'spec/Layers/ImageMapLayerSpec.js',
       'spec/Layers/DynamicMapLayerSpec.js',
-      'spec/**/*Spec.js'
+      'spec/Layers/BasemapLayerSpec.js',
+      'spec/Layers/TiledMapLayerSpec.js',
+      'spec/Layers/RasterLayerSpec.js',
+      // 'spec/Layers/*Spec.js',
+      'spec/Tasks/*Spec.js',
+      'spec/Services/*Spec.js',
+      'spec/*Spec.js'
     ],
 
     // list of files to exclude

diff --git a/src/Layers/DynamicMapLayer.js b/src/Layers/DynamicMapLayer.js
@@ -89,7 +89,7 @@ export var DynamicMapLayer = RasterLayer.extend({
 
     var identifyRequest;
     if (this.options.popup) {
-      identifyRequest = this.options.popup.at(e.latlng);
+      identifyRequest = this.options.popup.on(this._map).at(e.latlng);
     } else {
       identifyRequest = this.identify().on(this._map).at(e.latlng);
     }