WIP for test strategy #89
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
shoatman
commented
May 24, 2021
shoatman
commented
- Added test strategy document
|
Looks like some duplicated content from what is declared here: https://github.com/AzureAD/android-complete/blob/master/docs/Automation/testing_overview.md#android-auth-library-testing-philosophy Can we just keep all testing / automation docs in the existing automation folder? https://github.com/AzureAD/android-complete/tree/a430a4566336d3175c4a650d6dea9db5dd53342e/docs/Automation I personally also like to maintain an index file for easy linking: https://github.com/AzureAD/android-complete/blob/master/docs/Automation/index.md |
| - GetAccounts (MSAL) / GetUsers (ADAL) - Enumerate all accounts currently available to the client app. If the broker is not installed then the request is to the client local cache. If the broker is installed then account returned from the broker are unioned with accounts in the local cache. | ||
| - GetAccount (MSAL) / GetUser (ADAL) - Get a specific account from from the library OR broker if installed. Will check the local cache for the specified account by ID (upn/object id, etc...). If not in the local cache will also check the broker. | ||
| - Tenant Profiles (MSAL Only) for MSAL accounts we allow the user to access tenant profiles. These are tenant specific claims from the id_tokens associated with requests to different tenants. | ||
| - GetCurrentAccount (MSAL Single Account PCA Only) - Gets the current account from the local cache when the broker is not installed. Get the current account from the broker when installed. The account returned may be different (different account or null) for the current account. Null meaning that no account should be signed in currently. A different account meaning that |
There was a problem hiding this comment.
A different account meaning that
Probably info left to fill here.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.