CognitoUser.deleteUser(GenericHandler handler) doesn't call handler for success.

[AngusMorton]

In the Cognito Identity Provider, the deleteUser method does not call the provided GenericHandler when it has succeeded.

public void deleteUser(GenericHandler callback) {
  if (callback == null) {
    throw new CognitoParameterInvalidException("callback is null");
  }
  
  try {
    deleteUserInternal(this.getCachedSession());
  } catch (final Exception e) {
    callback.onFailure(e);
  }
}

Should probably be:

public void deleteUser(GenericHandler callback) {
  if (callback == null) {
    throw new CognitoParameterInvalidException("callback is null");
  }
  
  try {
    deleteUserInternal(this.getCachedSession());
    callback.onSuccess();
  } catch (final Exception e) {
    callback.onFailure(e);
  }
}

This would make the API consistent with deleteUserInBackground.

[mutablealligator]

Hi Angus,

Thank you for pointing this out. We will take a look at it.

[mutablealligator]

Fixed in 2.6.5 release.