Skip to content

Conversation

@Tobbe
Copy link
Collaborator

@Tobbe Tobbe commented Dec 15, 2023

Finally figured out how to get the types working for @testing-library/jest-dom!

So can revert #9713, which reverted #9673 and #9711.

More details can also be found in an issue I created over at the jest-dom repo testing-library/jest-dom#559

@Tobbe Tobbe added the release:breaking This PR is a breaking change label Dec 15, 2023
@Tobbe Tobbe added this to the v7.0.0 milestone Dec 15, 2023
Copy link
Contributor

@jtoar jtoar left a comment

Choose a reason for hiding this comment

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

Thanks @Tobbe! One comment about the CRWA template, and CI seems to be going ok but I think the test project fixture still needs to be updated

},
"typeRoots": ["../node_modules/@types", "./node_modules/@types"],
"types": ["jest", "@testing-library/jest-dom"],
"typeRoots": ["../node_modules/@types", "./node_modules/@types", "../node_modules/@testing-library"],
Copy link
Contributor

Choose a reason for hiding this comment

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

@Tobbe do we need jest and jest-dom?

Copy link
Collaborator Author

@Tobbe Tobbe Dec 16, 2023

Choose a reason for hiding this comment

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

I'm not sure. The test project seems to work fine without jest. But I haven't looked into it more than that. So I'll add it back just to be sure.

@Tobbe Tobbe force-pushed the tobbe-jest-dom-revert-revert branch from f6b11dd to 664b739 Compare December 16, 2023 00:12
@Tobbe Tobbe merged commit 5f89c01 into redwoodjs:main Dec 16, 2023
@Tobbe Tobbe deleted the tobbe-jest-dom-revert-revert branch December 16, 2023 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:breaking This PR is a breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants