fix(read-config): allow multiple projects with programmatic usage

[nicojs]

Summary

Allow for runCli with multiple projects.

Fixes #11213
Fixes #7415

const { runCLI } = require('jest');

const config = {
  projects: [ 
    { testMatch: [ "<rootDir>/client/**/*.test.js" ] }, 
    { testMatch: [ "<rootDir>/server/**/*.test.js" ] }
  ]
};

runCLI({ config: JSON.stringify(config) }, [process.cwd()])
 .then(() => console.log('✅ Done'))
 .catch(err => {
    console.error(err);
    process.exitCode = 1;
});

Test plan

I've tested this change in the reproduction zip of #11213. Output was:

[nicojs]

Please let me know if you want me to add an integration test for this.

[codecov-commenter]

Codecov Report

Merging #11307 (04d329b) into master (1be8d73) will decrease coverage by 0.01%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##           master   #11307      +/-   ##
==========================================
- Coverage   64.19%   64.18%   -0.02%     
==========================================
  Files         308      308              
  Lines       13528    13529       +1     
  Branches     3297     3298       +1     
==========================================
- Hits         8684     8683       -1     
  Misses       4131     4131              
- Partials      713      715       +2     

Impacted Files	Coverage Δ
packages/jest-config/src/index.ts	54.16% <25.00%> (+0.64%)	⬆️
packages/expect/src/utils.ts	94.83% <0.00%> (-1.30%)	⬇️

---

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 1be8d73...04d329b. Read the comment docs.

[SimenB]

ooh, nice! An e2e test for this would be great, yeah 🙂 Along with a changelog entry

[nicojs]

Thanks for the review @SimenB. I've added an e2e test, as well as the changelog entry. 👷‍♂️✅

[SimenB]

thanks! Just test nitpicks 🙂

[SimenB]

could you pass stdout through extractSummaries or something like that and snapshot it to ensure both tests show up?

[nicojs]

A great addition, will do that and use snapshotting 👍

[SimenB]

Also, CI complains about missing copyright headers 🙂

[nicojs]

Great feedback 👍

I've implemented all of it and added the copyright header (what a waste of disk space to have that in each file, but what can you do I guess 🤷‍♂️)

[SimenB]

I tweaked the test, btw. But it seems GH Actions does not like it. Maybe we need to pass the output through strip-ansi first?

[nicojs]

Hmm if I run the tests locally and update the snapshots I see this diff:

Maybe we should sort the test results? Is there a helper for that?

[SimenB]

Ah, flake in addition. We don't really care about rest, just summary is enough (as it says "2"). We don't care about the order they complete

[nicojs]

Ah yes, I see what you mean. I've removed rest. Let's see some green 💚🍿

[SimenB]

I think we still need to pass through strip-ansi for GH actions

[nicojs]

Ok, I think this should be it then 🍏🟢

[SimenB]

Thanks!

[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.