Merge pull request #259 from Sprokof/amadeus.builder-raise-IAE

replaced raise NPE by IAE In amadeus.builder

Author: tsolakoua

lombok.config

@@ -1 +1,2 @@
-lombok.addLombokGeneratedAnnotation = true
+lombok.addLombokGeneratedAnnotation = true
+lombok.nonNull.exceptionType = NullPointerException

src/main/java/com/amadeus/Amadeus.java

@@ -1,7 +1,6 @@
 package com.amadeus;
 
 import java.util.Map;
-import lombok.NonNull;
 
 /**
  * <p>
@@ -138,7 +137,13 @@ protected Amadeus(Configuration configuration) {
    * @param clientSecret Your API com.amadeus.client credential secret
    * @return a Configuration object
    */
-  public static Configuration builder(@NonNull String clientId, @NonNull String clientSecret) {
+  public static Configuration builder(String clientId, String clientSecret) {
+    if (clientId == null) {
+      throw new IllegalArgumentException("clientId can't be null");
+    }
+    if (clientSecret == null) {
+      throw new IllegalArgumentException("clientSecret can't be null");
+    }
     return new Configuration(clientId, clientSecret);
   }
 

src/main/java/com/amadeus/Configuration.java

@@ -2,6 +2,7 @@
 
 import java.util.Map;
 import java.util.logging.Logger;
+
 import lombok.Getter;
 import lombok.Setter;
 import lombok.ToString;
@@ -110,9 +111,9 @@ protected Configuration(String clientId, String clientSecret) {
    * Builds an Amadeus client with the provided credentials.
    *
    * @return an Amadeus client
-   * @throws NullPointerException when a client ID or client secret is missing
+   * @throws IllegalArgumentException when a client ID or client secret is missing
    */
-  public Amadeus build() throws NullPointerException {
+  public Amadeus build() throws IllegalArgumentException {
     return new Amadeus(this);
   }
 

src/test/java/com/amadeus/AmadeusTest.java

@@ -20,11 +20,11 @@ public class AmadeusTest {
   }
 
   @Test public void testBuilderWithNullClientId() {
-    assertThrows(NullPointerException.class, () -> Amadeus.builder(null, "secret").build());
+    assertThrows(IllegalArgumentException.class, () -> Amadeus.builder(null, "secret").build());
   }
 
   @Test public void testBuilderWithNullClientSecret() {
-    assertThrows(NullPointerException.class, () -> Amadeus.builder("client", null).build());
+    assertThrows(IllegalArgumentException.class, () -> Amadeus.builder("client", null).build());
   }
 
   @Test public void testBuilderWithEnvironment() {