moshen / node-googlemaps

A simple way to query the Google Maps API from Node.js
MIT License
559 stars 148 forks source link

Allow mocked google api for local development #111

Open antony opened 8 years ago

antony commented 8 years ago

I've been using this in my own project for a while but I thought it was due a pull request.

I like to be able to develop locally and also not hammer the google API during tests, so this allows a simple override of the gmaps api url using config.

fabriziomoscon commented 8 years ago

It looks like a good idea. Could you squash all your commits in 1

git rebase -i HEAD^^^
# replace pick with f for the commit you want to change
git commit --amend
git push origin master -f

Because you are making changes to the config I think these should be reported in the README. Could you please add some instructions on how to do add a custom url. I am curious to know how did you mock the Google map API server.

evansiroky commented 8 years ago

I do a similar thing where I send in a custom request function to GoogleMapsAPI constructor. The custom request function sends back mock result data on disk similar to how the unit tests work.

antony commented 8 years ago

Hey @fabriziomoscon will do - sorry I only just saw this.

Btw did you know that github can squash commits now? The merge button above can be dropped down to choose 'squash and merge' :)

fabriziomoscon commented 8 years ago

@moshen I think squash and merge is an option that administrators can enable: https://github.com/blog/2141-squash-your-commits#whats-changing would you mind enable the checks from the admin page please?

antony commented 8 years ago

I've added the docs which should help you @fabriziomoscon understand how I mock the API in my own applications, but also provide help to others wanting to do something similar.

If you don't have squash+merge enabled I can squash it tonight ready for merge.