Merge pull request #23 from Bandwidth/DX-666

Updated tollFreeWildCardPattern and quantity for search order requests

Author: dtolbert-bw

.travis.yml

@@ -1,3 +1,4 @@
+dist: trusty
 language: java
 jdk:
 - oraclejdk8

README.md

@@ -2,6 +2,12 @@
 
 Java SDK for Bandwidth's number management API.
 
+## Release Notes
+
+| Version | Notes |
+|:---|:---|
+| 0.3.0  | Fixed `quantity` and `tollFreeWildCardPattern` for search requests |
+
 ## Dependency
 
 ### Maven
@@ -58,6 +64,7 @@ AvailableNumberSearchRequest availableNumberSearchRequest = AvailableNumberSearc
    .city("CARY")
    .enableTNDetail(false)
    .quantity(10)
+   .tollFreeWildCardPattern("8**")
    .build();
 
 SearchResult searchResult = numbersClient.getAvailableTelephoneNumbers(availableNumberSearchRequest);
@@ -126,4 +133,4 @@ public class MyAwesomeBandwidthNumbersApp {
           }
        }
 }
-```
+```

build.gradle

@@ -51,7 +51,7 @@ publishing {
         maven(MavenPublication) {
             groupId = 'com.bandwidth.sdk'
             artifactId = 'numbers'
-            version = '0.2.0-SNAPSHOT'
+            version = '0.4.0-SNAPSHOT'
             from components.java
         }
     }
@@ -98,7 +98,7 @@ if (project.hasProperty("signing.keyId")) {
     // Configure group ID, artifact ID, and version
     group = "com.bandwidth.sdk"
     archivesBaseName = "numbers"
-    version = "0.2.0-SNAPSHOT"
+    version = "0.4.0-SNAPSHOT"
 
     // Build, sign, and upload
     uploadArchives {

src/main/java/com/bandwidth/sdk/numbers/models/AvailableNumberSearchRequest.java

@@ -52,7 +52,7 @@ public abstract class AvailableNumberSearchRequest {
    public abstract String getTollFreeVanity();
 
    @Nullable
-   public abstract String getTollFreeVanityWildCardPattern();
+   public abstract String getTollFreeWildCardPattern();
 
    @Nullable
    public abstract Boolean getEnableTNDetail();
@@ -67,11 +67,7 @@ public abstract class AvailableNumberSearchRequest {
    public abstract OrderBy getOrderBy();
 
    @Nullable
-   @Value.Default
-   @JacksonXmlProperty(localName = "Quantity")
-   public Integer getQuantity() {
-      return 1;
-   }
+   public abstract Integer getQuantity();
 
    public static ImmutableAvailableNumberSearchRequest.Builder builder() {
       return ImmutableAvailableNumberSearchRequest.builder();

src/test/java/com/bandwidth/sdk/numbers/NumbersClientImplTest.java

@@ -43,6 +43,7 @@ public class NumbersClientImplTest {
       .state("NC")
       .city("RALEIGH")
       .quantity(5)
+      .tollFreeWildCardPattern("8**")
       .build();
 
    private SearchResult searchResult = ImmutableSearchResult.builder()
@@ -178,4 +179,4 @@ public void orderNumbersWithError() {
       assertThat(thrown).isInstanceOf(NumbersApiException.class);
    }
 
-}
+}

src/test/java/com/bandwidth/sdk/numbers/serde/NumbersSerdeTest.java

@@ -12,6 +12,8 @@ public class NumbersSerdeTest {
    private AvailableNumberSearchRequest availableNumberSearchRequest = AvailableNumberSearchRequest.builder()
       .city("RALEIGH")
       .state("NC")
+      .tollFreeWildCardPattern("8**")
+      .quantity(5)
       .enableTNDetail(true)
       .build();