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

Upgrade enzyme and enzyme-adapter-react-16 to the latest #37064

Merged
merged 8 commits into from
May 29, 2019

Conversation

sulemanof
Copy link
Contributor

@sulemanof sulemanof commented May 24, 2019

Summary

The main purpose of this is an ability to test React functional components with Hooks API.
The necessity of writing such tests is in scope of EUIfication of an old vis editor #30922

Checklist

For maintainers

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@sulemanof sulemanof requested a review from maryia-lapata May 27, 2019 08:15
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

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

LGTM


/*
The issue with [email protected] -> the fix has not been released yet -> https://github.com/airbnb/enzyme/pull/2027
TODO: Enable the expectation after the next patch released
Copy link
Contributor

Choose a reason for hiding this comment

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

This module isn't being worked on a ton right now, would you mind adding something to your calendar to return to this in a couple weeks once the enzyme-adapter is updated?

Copy link
Contributor

Choose a reason for hiding this comment

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

@spalger sure.
@sulemanof is on vacation, so I will track this.

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@maryia-lapata maryia-lapata merged commit f6e91cc into elastic:master May 29, 2019
maryia-lapata pushed a commit to maryia-lapata/kibana that referenced this pull request May 29, 2019
* Upgrade enzyme and enzyme-adapter-react-16 to the latest

* Skip failed expectation

* Update yarn.lock

* Update snapshot
maryia-lapata added a commit that referenced this pull request May 29, 2019
…7320)

* Upgrade enzyme and enzyme-adapter-react-16 to the latest

* Skip failed expectation

* Update yarn.lock

* Update snapshot
jkakavas pushed a commit to jkakavas/kibana that referenced this pull request May 30, 2019
* Upgrade enzyme and enzyme-adapter-react-16 to the latest

* Skip failed expectation

* Update yarn.lock

* Update snapshot
@sulemanof sulemanof mentioned this pull request Jun 12, 2019
7 tasks
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.

5 participants