leaflet-environmental-layers icon indicating copy to clipboard operation
leaflet-environmental-layers copied to clipboard

Update aqicn.spec.js to address layer-specific test failures

Open jywarren opened this issue 3 years ago • 2 comments

Attempting some fixes to aqicn spec from #668 - reordering first, then will try ensuring we use fixture data instead of a real request on:

https://github.com/publiclab/leaflet-environmental-layers/blob/26826d527b23cd4c5a544011743fd96eec997675/cypress/integration/aqicn.spec.js#L26-L43

jywarren avatar Jun 07 '22 07:06 jywarren

gitpod-io[bot] avatar Jun 07 '22 07:06 gitpod-io[bot]

I see that for some reason it's zoomed into zoom 3, and some kind of random location.

-'#lat=42.94&lon=-82.97&zoom=3&layers=Standard,aqicnLayer'
+'#lat=51.15&lon=13.45&zoom=4&layers=Standard,aqicnLayer'

It doesn't look like clicking a marker and getting a popup, it doesn't pan the screen. Zoom 3 is pretty zoomed out, and -82 corrects to -52 because we can't look off the edge of the world. But otherwise looks innocuous. Not sure where 42x-82 coordinates are coming from? That's near Flint, Michigan, while the original was in Germany.

Aha - ok, the default map location is set in /example/layers.js - .setView([43, -83], 3);

jywarren avatar Jun 07 '22 08:06 jywarren