bitcrowd / tickety-tick

A browser extension that helps you name branches and write better commit messages
MIT License
57 stars 10 forks source link

Upgrade Jest and Babel packages, clean up config #242

Closed pmeinhardt closed 4 years ago

pmeinhardt commented 4 years ago
pmeinhardt commented 4 years ago

Why can we get rid of the globals? Because we always assign / mock them right where tests need them? Or are they a leftover from earlier days?

Yeah, not sure anymore why we had it in there. A left-over for some reason. Jest already comes with the "jsdom" test environment as its default and we could even switch to the "node" one for a marginal speed-up since we're not mounting any components in the tests and explicitly instantiate (JS)DOMs for some of our test cases.