add missing empty list tests

Author: lbalmaceda

build.gradle

@@ -22,7 +22,7 @@ jacocoTestReport {
 
 test {
     testLogging {
-        events "passed", "skipped", "failed", "standardError"
+        events "skipped", "failed"
         exceptionFormat "short"
     }
 }

src/main/java/com/auth0/AuthAPI.java

@@ -41,7 +41,7 @@ public AuthAPI(String domain, String clientId, String clientSecret) {
         this.clientSecret = clientSecret;
 
         HttpLoggingInterceptor logging = new HttpLoggingInterceptor();
-        logging.setLevel(Level.BODY);
+        logging.setLevel(Level.NONE);
         client = new OkHttpClient.Builder()
                 .addInterceptor(logging)
                 .build();
@@ -100,7 +100,8 @@ public Request<UserInfo> userInfo(String accessToken) {
                 .addPathSegment("userinfo")
                 .build()
                 .toString();
-        CustomRequest<UserInfo> request = new CustomRequest<>(client, url, "GET", new TypeReference<UserInfo>() {});
+        CustomRequest<UserInfo> request = new CustomRequest<>(client, url, "GET", new TypeReference<UserInfo>() {
+        });
         request.addHeader("Authorization", "Bearer " + accessToken);
         return request;
     }

src/main/java/com/auth0/MgmtAPI.java

@@ -30,7 +30,7 @@ public MgmtAPI(String domain, String apiToken) {
         this.apiToken = apiToken;
 
         HttpLoggingInterceptor logging = new HttpLoggingInterceptor();
-        logging.setLevel(HttpLoggingInterceptor.Level.BODY);
+        logging.setLevel(HttpLoggingInterceptor.Level.NONE);
         client = new OkHttpClient.Builder()
                 .addInterceptor(logging)
                 .build();

src/main/java/com/auth0/json/auth/UserInfo.java

@@ -2,11 +2,13 @@
 
 import com.fasterxml.jackson.annotation.JsonAnyGetter;
 import com.fasterxml.jackson.annotation.JsonAnySetter;
+import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonInclude;
 
 import java.util.HashMap;
 import java.util.Map;
 
+@JsonIgnoreProperties(ignoreUnknown = true)
 @JsonInclude(JsonInclude.Include.NON_NULL)
 public class UserInfo {
 

src/test/java/com/auth0/MgmtAPITest.java

@@ -1659,6 +1659,18 @@ public void shouldGetUserGuardianEnrollments() throws Exception {
         assertThat(response, is(notNullValue()));
     }
 
+    @Test
+    public void shouldReturnEmptyUserGuardianEnrollments() throws Exception {
+        Request<List<GuardianEnrollment>> request = api.getUserGuardianEnrollments("1");
+        assertThat(request, is(notNullValue()));
+
+        server.jsonResponse(MGMT_EMPTY_LIST, 200);
+        List<GuardianEnrollment> response = request.execute();
+
+        assertThat(response, is(notNullValue()));
+        assertThat(response, is(emptyCollectionOf(GuardianEnrollment.class)));
+    }
+
     @Test
     public void shouldThrowOnGetUserLogEventsWithNullId() throws Exception {
         exception.expect(IllegalArgumentException.class);
@@ -1776,8 +1788,6 @@ public void shouldReturnEmptyUserLogEvents() throws Exception {
         assertThat(response.getItems(), is(emptyCollectionOf(LogEvent.class)));
     }
 
-    //TODO: Add empty arrays validation
-
     @Test
     public void shouldThrowOnDeleteUserMultifactorProviderWithNullId() throws Exception {
         exception.expect(IllegalArgumentException.class);
@@ -1958,6 +1968,18 @@ public void shouldGetBlacklistedTokens() throws Exception {
         assertThat(response, hasSize(2));
     }
 
+    @Test
+    public void shouldReturnEmptyBlacklistedTokens() throws Exception {
+        Request<List<Token>> request = api.getBlacklistedTokens("myapi");
+        assertThat(request, is(notNullValue()));
+
+        server.jsonResponse(MGMT_EMPTY_LIST, 200);
+        List<Token> response = request.execute();
+
+        assertThat(response, is(notNullValue()));
+        assertThat(response, is(emptyCollectionOf(Token.class)));
+    }
+
     @Test
     public void shouldThrowOnBlacklistTokensWithNullToken() throws Exception {
         exception.expect(IllegalArgumentException.class);
@@ -2196,6 +2218,18 @@ public void shouldListGuardianFactors() throws Exception {
         assertThat(response, hasSize(2));
     }
 
+    @Test
+    public void shouldReturnEmptyGuardianFactors() throws Exception {
+        Request<List<GuardianFactor>> request = api.listGuardianFactors();
+        assertThat(request, is(notNullValue()));
+
+        server.jsonResponse(MGMT_EMPTY_LIST, 200);
+        List<GuardianFactor> response = request.execute();
+
+        assertThat(response, is(notNullValue()));
+        assertThat(response, is(emptyCollectionOf(GuardianFactor.class)));
+    }
+
     @Test
     public void shouldThrowOnUpdateGuardianFactorWithNullName() throws Exception {
         exception.expect(IllegalArgumentException.class);

src/test/java/com/auth0/json/auth/UserInfoTest.java

@@ -9,16 +9,13 @@
 
 public class UserInfoTest extends JsonTest<UserInfo> {
 
-    private static final String json = "{\"email_verified\":false,\"clientID\":\"q2hnj2iu...\",\"updated_at\":\"2016-12-05T15:15:40.545Z\",\"name\":\"test.account@userinfo.com\",\"email\":\"test.account@userinfo.com\"}";
+    private static final String json = "{\"email\":\"test.account@userinfo.com\",\"client_id\":\"q2hnj2iu...\"}";
 
     @Test
     public void shouldSerialize() throws Exception {
         UserInfo info = new UserInfo();
-        info.setValue("email_verified", false);
         info.setValue("email", "test.account@userinfo.com");
-        info.setValue("clientID", "q2hnj2iu...");
-        info.setValue("updated_at", "2016-12-05T15:15:40.545Z");
-        info.setValue("name", "test.account@userinfo.com");
+        info.setValue("client_id", "q2hnj2iu...");
 
         String serialized = toJSON(info);
         assertThat(serialized, is(notNullValue()));
@@ -31,10 +28,7 @@ public void shouldDeserialize() throws Exception {
 
         assertThat(info, is(notNullValue()));
 
-        assertThat(info.getValues(), IsMapContaining.hasEntry("email_verified", (Object) false));
         assertThat(info.getValues(), IsMapContaining.hasEntry("email", (Object) "test.account@userinfo.com"));
-        assertThat(info.getValues(), IsMapContaining.hasEntry("clientID", (Object) "q2hnj2iu..."));
-        assertThat(info.getValues(), IsMapContaining.hasEntry("updated_at", (Object) "2016-12-05T15:15:40.545Z"));
-        assertThat(info.getValues(), IsMapContaining.hasEntry("name", (Object) "test.account@userinfo.com"));
+        assertThat(info.getValues(), IsMapContaining.hasEntry("client_id", (Object) "q2hnj2iu..."));
     }
 }

src/test/java/com/auth0/json/mgmt/client/AddonTest.java

@@ -9,13 +9,12 @@
 
 public class AddonTest extends JsonTest<Addon> {
 
-    private static final String json = "{\"a-boolean\":true,\"a-string\":\"text\"}";
+    private static final String json = "{\"a-boolean\":true}";
 
     @Test
     public void shouldSerialize() throws Exception {
         Addon addon = new Addon();
         addon.setProperty("a-boolean", true);
-        addon.setProperty("a-string", "text");
 
         String serialized = toJSON(addon);
         assertThat(serialized, is(notNullValue()));
@@ -29,8 +28,6 @@ public void shouldDeserialize() throws Exception {
         assertThat(addon, is(notNullValue()));
 
         assertThat(addon.getProperties(), IsMapContaining.hasEntry("a-boolean", (Object) true));
-        assertThat(addon.getProperties(), IsMapContaining.hasEntry("a-string", (Object) "text"));
         assertThat(addon.getProperty("a-boolean"), is((Object) true));
-        assertThat(addon.getProperty("a-string"), is((Object) "text"));
     }
 }