-
Notifications
You must be signed in to change notification settings - Fork 455
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
Support jest 24.9.0 #1315
Comments
hi, Our CI is running with jest 24.9.0 and there aren't any issues. The breaking change you linked doesn't belong to 24.9.0. |
@ahnpnl Oops, you are right, I linked the wrong commit. I use a module in my globalSetup file that performs
|
would you please provide a minimum repo for your issue ? Thanks 👍 |
@ahnpnl Will do tomorrow or the day after tomorrow, thank you |
@ahnpnl https://github.com/ChrisLahaye/js-test-bug The compilation issues only occur after awaiting a promise in globalSetup:
Without it, the required files are compiled correctly. |
hi, it looks strange to me that removing the piece of code |
I'm getting an issue that seems to be 24.9 related as well.
Reverted back to 24.8 and it works just fine. Line 185 in dd816ab
seems to be returning no output files |
do you have a repo for your issue @dl748 ? |
That really is the case. You can try the repo with and without the await commented out. |
Yes I can see the error you encountered @ChrisLahaye |
@ahnpnl My apologies for asking this directly, but could you please help me fix/debug this issue? |
At the moment I still don’t have a clue about this issue, 1st time for me to see this. |
I got the same error as @ChrisLahaye, which is related to transpiling the import statements. The tools I'm using is ts-jest + typeorm. I can't transpile the entities and migrations files on version 24.9.0. I've rollbacked the version to 24.8.0 in other it to work. |
Is it related to globalSetup async too ? |
@ahnpnl yes, globalSetup and globalTeardown. |
There is a documentation updated for using setTimeout in jest v25.1 in section setupFilesAfterEnv suggesting that using settimeout should be inside setupFilesAfterEnv. See PR Does jest support setTimeout in globalSetup and globalTeardown ? |
@ahnpnl I just used setTimeout to have the promise resolve after a while. The problem however occurs for any promise, even without setTimeout. |
@kulshekhar is this one redundant now? |
There is an issue with Promise in globalSetup and globalTearDown seems to be related to 24.9, not sure yet what causes the issue. |
@kibertoad I'm not sure if this is redundant. If it's still a problem and if jest 24.9.0 is being used, this should be left open until fixed |
Didn't last pr drop support for 2.4? |
It did but only for v25+. It could, in theory, be that someone wants to keep using v24.x |
hi @ChrisLahaye , so I spent some times to debug your issue with the repo you provided. Here is the summary of the debugging. Please 1st take a look at how A few notes on how jest and
So what is the difference between the successful case vs error caseWhen commenting out the
|
Close as duplicate #1391 |
Hello,
Jest released a new minor version 24.9.0 that breaks ts-jest. It breaks for prior versions as well due to unconstrained nested dependencies.
The BREAKING change is jestjs/jest@1d8245d#diff-4ac32a78649ca5bdd8e0ba38b7006a1e
Overwriting the nested dependencies confirms this and can be used as temporary workaround:
It would be nice if ts-jest could be made compatible with the latest jest release.
Related issues:
Regenerated lock file for
"jest": "24.8.0"
without overwrite:The text was updated successfully, but these errors were encountered: