Use node env in Jest

[willdurand]

Fixes mozilla/addons#12713

---

According to jestjs/jest#1909, we should use node as the test environment. I am not sure what the implications are, but if the test suite passes with that, I guess it is alright.

I saw a similar patch related to webpack 4 too: mxmul/closure-loader#40

I marked the test file for the sriDataPlugin as "requires node", see also: mozilla/addons#10410

[willdurand]

According to facebook/jest#1909, we should use node as the test environment. I am not sure what the implications are, but if the test suite passes with that, I guess it is alright.

I saw a similar patch related to webpack 4 too: mxmul/closure-loader#40

Implications are clear: we have a test suite that requires a "browser" context, in most cases. Yet, tests for a webpack plugin (like the sriDataPlugin) should run in a node context IMO.

[muffinresearch]

/me looks wistfully at mozilla/addons#10410

[willdurand]

/me looks wistfully at mozilla/addons#10410

There is literally an issue filed from everything 😄

[willdurand]

Second attempt 🚀

[codecov-io]

Codecov Report

Merging #7032 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    mozilla/addons-frontend#7032   +/-   ##
=======================================
  Coverage   97.85%   97.85%           
=======================================
  Files         251      251           
  Lines        6852     6852           
  Branches     1264     1264           
=======================================
  Hits         6705     6705           
  Misses        133      133           
  Partials       14       14

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 54180d6...aed57af. Read the comment docs.

[willdurand]

Ok, that looks good!

[willdurand]

Merging this now because that's annoying :/