Gisgraphy tests fail due to missing key

[karussell]

We could put our API key for gisgraphy in the hidden section here at github, but highly likely it will be exposed in the logging. Or should we temporary disable gisgraphy tests for travis?

[gisgraphy]

I suggest using the demo server https://services.gisgraphy.com/

[karussell]

It would still fail:

testHandleReverse(com.graphhopper.converter.resource.ConverterResourceGisgraphyTest): expected:<[2]00> but was:<[4]00>
testHandleForward(com.graphhopper.converter.resource.ConverterResourceGisgraphyTest): expected:<[2]00> but was:<[4]00>

[gisgraphy]

I suggest that you put a user-agent that I accept on server side on the demo server, as I have done on this PR : DenisCarriere/geocoder@a0248e3

what about 'geocoder-converter'

[karussell]

Ok

[gisgraphy]

Please add a user agent only for tests (something like a variable only feed in tests) I don t want to white list all the users of the geocoder converter 😊

[boldtrn]

Please add a user agent only for tests (something like a variable only feed in tests) I don t want to white list all the users of the geocoder converter blush

Wouldn't it be easier to create a very limited key for this purpose? Changing the user-agent is a possible solution, but since we make this transparent, every user could just switch to the geocoding-converter user agent as well.

Anyway changing the user agent just for tests seems to be not straight forward. We could add a config variable to change the user-agent (maybe we do this globally for all providers or only gisgraphy). Then we would set geocoder-converter as user-agent in the test converter.yml. This also requires a change of ConverterResourceGisgraphy where we add the user-agent to every request to the server. This means that also production requests get a user-agent (can be different from the test user-agent though).

[gisgraphy]

That 's I got in mind too :-). Just put a user-agent different for the tests.

what about 'geocoder-converter' for the "main software" and 'geocoder-converter' for tests?