You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I would never expect enzyme to "just work" with a new major version of React out of the box :-) React breaks things in minors often enough as it is.
First, I have to create an adapter for React 17, and then we'll be able to look at the test failures and go from there.
Reacted by Oleksandr Fediashov, Kamil Dzieniszewski, Sibasish Mohanty, Tim, Song Jia, 俊宁, Antti Ahti, Vladimir Saveliev, Elnaz, Wilbert Cedeno and 10 more
Reacted by Marouane Trabelsi, Elle Baker and Ashfaq Mahmood Rafi
Thanks, but that isn't going to speed anything up. enzyme won't be guaranteed to work on React 17 until it's fully released, not a preview.
Reacted by Henry Q. Dineen, 俊宁, Rob Wierzbowski, Ashfaq Mahmood Rafi and Wilfried Lawson
Reacted by Oleksandr Fediashov, Ricky, Sebastian "Sebbie" Silbermann, 俊宁, Ali Bayatpour, Shrihari Murali, Aleksey Levenstein, Andrew E McKendrick, Amir Muhammad Hakim, Danila Loginov and 3 more
That is certainly my hope and plan, just not something i can commit to, since the react team hasn’t coordinated with me prior to releases in years.
Reacted by Alexander Quine, Allan Kimmer Jensen, gsimionato-daitan, Amir Muhammad Hakim, Bluefitdev, Wilbert Cedeno, Kane Rogers-Wong, Alireza Zarei, Ashfaq Mahmood Rafi and Carlos Bensant
It’s fair we haven’t reached out before this particular RC since we generally view an RC as a “reachout” to the whole community. But I believe I did file #2358 (16.14 compatibility). Our plans for 16.14 changed and that release turned into 17 but I’d expect the naming change described in that issue to be the primary one you’d need to address. (Although I could totally be missing something!) Sorry for the frustration.
Reacted by Brian Espinosa, Alexander Quine, Bill Sorensen, Xavier Ramirez, Aleksey Levenstein, gsimionato-daitan, Bluefitdev, Lata Tiwari, Nathan Victor, Said Akhmed Agitaev and 12 more
@gaearon fair point that there's been nonzero efforts, primarily from you personally, which I appreciate. Thanks for pointing me to that issue; I'll incorporate that into the update.
Reacted by dan, Alexander Quine, Christopher Svanefalk, Kane Rogers-Wong, Jordan Wallet, Alireza Zarei, Karl-Aksel Puulmann, Ashfaq Mahmood Rafi and Carlos Bensant
For those impatient ones (myself included), I published @wojtekmaj/enzyme-adapter-react-17 package. It's exactly like enzyme-adapter-react-16, just the necessary bits replaced.
I tested it on a semi large repository (2000 tests) with no failures, so it should be good enough to use, if the only thing blocking you was a broken adapter.
I will deprecate the package once an official adapter is released.lol
Enjoy!
Reacted by Anton Lavrenov, Filipp Riabchun, Oleksandr Fediashov, Tommy Chen, Takanori Ishikawa, Triguna Mutuguppe Sripathi, Arun Kumar, Mat, Tomasz Jaskula, Jiri Spac and 285 more
Reacted by Luis Felipe Martins, Maksim Yepifanau, Adam Scarlat, Lucas Simões, Daniel Shelton, Evgeny Bobkin, nhiguchi-stb, MCMXC, Кирилл Капитальцев, Artur and 11 more
Reacted by Nathan Victor, Meryl, Luis Toubes, Gabriel Varela, James Kim, Jesse Florig, Tata.Paweennuch, Arsi Halttunen, Cressence Tchounga, Maksim Yepifanau and 25 more
Reacted by Sasha (Alex), Marco Arduini, Naoise Golden Santos, Anna Branco, Francesco Giordano, Maxime Quandalle, Andrzej Kłapeć, Robert Danielsson, John, Angel Martinez and 102 more
Reacted by Mat, Sasha (Alex), Marco Arduini, Ileriayo Adebiyi, Joel Bourbonnais, M Giridhar, Raphael Amoedo, Riccardo Tempesta - The Rick, Florian Rappl, Ioannis Cherouvim and 50 more
Reacted by Meryl, Gabriel Varela, James Kim, Yukako IIDA, Maksim Yepifanau, Lucas Simões, Кирилл Капитальцев, Lucas Fernandes, Matthew Hill, Barry Michael Doyle and 2 more
Reacted by Wojciech Maj, Kyle Szombathy, Maarten Zuidhoorn, Brian Min, xcv58, Marco Arduini, Allan Kimmer Jensen, mehran motiee, fbontin, Gerardo Nava Pereda and 23 more
Indeed it has, thanks. I'm still working on getting the adapter set up and passing tests.
Reacted by Jiri Spac, Valentin, Christian Wiese, daedalic, Antti Ahti, Sanborn Hilland, Jason McPeak, John Ryan, Giovanni Fumagalli, abhishekKlenty and 70 more
Reacted by Henry Q. Dineen, Alexander Quine, Aaron Hickman, Mat, Masataka Pocke Kuwabara, Marco Arduini, Kitty Giraudel, Naoise Golden Santos, Sean, E-Liang Tan and 16 more
Reacted by flynntutoring, Stefanie L., Alessio Petrin, Andrei Horodinca, nhiguchi-stb and Ashfaq Mahmood Rafi
Reacted by Ileriayo Adebiyi, Joel Bourbonnais, Bryan Doxtdator, Bruno Domingos, Sergei Startsev, justinlnx, blagodus, Eugene, Aaron Ross, Alessio Petrin and 3 more
Activity
layershifter commentedon Aug 11, 2020
A CodeSandbox is ready: https://codesandbox.io/s/cranky-euclid-4qr9z
It's enough to use
mount(<div />)to repro it, seeindex.test.js.ljharb commentedon Aug 11, 2020
I would never expect enzyme to "just work" with a new major version of React out of the box :-) React breaks things in minors often enough as it is.
First, I have to create an adapter for React 17, and then we'll be able to look at the test failures and go from there.
layershifter commentedon Aug 12, 2020
@ljharb I started #2430 as I would like to run tests on React 17 ASAP 😺 Appreciate your eyes on it
ljharb commentedon Aug 12, 2020
Thanks, but that isn't going to speed anything up. enzyme won't be guaranteed to work on React 17 until it's fully released, not a preview.
wojtekmaj commentedon Aug 18, 2020
I think no one expects everything to work right from the start, but it would be nice to just have the adapter preview so we can find bugs early :)
ljharb commentedon Aug 18, 2020
That is certainly my hope and plan, just not something i can commit to, since the react team hasn’t coordinated with me prior to releases in years.
gaearon commentedon Aug 19, 2020
It’s fair we haven’t reached out before this particular RC since we generally view an RC as a “reachout” to the whole community. But I believe I did file #2358 (16.14 compatibility). Our plans for 16.14 changed and that release turned into 17 but I’d expect the naming change described in that issue to be the primary one you’d need to address. (Although I could totally be missing something!) Sorry for the frustration.
ljharb commentedon Aug 19, 2020
@gaearon fair point that there's been nonzero efforts, primarily from you personally, which I appreciate. Thanks for pointing me to that issue; I'll incorporate that into the update.
wojtekmaj commentedon Aug 24, 2020
For those impatient ones (myself included), I published
@wojtekmaj/enzyme-adapter-react-17package. It's exactly likeenzyme-adapter-react-16, just the necessary bits replaced.Use it like any other adapter, so:
I tested it on a semi large repository (2000 tests) with no failures, so it should be good enough to use, if the only thing blocking you was a broken adapter.
I will deprecate the package once an official adapter is released.lolEnjoy!
trigunam commentedon Sep 23, 2020
ljharb commentedon Sep 23, 2020
hengkx commentedon Oct 21, 2020
react 17 has been released.
ljharb commentedon Oct 21, 2020
Indeed it has, thanks. I'm still working on getting the adapter set up and passing tests.
Fix tests by switch to unofficial Enzyme adapter
100 remaining items
ljharb commentedon Jan 19, 2022
move from enzyme 16 to enzyme 17
JWBrownie commentedon Jul 18, 2023
ljharb commentedon Jul 18, 2023
VictorQueiroz commentedon Jul 18, 2023
ljharb commentedon Jul 18, 2023
aaryn101 commentedon Jul 18, 2023
ljharb commentedon Jul 19, 2023
sbaechler commentedon Jul 19, 2023
hoangph271 commentedon Jul 19, 2023
ljharb commentedon Jul 19, 2023
VictorQueiroz commentedon Jul 19, 2023
aaryn101 commentedon Jul 19, 2023
ljharb commentedon Jul 19, 2023