[FormRecognizer] Updated Form Recognizer sanitized properties #28151

kinelski · 2022-04-11T17:31:20Z

Long story short, this is the timeline of events that caused the bug we're fixing in this PR:

Test Framework introduces a bug that prevents tests from correctly finding mismatches in Playback mode.
A miscopy updates the FR live test base constructor, sanitizing the wrong property. This would make tests fail, but this goes undetected because of the Test Framework bug.
The Test Framework bug is detected and fixed. The FR bug is exposed.
We're fixing the FR bug now.

Fixes #27701.

azure-sdk · 2022-04-11T17:39:20Z

API change check for Azure.AI.FormRecognizer

API changes are not detected in this pull request for Azure.AI.FormRecognizer

kinelski added 2 commits April 11, 2022 09:49

Removed CompareBodies line

da475d9

Fixed sanitized property

9ccd09c

ghost added the Cognitive - Form Recognizer label Apr 11, 2022

kinelski marked this pull request as ready for review April 11, 2022 17:45

kinelski requested review from jsquire and maririos as code owners April 11, 2022 17:45

kinelski added the Client This issue points to a problem in the data-plane of the library. label Apr 11, 2022

jsquire approved these changes Apr 11, 2022

View reviewed changes

kinelski merged commit 24e3145 into main Apr 11, 2022

kinelski deleted the fr-tests branch April 11, 2022 21:02

kinelski mentioned this pull request Apr 12, 2022

[BUG] Need to investigate tests that were able to pass based on CompareBodies bug #27688

Closed

Provide feedback