deps(jest): upgrade to 27.0.3 - #12454
Conversation
| global.I18n = undefined; | ||
| global.ReportUIFeatures = undefined; | ||
| global.matchMedia = undefined; | ||
| global.self.matchMedia = undefined; |
There was a problem hiding this comment.
I.... have no idea how this was working ... but I am extremely curious.
maybe before global.self === global and now it does not? In Chrome self === globalThis.self, so circular global object references makes some sort of sense, but maybe Jest decided to remove that behavior for global.
| runStub.mockRestore(); | ||
| }); | ||
|
|
||
| let originalRun; |
There was a problem hiding this comment.
jestjs/jest#9828 disallowed these functions from being nested in a test case.
There was a problem hiding this comment.
oh dang yeah, I did not expect that to work haha
| runStub.mockRestore(); | ||
| }); | ||
|
|
||
| let originalRun; |
There was a problem hiding this comment.
oh dang yeah, I did not expect that to work haha
|
The |
|
|
||
| await Promise.all([ | ||
| new Promise(resolve => server.close(resolve)), | ||
| server.close(), |
There was a problem hiding this comment.
Another brainteaser!
server.close doesn't take any parameters. This file is not typechecked, so this wasn't caught. If this were a raw http.Server object, it would work as written. This mistake happened because of the StaticServer refactor.
The brainteaser is... why did this ever work? I suppose before, Jest simply gave up at some point and shutdown. But now it will fail if afterPass hangs.
|
can we avoid updating our test runner before the release which depends on our test runner, though :) |
|
this can land whenever idc. i had 10 minutes to kill to i upgraded the pr |
This is the latest beta release. The latest official release does not yet have #7792, andI'm fed up with manually fixing things when using-u, so here we are.You can verify the new feature works by adding a number to:
https://github.com/GoogleChrome/lighthouse/blob/30ebe53/lighthouse-core/audits/script-treemap-data.js#L209
and running
yarn jest script-treemap-data -u. Only the inline snapshot strings will change.