Skip to content

[draft][lens] Fix drag / drop tests in lens#36449

Closed
chrisdavies wants to merge 8 commits intoelastic:feature/lensfrom
chrisdavies:lens/drag-drop-tests
Closed

[draft][lens] Fix drag / drop tests in lens#36449
chrisdavies wants to merge 8 commits intoelastic:feature/lensfrom
chrisdavies:lens/drag-drop-tests

Conversation

@chrisdavies
Copy link
Contributor

This is the workaround we discussed to get drag / drop tests passing again. I don't know why GitHub is showing 8 commits. I made only one. The diff appears to be correct, though.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@flash1293
Copy link
Contributor

I think we don't need this, the hook invariant errors seem to happen because we have a separate package.json and node_modules which somehow pulls in a second react version. Try removing node_modules on current feature/lens and re-running the tests - works for me.

Can we get rid of our local package.json and just bump the elastic-charts version on the global level?

This is upstream/feature/lens after rm -rf x-pack/plugins/lens/node_modules && node x-pack/scripts/jest.js lens:
Screenshot 2019-05-10 at 18 26 44

@flash1293
Copy link
Contributor

See #36456

@chrisdavies
Copy link
Contributor Author

Closing in lieu of Joe's PR.

@chrisdavies chrisdavies deleted the lens/drag-drop-tests branch May 10, 2019 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants