Skip to content

Commit

Permalink
Add support to test multi service API version (#12719)
Browse files Browse the repository at this point in the history
* [WIP] prototype: multi-service-version test support

* Move multi-version test support into its own package

* Add some unit tests

* Add README

* Fix missing rollup error in CI build

* Disable browser test bundling

* Remove option to specify custom string comparison function

Now it's required to have a sorted list of API version strings
supported by the service from the oldest to the latest. This list is
used to compare version strings.

Add missing `skip` method to the chain object.

Add unit tests

* Fix README analysis issue

* Move `forEach()` and `onVersions()` into `versionsToTest()`

* Ignore README verification errors

* Fix rollup error

* Undo file moves

* Minor README tweak

* Fix build error in pipeline

* Add `DISABLE_MULTI_VERSION_TESTING` env var
  • Loading branch information
jeremymeng authored Jan 5, 2021
1 parent 0b18dd6 commit d6d5163
Show file tree
Hide file tree
Showing 16 changed files with 925 additions and 197 deletions.
31 changes: 29 additions & 2 deletions common/config/rush/pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions dataplane.code-workspace
Original file line number Diff line number Diff line change
Expand Up @@ -204,6 +204,10 @@
"name": "perfstress",
"path": "sdk/test-utils/perfstress"
},
{
"name": "multi-version test",
"path": "sdk/test-utils/multi-version"
},
{
"path": "sdk\\digitaltwins\\digital-twins-core"
}
Expand Down
1 change: 1 addition & 0 deletions eng/.docsettings.yml
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,7 @@ known_content_issues:
- ["sdk/synapse/synapse/README.md",  "#1583"]
- ["sdk/template/template/README.md",  "#1583"]
- ["sdk/test-utils/recorder/README.md",  "#1583"]
- ["sdk/test-utils/multi-version/README.md", "#1583"]
- ["sdk/test-utils/perfstress/CHANGELOG.md", "#1583"]

package_indexing_exclusion_list:
Expand Down
5 changes: 5 additions & 0 deletions rush.json
Original file line number Diff line number Diff line change
Expand Up @@ -597,6 +597,11 @@
"projectFolder": "sdk/test-utils/perfstress",
"versionPolicyName": "utility"
},
{
"packageName": "@azure/test-utils-multi-version",
"projectFolder": "sdk/test-utils/multi-version",
"versionPolicyName": "utility"
},
{
"packageName": "@azure/digital-twins-core",
"projectFolder": "sdk/digitaltwins/digital-twins-core",
Expand Down
1 change: 1 addition & 0 deletions sdk/keyvault/keyvault-keys/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,7 @@
"@azure/eslint-plugin-azure-sdk": "^3.0.0",
"@azure/identity": "^1.1.0",
"@azure/test-utils-recorder": "^1.0.0",
"@azure/test-utils-multi-version": "^1.0.0",
"@microsoft/api-extractor": "7.7.11",
"@rollup/plugin-commonjs": "11.0.2",
"@rollup/plugin-json": "^4.0.0",
Expand Down
Loading

0 comments on commit d6d5163

Please sign in to comment.