Use a terminal to get environment variables of an activated interpreter

[DonJayamanne]

For #8928
To be reviewed after #9739

[DonJayamanne]

Typical CI

[codecov-io]

Codecov Report

Merging #9744 into master will increase coverage by 0.12%.
The diff coverage is 91.61%.

@@            Coverage Diff             @@
##           master    #9744      +/-   ##
==========================================
+ Coverage   60.58%   60.71%   +0.12%     
==========================================
  Files         552      556       +4     
  Lines       29860    29972     +112     
  Branches     4529     4546      +17     
==========================================
+ Hits        18092    18198     +106     
+ Misses      10750    10748       -2     
- Partials     1018     1026       +8

Impacted Files	Coverage Δ
src/client/common/terminal/types.ts	100% <ø> (ø)	⬆️
src/client/telemetry/index.ts	85.58% <ø> (ø)	⬆️
src/client/common/terminal/factory.ts	82.85% <0%> (-2.44%)	⬇️
src/client/datascience/preWarmVariables.ts	100% <100%> (ø)
src/client/interpreter/activation/service.ts	86.95% <100%> (+2.02%)	⬆️
...activation/terminalEnvironmentActivationService.ts	100% <100%> (ø)
src/client/common/constants.ts	100% <100%> (ø)	⬆️
.../client/interpreter/activation/preWarmVariables.ts	100% <100%> (ø)
src/client/terminals/activation.ts	84.21% <100%> (+0.87%)	⬆️
src/client/common/experimentGroups.ts	100% <100%> (ø)	⬆️
... and 9 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 e6707d6...2050926. Read the comment docs.

[rchiodo]

export class InMemoryInterpreterSpecificCache {

Could this one derive from the new one you made? Looks pretty similar.

---

Refers to: src/client/common/utils/cacheUtils.ts:81 in ae7ca3c. [](commit_id = ae7ca3c, deletion_comment = False)

[rchiodo]

🕐

[DonJayamanne]

Could this one derive from the new one you made? Looks pretty similar.

@rchiodo Yes, but that would complicate it. Hence choose not to
Right now both use different caching providers. Also, one is specific to interpreters and other is a general caching class.

[rchiodo]

Okay then how bout a common base class.

---

In reply to: 577929350 [](ancestors = 577929350)

[DonJayamanne]

Okay then how bout a common base class.

@rchiodo
Good suggestion, cleaned up.

[rchiodo]

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

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

No Coverage information
0.0% Duplication