-
Notifications
You must be signed in to change notification settings - Fork 5.6k
add apitestErrorCode doc #14749
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
add apitestErrorCode doc #14749
Changes from 4 commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
dfbaeb2
add apitestErrorCode doc
ruowan 3c69568
add armRPC doc
ruowan d97b4e6
add roundTripInconsistentProperty doc
ruowan 29b0c02
small fix
ruowan 27ab05e
update
ruowan 01cdf73
update signalR armTemplate
ruowan 7fdac41
update yaml
ruowan ffd67dc
new file
ruowan cfa5608
add testScenario with armTemplate
ruowan 7d632d1
update doc
ruowan 5a4f6d7
update readme.md
ruowan ca65d70
update doc
ruowan bd9c0ca
update NOTE section
ruowan 9c1cf46
add signalRCreateOrUpdate example file
ruowan 882668a
udpate doc
ruowan f4c3994
add generate test scenario section
ruowan 90618cc
update doc
ruowan 75535cb
add serviceFacbric test scenario file
ruowan 5c6dc7d
update managedClusters.yaml
ruowan b0af73c
update generateABasicTestScenario.md
ruowan 983d796
update features doc
ruowan 6a5f970
add run api test gif
ruowan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,123 @@ | ||
| <!-- | ||
| Copyright (c) 2021 Microsoft Corporation | ||
|
|
||
| This software is released under the MIT License. | ||
| https://opensource.org/licenses/MIT | ||
| --> | ||
|
|
||
| # API testing error code | ||
|
|
||
| OAV api testing define some rules and check whether actual service response match with example. Example is very important for downstream SDK test code generation, code samples and docs. The goal to detect example quality issues and check service behaviors consistent with example. | ||
|
|
||
| ## Rule descriptions | ||
|
|
||
| #### INCORRECT_PROVISIONING_STATE | ||
|
|
||
| If service return 200 status code,it means the operation is succeed and finished. So the provisioning state should be one of the terminal states ["succeeded", "failed", "canceled", "ready", "created", "deleted"]. | ||
|
|
||
| The provisioning state is very important for downstream terraform or cli to manage resource status. For more details about provisioning state, please refer to this [ARM RPC provisioning state](https://github.com/Azure/azure-resource-manager-rpc/blob/master/v1.0/Addendum.md#provisioningstate-property) | ||
|
|
||
| #### RESPONSE_MISSING_VALUE | ||
|
|
||
| **Error message**: The response value is missing. Path: {}. Expected: {}. Actual: undefined | ||
|
|
||
| The example has defined response value, but actually the server doesn't return that value. | ||
|
|
||
| Example: | ||
|
|
||
| ```diff | ||
| { | ||
| "properties":{ | ||
| "targetType":"blobNfs", | ||
| "junctions":[ | ||
| { | ||
| "namespacePath":"/blobnfs" | ||
| } | ||
| ], | ||
| "blobNfs":{ | ||
| "target":"/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/scgroup/providers/Microsoft.Storage/storageAccounts/blofnfs/blobServices/default/containers/blobnfs", | ||
| - "usageModel":"WRITE_WORKLOAD_15" | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. suggest to show what is actual response and what correct should be in 2 example
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sure. Will be covered in next PR. |
||
| } | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| #### RESPONSE_ADDITIONAL_VALUE | ||
|
|
||
| **Error message** Return additional response value. Path: {}. Expected: undefined. Actual: {} | ||
|
|
||
| The example doesn't define the response value, but service actually return this value. | ||
|
|
||
| Example: | ||
|
|
||
| ```diff | ||
| { | ||
| "properties":{ | ||
| "targetType":"blobNfs", | ||
| "junctions":[ | ||
| { | ||
| "namespacePath":"/blobnfs" | ||
| } | ||
| ], | ||
| "blobNfs":{ | ||
| "target":"/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/scgroup/providers/Microsoft.Storage/storageAccounts/blofnfs/blobServices/default/containers/blobnfs", | ||
| "usageModel":"WRITE_WORKLOAD_15" | ||
| + "enableFeature": true | ||
| } | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| #### RESPONSE_INCONSISTENT_VALUE | ||
|
|
||
| **Error message** The actual response value is different from example. Path: {}. Expected: {}. Actual: {} | ||
|
|
||
| The service returned value is different from example value. | ||
|
|
||
| Example: | ||
|
|
||
| ```diff | ||
| { | ||
| "properties":{ | ||
| "targetType":"blobNfs", | ||
| "junctions":[ | ||
| { | ||
| "namespacePath":"/blobnfs" | ||
| } | ||
| ], | ||
| "blobNfs":{ | ||
| "target":"/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/scgroup/providers/Microsoft.Storage/storageAccounts/blofnfs/blobServices/default/containers/blobnfs", | ||
| - "usageModel":"WRITE_WORKLOAD_15" | ||
| + "usageModel":"WORK_LOAD_14" | ||
| } | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| #### ROUNDTRIP_INCONSISTENT_PROPERTY | ||
|
|
||
| **Error message** The property's value in the response is different from what was set in the request. Path: {}. Request: {}. Response: {} | ||
|
|
||
| Example: The sku in request parameters is `default`, but actual return is `standard`. | ||
|
|
||
| ```diff | ||
| { | ||
| "parameters":{ | ||
| "properties":{ | ||
| "name":"myService", | ||
| "SKU":"default" | ||
| } | ||
| }, | ||
| "responses":{ | ||
| "200":{ | ||
| "properties":{ | ||
| "name":"myService", | ||
| + "SKU":"standard" | ||
| - "SKU":"default" | ||
|
|
||
| } | ||
| } | ||
| } | ||
| } | ||
|
|
||
| ``` | ||
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a great starting point we have these error codes, can we add a section focus on each rule to describe how to fix? It's good to be more specific to guide service team to fix them, and it seems missing x-ms-secret or readonly are common issues spotting from the pilot
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. Maybe I need to write a doc about validation logic. It will be covered in next PR.