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

deps(jest): fix chromatic jest related bug #199

Merged
merged 2 commits into from
Oct 11, 2023

Conversation

billhimmelsbach
Copy link
Contributor

@billhimmelsbach billhimmelsbach commented Oct 10, 2023

Chromatic tests were failing with a weird Error: page.evaluate: TypeError: expect.extend: default is not a valid matcher. Must be a function, is "object" error.

Changes

  • updates the version for "@storybook/jest" to "^0.2.2", to fix a bug with the jest/storybook integration (see the issue thread)

How to test this PR

  1. Make sure storybook still works as intended
  2. Note that chromatic no longer fails in CI on this PR

Screenshots

Screenshot 2023-10-11 at 7 35 34 AM

@netlify
Copy link

netlify bot commented Oct 10, 2023

Deploy Preview for cfpb-design-system-react ready!

Name Link
🔨 Latest commit 63e84e5
🔍 Latest deploy log https://app.netlify.com/sites/cfpb-design-system-react/deploys/652673142371fe0008cc7830
😎 Deploy Preview https://deploy-preview-199--cfpb-design-system-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@billhimmelsbach billhimmelsbach marked this pull request as ready for review October 11, 2023 14:38
Copy link
Contributor

@meissadia meissadia left a comment

Choose a reason for hiding this comment

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

Woohoo! 👍🏾

@billhimmelsbach
Copy link
Contributor Author

Heyo! I dug around Chromatic @meissadia and figured out how to update the baseline images for comparison. Hopefully that'll make it easier going forward. 🤞 I'll try to demo it a bit tomorrow during our frontend meeting.
Screenshot 2023-10-11 at 1 44 48 PM

@billhimmelsbach billhimmelsbach merged commit 91ad005 into main Oct 11, 2023
@billhimmelsbach billhimmelsbach deleted the deps/update-jest-to-fix-chromatic branch October 11, 2023 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants