feat(jest config): set notify to false in JEST_OVERRIDE_OPTIONS

[ollelauribostrom]

I propose turning of jest notifications by default. Having test notifications turned on in a project can ofc be a great thing but when stryker runs the tests this adds a lot of noise (one notification each time stryker runs the tests against a mutant).

Summary of changes

• Adding notify: false to jestOverrideOptions.ts
• Updating tests to verify that this property is overridden

[simondel]

What does this look like without explicitly setting it to off?

And what does it look like with the setting explicitly set to off but Stryker's logging set to 'TRACE'?

[ollelauribostrom]

Looks like this without explicitly turning this off:
https://media.giphy.com/media/8UGFvYxcwWWDrblA77/giphy.gif

Trace:
https://gist.github.com/ollelauribostrom/27f9e8dc8afd2c8abc3a1391cd0fd0ad
(Note that im running Stryker programatically from a custom script if that could have an impact on the trace 🙂)

[nicojs]

Lol @ notifications :)

Turning it off sounds like the best approach.

[simondel]

Wow that's something you don't want to have enabled when doing mutation testing 😂