-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Graph] App state management saga tests #46914
[Graph] App state management saga tests #46914
Conversation
💔 Build Failed |
💚 Build Succeeded |
💚 Build Succeeded |
…anagement-saga-tests
💔 Build Failed |
Jenkins, test this. |
💚 Build Succeeded |
@elasticmachine merge upstream |
💚 Build Succeeded |
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.
Code LGTM
Apart from the console.error that should go away when we hopefully upgrade to the next react version, there's one thing I've noticed, when I start the test (node scripts/jest legacy/plugins/graph
), I got several The corejs option only has an effect when the useBuiltIns option is not false
. However I don't know I this is related to this PR, and it's just a observation, just wanted to mention it.
Depends on #46133
Adds tests to sagas and critical parts of state management