Removed source from lookup constructor in US Reverse Geo.

smartystreets · Jun 27, 2023 · 30ab755 · 30ab755
1 parent cd8dfbe
commit 30ab755
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/src/main/java/com/smartystreets/api/us_reverse_geo/Client.java b/src/main/java/com/smartystreets/api/us_reverse_geo/Client.java
@@ -24,7 +24,7 @@ public void send(Lookup lookup) throws SmartyException, IOException, Interrupted
  DecimalFormat decimalFormat = new DecimalFormat("#.########");
  request.putParameter("latitude", decimalFormat.format(lookup.getLatitude()));
  request.putParameter("longitude", decimalFormat.format(lookup.getLongitude()));
- request.putParameter("source", getSource();
+ request.putParameter("source", getSource());
 
  Response httpResponse = this.sender.send(request);
 

diff --git a/src/main/java/com/smartystreets/api/us_reverse_geo/Lookup.java b/src/main/java/com/smartystreets/api/us_reverse_geo/Lookup.java
@@ -16,11 +16,10 @@ public class Lookup implements Serializable {
 
  public Lookup() { this.response = new SmartyResponse(); }
 
- public Lookup(double latitude, double longitude, String source) {
+ public Lookup(double latitude, double longitude) {
  this();
  this.latitude = latitude;
  this.longitude = longitude;
- this.source = source;
  }
 
  public Double getLatitude() { return this.latitude; }
@@ -29,6 +28,8 @@ public Lookup(double latitude, double longitude, String source) {
 
  public String getSource() { return this.source; }
 
+ public void setSource(String source) { this.source = source; }
+
  public SmartyResponse getResponse() { return this.response; }
 
  public void setResponse(SmartyResponse response) {

diff --git a/src/test/java/com/smartystreets/api/us_reverse_geo/ClientTest.java b/src/test/java/com/smartystreets/api/us_reverse_geo/ClientTest.java
@@ -20,7 +20,7 @@ public void testAddressLookupSerializedAndSentWithContext() throws Exception {
  Lookup lookup = new Lookup(44.888888888, -111.111111111);
  client.send(lookup);
 
- assertEquals("http://localhost/lookup?latitude=44.88888889&longitude=-111.11111111", capturingSender.getRequest().getUrl());
+ assertEquals("http://localhost/lookup?latitude=44.88888889&longitude=-111.11111111&source=", capturingSender.getRequest().getUrl());
  }
 
  //endregion