-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
test: batch 1 remove strings testid app code #8011
test: batch 1 remove strings testid app code #8011
Conversation
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.
Left a few comments, please run regression and smoke.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #8011 +/- ##
=======================================
Coverage 36.61% 36.61%
=======================================
Files 1120 1120
Lines 29196 29196
Branches 2720 2720
=======================================
Hits 10691 10691
Misses 17893 17893
Partials 612 612 ☔ View full report in Codecov by Sentry. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Description
Replace all string definitions used for testIDs. These files should no longer selector strings in the TestID prop:
Related issues
Fixes: 7865
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist