Fix serialization issues with RegExp instances in the normalized config

[rubennorte]

Summary

Fixes #7242 introduced in #7209, where the configuration was incorrectly serialized as it contained RegExp instances (incompatible with the automatic JSON serialization done natively by Node.js in child_process.send.

Test plan

Updated unit and e2e tests.

[SimenB]

Makes sense! Should revert (but keep the test) later :)

[rafeca]

Wow! That was fast! Thanks a lot! ❤️

[rubennorte]

@SimenB I'm not sure I got that. Do you mean that we should revert this when we decide what to do with the serialization issue?

[SimenB]

Do you mean that we should revert this when we decide what to do with the serialization issue?

Yes :) I think we should normalize all config in jest-config before passing it on to other parts of jest

[rubennorte]

It's tricky when the workers are involved, but I agree.

[rubennorte]

I didn't notice that #7207 that was created as a prerequisite for #7209 and introduced changes to the original ValidConfig. I changed the new MultipleValidOptions thing to return the same array so when printing the configuration it's correct and minor changes in the naming.

[codecov-io]

Codecov Report

Merging #7251 into master will decrease coverage by 0.04%.
The diff coverage is 90%.

@@            Coverage Diff            @@
##           master   #7251      +/-   ##
=========================================
- Coverage   66.55%   66.5%   -0.05%     
=========================================
  Files         237     241       +4     
  Lines        9317    9328      +11     
  Branches        4       6       +2     
=========================================
+ Hits         6201    6204       +3     
- Misses       3115    3121       +6     
- Partials        1       3       +2

Impacted Files	Coverage Δ
packages/babel-jest/src/index.js	40.74% <ø> (ø)	⬆️
packages/jest-config/src/ValidConfig.js	100% <ø> (ø)	⬆️
packages/jest-validate/src/index.js	0% <0%> (ø)	⬆️
packages/jest-config/src/normalize.js	85.47% <100%> (-0.3%)	⬇️
packages/jest-runtime/src/should_instrument.js	100% <100%> (ø)	⬆️
packages/jest-cli/src/SearchSource.js	57.69% <100%> (ø)	⬆️
packages/jest-validate/src/condition.js	100% <100%> (ø)	⬆️
packages/jest-cli/src/getNoTestsFoundMessage.js	60% <0%> (ø)	⬆️
packages/jest-cli/src/getNoTestFoundFailed.js	0% <0%> (ø)	⬆️
... and 16 more

---

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 319329f...8c633db. Read the comment docs.

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.