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
This ticket is for us to keep track if/when rewire is updated or maybe to discuss alternative solutions, depending how long it will take for rewire to update. Their last update was 20 days ago, so could be some hope available.
The text was updated successfully, but these errors were encountered:
I don't think moving away from rewire is a realistic approach as our repos are quite heavily invested in rewire. Without rewire a lot of code I think will need to be rewritten and exposed in a way so that the unit tests can have access to attach spies and such. It would be considerable amount of work to do, especially to maintain the same level of unit testing that we have.
If it takes a long time to see a fix from either NodeJS, or have my PR merged into rewire, would using my fork be acceptable?
Tests that try to
rewire
any module with a shebang declaration are broken starting NodeJS 12.16.0.The issue is being tracked at jhnns/rewire#178
I've created a PR for rewire at jhnns/rewire#179
This ticket is for us to keep track if/when rewire is updated or maybe to discuss alternative solutions, depending how long it will take for rewire to update. Their last update was 20 days ago, so could be some hope available.
The text was updated successfully, but these errors were encountered: