Skip to content

Commit 50ccf2c

Browse files
committed
fix injection
1 parent 2d334f5 commit 50ccf2c

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

src/features/terminal/terminalEnvVarInjector.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import {
1313
} from 'vscode';
1414
import { traceError, traceVerbose } from '../../common/logging';
1515
import { resolveVariables } from '../../common/utils/internalVariables';
16-
import { getConfiguration, getWorkspaceFolder } from '../../common/workspace.apis';
16+
import { getConfiguration, getWorkspaceFolder, onDidChangeConfiguration } from '../../common/workspace.apis';
1717
import { EnvVarManager } from '../execution/envVariableManager';
1818

1919
/**
@@ -81,7 +81,7 @@ export class TerminalEnvVarInjector implements Disposable {
8181

8282
// Listen for changes to the python.envFile setting
8383
this.disposables.push(
84-
workspace.onDidChangeConfiguration((e) => {
84+
onDidChangeConfiguration((e) => {
8585
if (e.affectsConfiguration('python.envFile')) {
8686
traceVerbose(
8787
'TerminalEnvVarInjector: python.envFile setting changed, updating environment variables',

src/test/features/terminalEnvVarInjectorBasic.unit.test.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import * as sinon from 'sinon';
55
import * as typeMoq from 'typemoq';
66
import { Disposable, GlobalEnvironmentVariableCollection, workspace, WorkspaceFolder } from 'vscode';
77
import { DidChangeEnvironmentVariablesEventArgs } from '../../api';
8+
import * as workspaceApis from '../../common/workspace.apis';
89
import { EnvVarManager } from '../../features/execution/envVariableManager';
910
import { TerminalEnvVarInjector } from '../../features/terminal/terminalEnvVarInjector';
1011

@@ -56,7 +57,7 @@ suite('TerminalEnvVarInjector Basic Tests', () => {
5657
});
5758

5859
// Mock workspace.onDidChangeConfiguration to return a Disposable
59-
sinon.stub(workspace, 'onDidChangeConfiguration').returns({
60+
sinon.stub(workspaceApis, 'onDidChangeConfiguration').returns({
6061
dispose: () => {},
6162
} as Disposable);
6263
});
@@ -102,7 +103,7 @@ suite('TerminalEnvVarInjector Basic Tests', () => {
102103
});
103104

104105
// Mock workspace.onDidChangeConfiguration to return a Disposable
105-
sinon.stub(workspace, 'onDidChangeConfiguration').returns({
106+
sinon.stub(workspaceApis, 'onDidChangeConfiguration').returns({
106107
dispose: () => {},
107108
} as Disposable);
108109

0 commit comments

Comments
 (0)