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
refactor: test utils hash helper #1110
Conversation
colincclark
commented
Jul 13, 2018
- make iterator a non-default export from test utils
- add hash test helper function for long keys in snapshots
- rolled out and replaced hashing in icon, pagination and watermark packages
packages/test-utils/package.json
Outdated
@@ -50,6 +49,9 @@ | |||
"webpack": "4.6.0", | |||
"webpack-cli": "2.1.4" | |||
}, | |||
"dependencies": { |
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.
should test-utils
just provide the hashing ability rather than tying it into serialization?
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.
changed
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.
on reflection replaceLongKeys
isn't the best name because it replaces the value?
658be9c
to
bfddb09
Compare