Fix multiroot flaky CI tests

[karrtikr]

For #11477

[codecov-io]

Codecov Report

Merging #11544 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11544      +/-   ##
==========================================
- Coverage   60.46%   60.43%   -0.03%     
==========================================
  Files         618      618              
  Lines       33776    33774       -2     
  Branches     4764     4763       -1     
==========================================
- Hits        20421    20413       -8     
- Misses      12347    12351       +4     
- Partials     1008     1010       +2     

Impacted Files	Coverage Δ
src/datascience-ui/react-common/arePathsSame.ts	75.00% <0.00%> (-12.50%)	⬇️
src/client/common/utils/platform.ts	64.70% <0.00%> (-11.77%)	⬇️
src/client/linters/pydocstyle.ts	86.66% <0.00%> (-2.23%)	⬇️
src/client/datascience/debugLocationTracker.ts	76.56% <0.00%> (-1.57%)	⬇️
src/client/common/process/proc.ts	14.49% <0.00%> (-0.73%)	⬇️
...client/datascience/kernel-launcher/kernelFinder.ts	63.96% <0.00%> (-0.64%)	⬇️

---

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 262c14f...e715f8d. Read the comment docs.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

No Coverage information
0.4% Duplication

[karthiknadig]

This is magic.

[DonJayamanne]

How does this make any difference? It's a noop

[karrtikr]

This didn't change anything. It's the other line I added. The variable was not set earlier when running multiroot tests.

[kimadeline]

As Karthik said, this is magic ✨ you explain how adding process.env.VSC_PYTHON_CI_TEST fixes the flakiness?

[karrtikr]

@kimadeline @karthiknadig If process.env.VSC_PYTHON_CI_TEST is not set, api object is returned with the serviceContainer as undefined, which is why using serviceContainer.get failed in tests.

Here's the implementation

    if (process.env.VSC_PYTHON_CI_TEST === '1' || isUnitTestExecution()) {
        // tslint:disable:no-any
        (api as any).serviceContainer = serviceContainer;
        (api as any).serviceManager = serviceManager;
        // tslint:enable:no-any
    }

So I explicitly set it before launching multiroot tests, it was already being done for single workspace and other types of tests.