Migrate jest-runner to jest-worker #4825
Conversation
|
This PR supersedes #4596, which was incorrect from a branch name, description and content point of view. |
| ); | ||
| } catch (error) { | ||
| callback(formatError(error)); | ||
| } catch (err) { |
There was a problem hiding this comment.
can you call it error again? It's an error.
There was a problem hiding this comment.
Done; I didn't even notice; usually I use err by convention; like evt for events.
packages/jest-runner/src/index.js
Outdated
| rawModuleMap: watcher.isWatchMode() | ||
| ? test.context.moduleMap.getRawModuleMap() | ||
| : null, | ||
| : undefined, |
There was a problem hiding this comment.
Mistake I guess. I reverted that one.
There was a problem hiding this comment.
Oh, I found why. The expressed type was incorrect. I fixed it in a second commit.
Codecov Report
@@ Coverage Diff @@
## master #4825 +/- ##
=========================================
+ Coverage 59.15% 59.4% +0.24%
=========================================
Files 200 200
Lines 6660 6643 -17
Branches 4 3 -1
=========================================
+ Hits 3940 3946 +6
+ Misses 2720 2697 -23
Continue to review full report at Codecov.
|
This reverts commit eb70265.
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
4 participants
This PR migrates
jest-runnerto usejest-worker, instead ofworker-farm. As a consequence of this, some additional changes were done:pifyis not needed anymore as well.