Skip to content
This repository has been archived by the owner on Dec 21, 2020. It is now read-only.

How to test #2

Open
boldtrn opened this issue Aug 21, 2018 · 3 comments
Open

How to test #2

boldtrn opened this issue Aug 21, 2018 · 3 comments
Labels
question Further information is requested

Comments

@boldtrn
Copy link
Member

boldtrn commented Aug 21, 2018

How do we make this repository testable?

Before we would start a webserver and send requests and look at the responses. Should we use a similar approach here? If yes, we need to be able to start a server in this repo, otherwise we might only test the MapboxResponseConverter, but then we still need to start a GH instance.

@boldtrn boldtrn added the question Further information is requested label Aug 21, 2018
@karussell
Copy link
Member

If yes, we need to be able to start a server in this repo, otherwise we might only test the MapboxResponseConverter, but then we still need to start a GH instance

I see what you mean. It is probably not wrong to have it as a test dependency or with a separate maven module.

BTW: can we rename graphhopper-mapbox to graphhopper-navigation-mapbox? Although longer it better describes the package IMO.

@boldtrn
Copy link
Member Author

boldtrn commented Aug 21, 2018

BTW: can we rename graphhopper-mapbox to graphhopper-navigation-mapbox? Although longer it better describes the package IMO.

Ok sure, I can do that :). My idea was to call the overall package graphhopper-navigation and add different modules for specific conversions, so graphhopper-mapbox is a module of graphhopper-navigation, but I agree, it would make more sense to rename it as well :).

@boldtrn
Copy link
Member Author

boldtrn commented Aug 22, 2018

With #3 this should be mostly fixed. I still think we should somehow test the actual server execution to make sure that MapboxResource is tested as well, but at least we have an easy way to test the converter itself.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants