Test null, empty and malformed data corner cases

Author: onufryk

core-api/src/main/java/com/optimizely/ab/Optimizely.java

@@ -498,13 +498,7 @@ private void track(@Nonnull String eventName,
      * @return a {@link ProjectConfig} instance given a json string
      */
     private static ProjectConfig getProjectConfig(String datafile) throws ConfigParseException {
-        ProjectConfig projectConfig;
-        try {
-            projectConfig = DefaultConfigParser.getInstance().parseProjectConfig(datafile);
-        } catch (Exception e) {
-            throw new ConfigParseException(e.getMessage(), e);
-        }
-        return projectConfig;
+        return DefaultConfigParser.getInstance().parseProjectConfig(datafile);
     }
 
     //======== Notification listeners ========//

core-api/src/test/java/com/optimizely/ab/OptimizelyBuilderTest.java

@@ -63,18 +63,19 @@ public class OptimizelyBuilderTest {
     @Mock private ErrorHandler mockErrorHandler;
 
     @Test(expected = ConfigParseException.class)
+    @SuppressFBWarnings(value="NP_NONNULL_PARAM_VIOLATION", justification="Testing nullness contract violation")
     public void withNullConfig() throws Exception {
-        Optimizely optimizelyClient = Optimizely.builder(null, mockEventHandler).build();
+        Optimizely.builder(null, mockEventHandler).build();
     }
 
     @Test(expected = ConfigParseException.class)
     public void withEmptyConfig() throws Exception {
-        Optimizely optimizelyClient = Optimizely.builder("", mockEventHandler).build();
+        Optimizely.builder("", mockEventHandler).build();
     }
 
     @Test(expected = ConfigParseException.class)
     public void withInvalidConfig() throws Exception {
-        Optimizely optimizelyClient = Optimizely.builder("malformed json", mockEventHandler).build();
+        Optimizely.builder("malformed json", mockEventHandler).build();
     }
 
     @Test