Skip to content

Conversation

@Byte-Lab
Copy link
Contributor

  • When an invalid clientID / privateKey
    combination is used with the
    GoogleMapsBusinessProvider, throw an
    InvalidCredentialsException instead of a
    NoResultException.
  • Test that the exception is thrown with the
    testGetGeocodedDataWithInvalidClientIdAndKey(),
    and
    testGetGeocodedDataWithINvalidClientIdAndKeyNoSsl()
    methods.
  • Add two tests for the GoogleMapsProvider to
    test that expected results are returned when using
    a valid API Key (commented example added to
    phpunit.xml.dist and README.md) and that an
    InvalidCredentialsException is thrown when an
    incorrect API Key is used.

* When an invalid clientID / privateKey
  combination is used with the
  GoogleMapsBusinessProvider, throw an
  InvalidCredentialsException instead of a
  NoResultException.

* Test that the exception is thrown with the
  testGetGeocodedDataWithInvalidClientIdAndKey(),
  and
  testGetGeocodedDataWithINvalidClientIdAndKeyNoSsl()
  methods.

* Add two tests for the GoogleMapsProvider to
  test that expected results are returned when using
  a valid API Key (commented example added to
  phpunit.xml.dist and README.md) and that an
  InvalidCredentialsException is thrown when an
  incorrect API Key is used.
@willdurand
Copy link
Member

looks good to me

@toin0u
Copy link
Member

toin0u commented May 21, 2014

👍

willdurand added a commit that referenced this pull request May 21, 2014
…eption_with_bad_google_maps_key

Throw InvalidCredentialsException with GoogleMBP
@willdurand willdurand merged commit 1b385bb into geocoder-php:master May 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants