Fix set user state add none existed authData bug

Author: wangmengyan95

Parse/src/main/java/com/parse/ParseUser.java

@@ -510,7 +510,7 @@ public Task<Void> then(Task<Void> task) throws Exception {
         newStateBuilder.put(KEY_SESSION_TOKEN, getSessionToken());
       }
       // Avoid clearing authData when updating the current user's State via ParseQuery result
-      if (getAuthData() != null && newState.get(KEY_AUTH_DATA) == null) {
+      if (getAuthData().size() > 0 && newState.get(KEY_AUTH_DATA) == null) {
         newStateBuilder.put(KEY_AUTH_DATA, getAuthData());
       }
       newState = newStateBuilder.build();

Parse/src/test/java/com/parse/ParseUserTest.java

@@ -1435,6 +1435,35 @@ public void testSetCurrentUserStateWithoutAuthData() throws Exception {
     assertEquals("valueAgain", user.get("key"));
   }
 
+  @Test
+  public void testSetNoAuthDataCurrentUserStateWithoutAuthData() throws Exception {
+    // Set user initial state
+    ParseUser.State userState = new ParseUser.State.Builder()
+        .objectId("test")
+        .put("oldKey", "oldValue")
+        .put("key", "value")
+        .build();
+    ParseUser user = ParseObject.from(userState);
+    user.setIsCurrentUser(true);
+    // Build new state
+    ParseUser.State newUserState = new ParseUser.State.Builder()
+        .objectId("testAgain")
+        .put("key", "valueAgain")
+        .build();
+
+    user.setState(newUserState);
+
+    // Make sure we do not add none existed authData
+    assertFalse(user.keySet().contains("authData"));
+    assertEquals(1, user.keySet().size());
+    assertEquals(0, user.getAuthData().size());
+    // Make sure old state is deleted
+    assertFalse(user.has("oldKey"));
+    // Make sure new state is set
+    assertEquals("testAgain", user.getObjectId());
+    assertEquals("valueAgain", user.get("key"));
+  }
+
   //endregion
 
   private static void setLazy(ParseUser user) {