Add API for searching addresses via query

Author: greenhalos

.mvn/wrapper/maven-wrapper.properties

@@ -1 +1 @@
-distributionUrl=https://repo1.maven.org/maven2/org/apache/maven/apache-maven/3.3.9/apache-maven-3.3.9-bin.zip
+distributionUrl=https://repo1.maven.org/maven2/org/apache/maven/apache-maven/3.5.2/apache-maven-3.5.2-bin.zip

src/main/java/net/contargo/iris/address/api/SimpleAddressApiController.java

@@ -0,0 +1,46 @@
+package net.contargo.iris.address.api;
+
+import com.wordnik.swagger.annotations.Api;
+import com.wordnik.swagger.annotations.ApiOperation;
+
+import net.contargo.iris.address.dto.AddressDto;
+import net.contargo.iris.address.dto.AddressDtoService;
+
+import org.springframework.beans.factory.annotation.Autowired;
+
+import org.springframework.stereotype.Controller;
+
+import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RequestParam;
+
+import java.util.List;
+
+import static org.springframework.web.bind.annotation.RequestMethod.GET;
+
+
+/**
+ * @author  Ben Antony - antony@synyx.de
+ * @author  Sandra Thieme - thieme@synyx.de
+ */
+@Controller
+@Api(description = "API for querying addresses.", value = "")
+public class SimpleAddressApiController {
+
+    private final AddressDtoService nominatimAddressService;
+
+    @Autowired
+    public SimpleAddressApiController(AddressDtoService nominatimAddressService) {
+
+        this.nominatimAddressService = nominatimAddressService;
+    }
+
+    @ApiOperation(
+        value = "Returns a list of matching addresses.",
+        notes = "Can be static addresses or nomniatim resolved addresses."
+    )
+    @RequestMapping(value = "/addresses/simpleaddress", method = GET)
+    public List<AddressDto> getAddresses(@RequestParam("query") String query) {
+
+        return nominatimAddressService.getAddressesByQuery(query);
+    }
+}

src/main/java/net/contargo/iris/address/dto/AddressDtoService.java

@@ -46,8 +46,8 @@ public interface AddressDtoService {
 
     /**
      * Resolves an address (described by the given parameters) to a {@link java.util.List} of
-     * {@link net.contargo.iris.address.Address} objects with the attributes name, latitude and longitude. Uses multiple
-     * fallback strategies to find addresses if not all parameters are provided
+     * {@link net.contargo.iris.address.Address} objects with the attributes name, latitude and longitude. Uses
+     * multiple fallback strategies to find addresses if not all parameters are provided
      *
      * @param  addressDetails  The parameters describing the addresses we are looking for
      *
@@ -58,8 +58,8 @@ public interface AddressDtoService {
 
     /**
      * Resolves an address (described by the given parameters) to a {@link java.util.List} of
-     * {@link net.contargo.iris.address.Address} objects with the attributes name, latitude and longitude. Uses multiple
-     * fallback strategies to find addresses if not all parameters are provided
+     * {@link net.contargo.iris.address.Address} objects with the attributes name, latitude and longitude. Uses
+     * multiple fallback strategies to find addresses if not all parameters are provided
      *
      * @param  addressDetails  The parameters describing the addresses we are looking for
      *
@@ -86,4 +86,14 @@ public interface AddressDtoService {
      * @return  the {@link AddressDto} with the given hashKey
      */
     AddressDto getAddressesByHashKey(String hashKey);
+
+
+    /**
+     * Returns a list of addresses matching the query.
+     *
+     * @param  query  the address query
+     *
+     * @return  a list of matching addresses
+     */
+    List<AddressDto> getAddressesByQuery(String query);
 }

src/main/java/net/contargo/iris/address/dto/AddressDtoServiceImpl.java

@@ -8,9 +8,9 @@
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
-import java.util.stream.Collectors;
 
 import static java.util.Collections.singletonList;
