Skip to content

Conversation

scttcper
Copy link
Member

@scttcper scttcper commented Jun 5, 2024

upgrades @testing-library/react from 14 -> 16.
upgrades @testing-library/dom from 9 -> 10

Notes:

upgrades `@testing-library/react` from 14 -> 16.
upgrades `@testing-library/dom` from 9 -> 10

Notes:
- @testing-library/dom is now a peer dependency, have to install
- Support for react 19
- Upgrades aria-query to 5.3.0 testing-library/dom-testing-library#1241
  - changes getByRole('link') `a element now requires a set href attribute to have an implicit link role`
@scttcper scttcper requested a review from a team as a code owner June 5, 2024 17:31
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 5, 2024
@@ -92,11 +92,11 @@ describe('SwitchOrganization', function () {

expect(screen.getByRole('list')).toBeInTheDocument();

const org1Link = screen.getByRole('link', {name: 'org1 Organization 1'});
const org1Link = screen.getByRole('link', {name: 'O Organization 1'});
Copy link
Member Author

@scttcper scttcper Jun 5, 2024

Choose a reason for hiding this comment

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

i think it was able to get the aria label on the svg before.
now it is the literal text of the avatar

the link content looks like this
<avatar aria-label="org1">O</avatar> Organization 1

Copy link

codecov bot commented Jun 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.92%. Comparing base (be2219c) to head (8c5056e).
Report is 8 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #72137       +/-   ##
===========================================
+ Coverage   63.54%   77.92%   +14.38%     
===========================================
  Files        6571     6571               
  Lines      292731   292749       +18     
  Branches    50530    50533        +3     
===========================================
+ Hits       186002   228131    +42129     
+ Misses      98399    58367    -40032     
+ Partials     8330     6251     -2079     

see 2292 files with indirect coverage changes

@scttcper scttcper merged commit fd5acbc into master Jun 5, 2024
@scttcper scttcper deleted the scttcper/deps-testing-lib branch June 5, 2024 18:19
@github-actions github-actions bot locked and limited conversation to collaborators Jun 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants