Skip to content
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

[formrecognizer] Migrate to test proxy #21822

Merged
merged 62 commits into from
Jan 4, 2022

Conversation

catalinaperalta
Copy link
Member

@catalinaperalta catalinaperalta commented Nov 18, 2021

Fixes #21144

Copy link
Member

@kristapratico kristapratico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

consider running live/without test proxy in CI as well just to verify all changes are good

@catalinaperalta
Copy link
Member Author

/azp run python - formrecognizer - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@catalinaperalta
Copy link
Member Author

/azp run python - formrecognizer - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@catalinaperalta catalinaperalta merged commit 9bbd54c into Azure:main Jan 4, 2022
@catalinaperalta catalinaperalta deleted the mq/testProxy branch January 4, 2022 19:37
rakshith91 pushed a commit to rakshith91/azure-sdk-for-python that referenced this pull request Apr 10, 2022
* use assert in testcase

* replace with assert is not none

* update remaining assertions

* initial test proxy work

* add ci.yml update

* temp

* temp

* test fixes

* change assert for error

* rstrip endpoint

* add sanitizers

* clean up replacers and FormRecognizerTest, move test vars

* use pytest.raises

* update test_business_card.py

* add business card recordings

* replace blob sas urls

* fix deleted receipt test

* update business card tests

* update content tests

* update custom forms tests

* update custom model tests

* update container url sanitizer and custom model import

* update general document tests

* update layout tests

* delete business card recordings

* remove all old recordings

* update dac prebuilt tests

* rstrip async client endpoint

* add oauth functions back in

* update compose tests

* skip copy model tests

* update dmac mgmt tests

* update dmac training tests

* update ftc compose tests

* update ftc copy model

* update conftest access token sanitizer

* update ftc mgmt tests

* update ftc training tests

* update get children tests

* update frc identity doc tests

* update frc invoice tests

* update frc receipt tests

* update multiapi tests

* clean up empty recordings

* remove extra empty recording

* update preparer values to match recordings

* dont check for body matching in requests that change during playback

* add playback condition in _get_blob_url

* default testing_container_sas_url

* set bodiless matcher for url tests

* set bodiless matcher for mgmt tests using DMAC and FTC

* move rstrip up in code

* remove unused code from conftest.py

* make large file tests run only live

* add missing imports

* remove extra space

* remove some more large recordings

* add missing recording

* add more recordings
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-python that referenced this pull request Feb 1, 2023
Machinelearningservices microsoft.machine learning services 2022 12 01 preview (Azure#21761)

* Adds base for updating Microsoft.MachineLearningServices from version preview/2022-10-01-preview to version 2022-12-01-preview

* Updates readme

* Updates API version in new specs and examples

* Add Dec API Registries Swagger (Azure#21419)

* add december registries swagger + examples

* add status code 202 in examples

* fix 202 examples

* fixes

* fixes

* fix

* add 202 back in for put/patch

Co-authored-by: Komal Yadav <komalyadav@microsoft.com>

* remove location (Azure#21430)

Co-authored-by: Komal Yadav <komalyadav@microsoft.com>

* remove readonly flag on schedules property for CI (Azure#21653)

Co-authored-by: Naman Agarwal <naagarw@microsoft.com>

* add missing workspace properties (Azure#21725)

* December preview updating mfe.json specs (Azure#21510)

* December preview updating mfe.json specs

* MFE Dec 2022 Preview API - Adding logbase

* MFE 2022-12-01-preview swagger spec model validation fix

* MFE 2022-12-01-preview swagger spec model validation fix, add missing location

* MFE 2022-12-01-preview swagger spec model validation - typo fix

* MFE 2022-12-01-preview swagger spec model validation - fix api version in automljob example

* MFE 2022-12-01-preview swagger spec model validation - fix for multiselectenabled error

* MFE 2022-12-01-preview swagger spec model validation - fix for multiselectenabled error

* Fix  for 1006 - RemovedDefinition (RecurrenceTrigger,CronTrigger) (Azure#21822)

* fix ReadonlyPropertyChanged of MLC (Azure#21814)

Co-authored-by: Bingchen Li <bingchenli@microsoft.com>

* fixed custom-words conflict (Azure#21829)

* fix custom-words conflict merge (Azure#21830)

* example fix (INVALID_REQUEST_PARAMETER) (Azure#21832)

Co-authored-by: Ivaliy Ivanov <ivaliyivanov@Ivaliys-MacBook-Air.local>

* example fix, use correct api preview version  - (INVALID_REQUEST_PARAMETER) (Azure#21833)

Co-authored-by: Ivaliy Ivanov <ivaliyivanov@Ivaliys-MacBook-Air.local>

* Revert breaking change for MLC swagger 2022-12-01-preview (Azure#21885)

Co-authored-by: Bingchen Li <bingchenli@microsoft.com>

* Revert Connection Category back to enum. (Azure#21939)

* revert provisioning state change (Azure#21940)

* remove body (Azure#21978)

Co-authored-by: Komal Yadav <komalyadav@microsoft.com>

* Addressed comments, added x-ms-long-running-operation to a patch call (Azure#22005)

* Addressed comments, added x-ms-long-running-operation to a patch call

* fix examples for patch - remove body

* fixed formatting

* Ivalbert fix patch2 (Azure#22006)

* Addressed comments, added x-ms-long-running-operation to a patch call

* fix examples for patch - remove body

* fixed formatting

* fixed formatting

* Updated custom words (Azure#22262)

* Fixed prettier errors (Azure#22237)

* fixed examples for LRO_RESPONSE_HEADER check (Azure#22293)

* fixed examples for LRO_RESPONSE_HEADER check (Azure#22294)

* Example fix - OBJECT_MISSING_REQUIRED_PROPERTY - Missing required property: triggerType (Azure#22317)

---------

Co-authored-by: Komal Yadav <23komal.yadav23@gmail.com>
Co-authored-by: Komal Yadav <komalyadav@microsoft.com>
Co-authored-by: Naman Agarwal <namanag16@gmail.com>
Co-authored-by: Naman Agarwal <naagarw@microsoft.com>
Co-authored-by: ZhidaLiu <zhili@microsoft.com>
Co-authored-by: libc16 <88697960+libc16@users.noreply.github.com>
Co-authored-by: Bingchen Li <bingchenli@microsoft.com>
Co-authored-by: Ivaliy Ivanov <ivaliyivanov@Ivaliys-MacBook-Air.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[formrecognizer] Migrate tests to test proxy
2 participants