+import static java.util.stream.Collectors.toList;
 
 
 /**
@@ -64,7 +64,7 @@ public List<AddressListDto> getAddressesByDetails(Map<String, String> addressDet
         return addressServiceWrapper.getAddressesByDetails(addressDetails)
             .stream()
             .map(AddressListDto::new)
-            .collect(Collectors.toList());
+            .collect(toList());
     }
 
 
@@ -85,10 +85,7 @@ public List<AddressDto> getAddressesByDetailsPlain(Map<String, String> addressDe
     @Override
     public List<AddressDto> getAddressesWherePlaceIsIn(Long placeId) {
 
-        return addressService.getAddressesWherePlaceIsIn(placeId)
-            .stream()
-            .map(AddressDto::new)
-            .collect(Collectors.toList());
+        return addressService.getAddressesWherePlaceIsIn(placeId).stream().map(AddressDto::new).collect(toList());
     }
 
 
@@ -97,4 +94,11 @@ public AddressDto getAddressesByHashKey(String hashKey) {
 
         return new AddressDto(addressServiceWrapper.getByHashKey(hashKey));
     }
+
+
+    @Override
+    public List<AddressDto> getAddressesByQuery(String query) {
+
+        return addressServiceWrapper.getAddressesByQuery(query).stream().map(AddressDto::new).collect(toList());
+    }
 }

src/main/java/net/contargo/iris/address/nominatim/service/AddressService.java

@@ -14,13 +14,15 @@
  * @author  Sven Mueller - mueller@synyx.de
  * @author  Aljona Murygina - murygina@synyx.de
  * @author  Arnold Franke - franke@synyx.de
+ * @author  Sandra Thieme - thieme@synyx.de
+ * @author  Ben Antony - antony@synyx.de
  */
 public interface AddressService {
 
     /**
      * Resolves an address (described by the given parameters) to a {@link java.util.List} of
-     * {@link net.contargo.iris.address.Address} objects with the attributes name, latitude and longitude. Uses multiple
-     * fallback strategies to find addresses if not all parameters are provided
+     * {@link net.contargo.iris.address.Address} objects with the attributes name, latitude and longitude. Uses
+     * multiple fallback strategies to find addresses if not all parameters are provided
      *
      * @param  addressDetails  @return
      */
@@ -51,4 +53,14 @@ public interface AddressService {
      * @return  The address for the given geolocation.
      */
     Address getAddressByGeolocation(GeoLocation location);
+
+
+    /**
+     * Returns a list of addresses matching the query.
+     *
+     * @param  query  the address query
+     *
+     * @return  a list of matching addresses
+     */
+    List<Address> getAddressesByQuery(String query);
 }

src/main/java/net/contargo/iris/address/nominatim/service/NominatimAddressService.java

@@ -120,6 +120,15 @@ public Address getAddressByGeolocation(GeoLocation geoLocation) {
     }
 
 
+    @Override
+    public List<Address> getAddressesByQuery(String query) {
+
+        String url = nominatimUrlBuilder.buildSearchUrl(query);
+
+        return nominatimResponder.getAddresses(url);
+    }
+
+
     private List<Address> searchSuburbsViaNominatimsDetailPage(Long osmPlaceId, SuburbType suburbType,
         Set<String> suburbGlobalDisplayNames) {
 
@@ -190,7 +199,7 @@ public List<Address> getAddressesByDetails(Map<String, String> addressDetails) {
     }
 
 
-    private String[][] createResolvingStrategies(String postalCode, String city, String country, String name,
+    private static String[][] createResolvingStrategies(String postalCode, String city, String country, String name,
         String internStreet) {
 
         return new String[][] {

src/main/java/net/contargo/iris/address/nominatim/service/NominatimUrlBuilder.java

@@ -174,4 +174,10 @@ String buildUrl(GeoLocation geoLocation) {
 
         return url;
     }
+
+
+    public String buildSearchUrl(String query) {
+
+        return baseUrl + "search/" + query + "?format=json&addressdetails=1";
+    }
 }

src/main/java/net/contargo/iris/address/service/AddressServiceWrapper.java

@@ -175,7 +175,7 @@ private List<AddressList> resolveByNominatim(Map<String, String> addressDetails)
     }
 
 
-    private List<AddressList> getSimpleAddressList(List<Address> addresses) {
+    private static List<AddressList> getSimpleAddressList(List<Address> addresses) {
 
         List<AddressList> addressListList = new ArrayList<>();
 
@@ -189,4 +189,10 @@ private List<AddressList> getSimpleAddressList(List<Address> addresses) {
 
         return addressListList;
     }
+
+
+    public List<Address> getAddressesByQuery(String query) {
+
+        return addressService.getAddressesByQuery(query);
+    }
 }

src/test/java/net/contargo/iris/address/api/SimpleAddressApiControllerUnitTest.java

@@ -0,0 +1,71 @@
+package net.contargo.iris.address.api;
+
+import net.contargo.iris.address.Address;
+import net.contargo.iris.address.dto.AddressDto;
+import net.contargo.iris.address.dto.AddressDtoService;
+
+import org.junit.Before;
+import org.junit.Test;
+
+import org.junit.runner.RunWith;
+
+import org.springframework.beans.factory.annotation.Autowired;
+
+import org.springframework.test.context.ContextConfiguration;
+import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.web.WebAppConfiguration;
+import org.springframework.test.web.servlet.MockMvc;
+
+import org.springframework.web.context.WebApplicationContext;
+
+import static org.hamcrest.Matchers.hasSize;
+import static org.hamcrest.Matchers.is;
+
+import static org.mockito.Mockito.when;
+
+import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+import static org.springframework.test.web.servlet.setup.MockMvcBuilders.webAppContextSetup;
+
+import static java.util.Collections.singletonList;
+
+
+/**
+ * @author  Ben Antony - antony@synyx.de
+ * @author  Sandra Thieme - thieme@synyx.de
+ */
+@RunWith(SpringJUnit4ClassRunner.class)
+@ContextConfiguration(locations = { "classpath:public-api-context.xml" })
+@WebAppConfiguration
+public class SimpleAddressApiControllerUnitTest {
+
+    @Autowired
+    private WebApplicationContext webApplicationContext;
+    @Autowired
+    private AddressDtoService addressDtoServiceMock;
+
+    private MockMvc mockMvc;
+
+    @Before
+    public void setUp() {
+
+        mockMvc = webAppContextSetup(webApplicationContext).build();
+    }
+
+
+    @Test
+    public void getAddresses() throws Exception {
+
+        Address address = new Address();
+        address.setDisplayName("Gartenstr. 67, Karlsruhe (Südweststadt)");
+
+        when(addressDtoServiceMock.getAddressesByQuery("Gartenstraße 67, Karlsruhe")).thenReturn(singletonList(
+                new AddressDto(address)));
+
+        mockMvc.perform(get("/addresses/simpleaddress").param("query", "Gartenstraße 67, Karlsruhe"))
+            .andExpect(status().isOk())
+            .andExpect(jsonPath("$.addressDtoList", hasSize(1)))
+            .andExpect(jsonPath("$.addressDtoList[0].displayName", is("Gartenstr. 67, Karlsruhe (Südweststadt)")));
+    }
+}

src/test/java/net/contargo/iris/address/dto/AddressDtoServiceImplUnitTest.java

@@ -23,6 +23,7 @@
 
 import static org.hamcrest.MatcherAssert.assertThat;
 
+import static org.hamcrest.Matchers.hasSize;
 import static org.hamcrest.Matchers.is;
 import static org.hamcrest.Matchers.nullValue;
 
@@ -45,16 +46,17 @@ public class AddressDtoServiceImplUnitTest {
     private static final int OSM_ID = 1;
     private static final GeoLocation GEOLOCATION = new GeoLocation(BigDecimal.ONE, BigDecimal.ONE);
     private static final Long PLACE_ID = 1L;
+
     private final AddressService addressServiceMock = mock(AddressService.class);
     private final AddressServiceWrapper addressServiceWrapperMock = mock(AddressServiceWrapper.class);
     private final AddressDtoServiceImpl sut = new AddressDtoServiceImpl(addressServiceMock, addressServiceWrapperMock);
-    Address address = new Address(BigDecimal.ONE, BigDecimal.ONE);
-    private Map<String, String> addressDetails;
+
+    private final Address address = new Address(BigDecimal.ONE, BigDecimal.ONE);
+    private final Map<String, String> addressDetails = new HashMap<>();
 
     @Before
-    public void setUp() throws Exception {
+    public void setUp() {
 
-        addressDetails = new HashMap<>();
         addressDetails.put(CITY.getKey(), "city");
         addressDetails.put(STREET.getKey(), "street");
         addressDetails.put(POSTAL_CODE.getKey(), "postalcode");
@@ -175,4 +177,19 @@ public void getAddressesByHashKey() {
         AddressDto addressDto = sut.getAddressesByHashKey(hashKey);
         assertThat(addressDto.getDisplayName(), is(displayName));
     }
+
+
+    @Test
+    public void getAddressesByQuery() {
+
+        address.setDisplayName("Gartenstr. 67, Karlsruhe (Südweststadt)");
+
+        when(addressServiceWrapperMock.getAddressesByQuery("Gartenstraße 67, Karlsruhe")).thenReturn(singletonList(
+                address));
+
+        List<AddressDto> addresses = sut.getAddressesByQuery("Gartenstraße 67, Karlsruhe");
+
+        assertThat(addresses, hasSize(1));
+        assertThat(addresses.get(0).getDisplayName(), is("Gartenstr. 67, Karlsruhe (Südweststadt)"));
+    }
 }