Page MenuHomePhabricator

[CASHTAB] Update the jest dependency

Authored by Fabien on Thu, Jan 7, 14:47.


Group Reviewers
Restricted Project
rABC82da5b3491b1: [CASHTAB] Update the jest dependency

Update jest to the latest version. This will allow for creating
coverage report files as needed, which is not supported with current
Note that the collectCoverageFrom regex exclusion has been removed. It
was not matching any file and is very likely broken:, so it's better to no rely
on it.
The coverage behavior seems to change quite a lot, because it takes into
acount the files with 0 coverage which were previously not accounted

Test Plan
npm install
npm test
npm run test:coverage

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien requested review of this revision.Thu, Jan 7, 14:47
deadalnix requested changes to this revision.Thu, Jan 7, 15:39
deadalnix added a subscriber: deadalnix.

If you need --legacy-peer-deps you have regressed something.

This revision now requires changes to proceed.Thu, Jan 7, 15:39
Fabien requested review of this revision.Thu, Jan 7, 16:10
Fabien edited the test plan for this revision. (Show Details)
Fabien added a subscriber: josephroyking.

I got this from the README, I did not introduce it for this diff. I tried without and it seems to run just fine, it's probably no longer needed @josephroyking ?
I updated the test plan.

This revision is now accepted and ready to land.Thu, Jan 7, 18:57
This revision was automatically updated to reflect the committed changes.