tarakc02 / rmapzen

R client for the Mapzen API
https://tarakc02.github.io/rmapzen/
Other
34 stars 4 forks source link

mz_reverse_geocode doesn't work and neither do the unit tests #2

Closed tarakc02 closed 7 years ago

tarakc02 commented 7 years ago

It creates a URL with parameters lat and lon instead of point.lat and point.lon, as specified here. This was a bug that was introduced when I switched from unwrap to as.mz_location, and was not caught by unit tests, because I am an idiot and setequal does not consider names. This means that all of the build_*_url tests are not doing what I thought they were doing.