-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Add tests for ResponseSize component #4676
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
Add tests for ResponseSize component #4676
Conversation
packages/bruno-app/package.json
Outdated
"mini-css-extract-plugin": "^2.4.5", | ||
"postcss": "8.4.47", | ||
"style-loader": "^3.3.1", | ||
"styled-components": "^6.1.18", |
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.
Could you please move styled-components
from devDependencies
to dependencies
and revert it back to version 5.3.3
, which is the version we're currently using?
packages/bruno-app/jest.config.js
Outdated
projects: [ | ||
{ | ||
displayName: 'DOM', | ||
testMatch: ['<rootDir>/src/components/**/*.(spec|test).[jt]s?(x)'], |
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.
Change to /**/*.{spec,test}.[jt]s?(x)
parentheses syntax is incorrect for globs
packages/bruno-app/jest.config.js
Outdated
}, | ||
{ | ||
displayName: 'NODE', | ||
testMatch: ['<rootDir>/src/**/*.(spec|test).[jt]s?(x)'], |
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.
Change to /**/*.{spec,test}.[jt]s?(x)
parentheses syntax is incorrect for globs
Closing the PR since there was a problem in the build with the branch. See new PR here. |
Description
Contribution Checklist:
Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.