Raise descriptive IllegalArgumentException when zipcode is - (#468)

Summary:
Fixes this issue - #460

> When executing a postal code validation on a String containing only "-" an ArrayIndexOutOfBoundsException is thrown.

Pull Request resolved: #468

Reviewed By: yoongyj

Differential Revision: D62392830

Pulled By: stcheng

fbshipit-source-id: ba84bf0dd10eb9006dbde0da966f09542857e2c1

src/main/java/com/facebook/ads/utils/ServerSideApiUtil.java

@@ -49,7 +49,7 @@ public static String hash(String input) {
     * Checks whether the input is already hashed with SHA256 or MD5.
     *
     * @param input value that need to be validated.
-    * @return bool representing the whether the input is hashed
+    * @return bool representing whether the input is hashed
     */
    public static boolean isAlreadyHashed(String input) {
 
@@ -170,9 +170,15 @@ private static String normalizePostalCode(String postalCode) {
      // Remove space characters in the Zip Code
      postalCode = postalCode.replaceAll("[\\s]+", "");
 
-     // If the code has more than one part, retain the first part.
-     postalCode = postalCode.split("-")[0];
+     // If the code is just a hyphen throw a descriptive IllegalArgumentException instead of
+     // ArrayIndexOutOfBoundsException below.
+     String[] postalCodeArray = postalCode.split("-");
+     if (postalCodeArray.length == 0) {
+       throw new IllegalArgumentException("Invalid postalcode for the passed postalCode: '" + postalCode + "'.");
+     }
 
+     // If the code has more than one part, retain the first part.
+     postalCode = postalCodeArray[0];
      if (postalCode.length() < 2) {
        throw new IllegalArgumentException("Invalid postalcode format for the passed postalCode:" + postalCode + ". Please check the passed postalcode.");
      }

src/test/java/com/facebook/ads/utils/ServerSideApiUtilTest.java

@@ -216,4 +216,11 @@ public void NormalizeSanitizesEmptyFieldsToNull() {
       assertNull(normalize(" \t   ", param));
     }
   }
-}
+
+  @Test
+  public void NormalizeHyphenPostalCodeThrowsIllegalArgumentException() {
+    thrown.expect(IllegalArgumentException.class);
+    thrown.expectMessage("Invalid postalcode for the passed postalCode: '-'.");
+    normalize("-", ZIP_CODE);
+  }
+}