[Communication]: Added identity exception tests #18191
Merged
Conversation
This file contains 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
ghost
added
the
|
/azp run python - communication - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
lsundaralingam
requested review from
jbeauregardb,
beltr0n,
sacheu and
minnieliu
lsundaralingam
requested review from
ankitarorabit,
memontic-ms and
a team
as code owners
jbeauregardb
approved these changes
Apr 21, 2021
| uri: https://sanitized.communication.azure.com/identities/sanitized/:issueAccessToken?api-version=2021-03-07 | ||
| response: | ||
| body: | ||
| string: '{"error": {"code": "ValidationError", "message": "Scopes: The Scopes |
There was a problem hiding this comment.
What is the motivation for testing this? scopes is a mandatory parameter of this route:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/communication/data-plane/Microsoft.CommunicationServicesIdentity/stable/2021-03-07/CommunicationIdentity.json#L265
There was a problem hiding this comment.
This is to make sure that we are indeed getting back a proper error and handling it when passing in an invalid set of input.
iscai-msft
added a commit
to iscai-msft/azure-sdk-for-python
that referenced
this pull request
Apr 22, 2021
…into azure_purview_catalog * 'master' of https://github.com/Azure/azure-sdk-for-python: (205 commits) update FormField docstring (Azure#18226) [Event Hubs] combine conn str parsing logic (Azure#18059) [Blob]Edit Test for Single Thread Substream Retry (Azure#18206) maintenance changelog config (Azure#18216) Privated hierarchy traversal and fixed encryption algorithm (Azure#18212) Remove duplicate calls to bypass-local-dns.yml (Azure#18180) Patch changelog (Azure#18188) enable polling by default (Azure#18184) [Communication]: Added identity exception tests (Azure#18191) version (Azure#18202) [AutoRelease] t2-maintenance-2021-04-20-69227 (Azure#18166) Updated identity samples (Azure#18189) Sync eng/common directory with azure-sdk-tools for PR 1565 (Azure#18155) Clean Up PR Triggers (Azure#18186) [Tables] on a 413 throw a RequestEntity Too Large Error (Azure#18181) Add environment.yml to unify conda versioning (Azure#18141) [Fix]Add retry for streaming download (Azure#18164) Sync eng/common directory with azure-sdk-tools for PR 1558 (Azure#18178) Stop filtering projects by ci.yml files (Azure#18174) [Communication] - SDK - Added contributing file (Azure#18018) ...
openapi-sdkautomation bot
pushed a commit
to AzureSDKAutomation/azure-sdk-for-python
that referenced
this pull request
Mar 15, 2022
update readme 2_5 (Azure#18191) * update readme 2_5 * Update readme.python.md
openapi-sdkautomation bot
pushed a commit
to AzureSDKAutomation/azure-sdk-for-python
that referenced
this pull request
Mar 15, 2022
update readme 2_5 (Azure#18191) * update readme 2_5 * Update readme.python.md
openapi-sdkautomation bot
pushed a commit
to AzureSDKAutomation/azure-sdk-for-python
that referenced
this pull request
Mar 15, 2022
update readme 2_5 (Azure#18191) * update readme 2_5 * Update readme.python.md
openapi-sdkautomation bot
pushed a commit
to AzureSDKAutomation/azure-sdk-for-python
that referenced
this pull request
Mar 15, 2022
update readme 2_5 (Azure#18191) * update readme 2_5 * Update readme.python.md
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.