bug fix + test

graphhopper · Sep 4, 2023 · 9923e70 · 9923e70
1 parent b50f250
commit 9923e70
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/main/java/com/graphhopper/converter/resources/ConverterResourcePhoton.java b/src/main/java/com/graphhopper/converter/resources/ConverterResourcePhoton.java
@@ -49,7 +49,7 @@ public Response handle(@QueryParam("q") @DefaultValue("") String query,
         WebTarget target;
         if (reverse) {
             target = buildReverseTarget();
-            target.queryParam("radius", radius);
+            target = target.queryParam("radius", radius);
         } else {
             target = buildForwardTarget(query);
         }

diff --git a/src/test/java/com/graphhopper/converter/resource/ConverterResourcePhotonTest.java b/src/test/java/com/graphhopper/converter/resource/ConverterResourcePhotonTest.java
@@ -115,6 +115,18 @@ public void osmTags() {
         assertEquals("battlefield", entry.getHits().get(0).getOsmValue());
     }
 
+    @Test
+    public void testReverseWithOSMTags() {
+        Response response = client.target(String.format("http://localhost:%d/photon?point=40.694632,-74.097403&reverse=true&osm_tag=aeroway:aerodrome&radius=30", RULE.getLocalPort()))
+                .request()
+                .get();
+
+        assertThat(response.getStatus()).isEqualTo(200);
+
+        GHResponse entry = response.readEntity(GHResponse.class);
+        assertEquals("Newark Liberty International Airport", entry.getHits().get(0).getName());
+    }
+
     @Test
     public void testCorrectLocale() {
         Response response = client.target(