GitLab

- rework the tests strategy with Fred feedback. Now we test passing parameters and giving back results.
- some code refactoring for improving reusability
- add tests for addAccountExplicitly() and removeAccount()
- add test for getAccounts() and getAccountsByType()
- add test for getAuthenticatorTypes()
- take care of Fred second review (except the one with the callback parameter):
	- final for all constants
	- use of mX for member definition
	- improve "if" spacing
	- eliminate reference of the MockAuthenticator into the MockAuthenticatorService
- add test for addAccount() with a callback
- add test for removeAccount() with a callback
- fix non cleaned state for testGetAccountsAndGetAccountsByType()
- improve readability
- add test for setPassword() and getPassword() and clearPassword()
- add test for getUserData() and setUserData()
- add test for getAccountsByTypeAndFeatures()
- suppress non needed interface for constants and migrate them to the test itself
- fix typo
- rework test for getAccountsByTypeAndFeatures()
- fix typo
- add test for setAuthToken() and peekAuthToken() and invalidateAuthToken()
- add test for blockingGetAuthToken()
- improve addAccount() test with checking Options
- add more tests for both getAuthToken() signatures
- add test for confirmCredentials()
- add test for updateCredential()
- add test for editProperties()
- add test for addOnAccountsUpdatedListener()
- add test for removeOnAccountsUpdatedListener()
- code refactoring for suppressing redundencies
- fix missing parenthesis
- take care of Brett second review:
	- use of @Override when needed
	- do not catch exception so that the JUnit runner can deal with them (except for callback as we
	cannot change the API
	- add Javadoc for public methods
	- HAVE NOT changed package name to "com.android.cts.accounts" as Brett and I could not find why the
	tests where blocking
- suppress non needed dumpsys-packages.txt
- add super.setUp() and super.tearDown() calls
- suppressing trailing whitespaces
- take care of Fred second review:
	- renamed LATCH_TIMEOUT to LATCH_TIMEOUT_MS
	- renamed validateAccount() to validateAccountAndAuthTokenType()
	- do not rethrow exceptions when not needed (because taken care by the test framework)
	- confirmed that some retrieved accounts are correct
	- no need to confirm that the callback is called because of the latch itself
- suppress trailing whitespaces again
- fix bug #2342468 (android.accounts.cts.AccountManagerTest#testAddAndRemoveAccount fails if another account is present)
- refactor tests to take care about handlers
- add test for UpdateCredentials() with callback and handler
- take care of Fred third review
	- method renaming
	- code formatting
	- use of array instead of list
	- the comment was correct.
	- add one missing assert
	- suppressed duplicate comment
- fix label and change account authenticator icon for using the Android standard one

Change-Id: I84661fdc108943693947f807dd3f97837cac2a0f