Skip to content

Commit 49d8e41

Browse files
committed
update where isWindows is
1 parent e2a94b5 commit 49d8e41

File tree

5 files changed

+9
-9
lines changed

5 files changed

+9
-9
lines changed

src/client/common/configSettings.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,7 @@ import {
3636
} from './types';
3737
import { debounceSync } from './utils/decorators';
3838
import { SystemVariables } from './variables/systemVariables';
39-
import { getOSType, OSType } from './utils/platform';
40-
import { isWindows } from './platform/platformService';
39+
import { getOSType, OSType, isWindows } from './utils/platform';
4140
import { untildify } from './helpers';
4241

4342
export class PythonSettings implements IPythonSettings {

src/client/common/platform/platformService.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import { injectable } from 'inversify';
77
import * as os from 'os';
88
import { coerce, SemVer } from 'semver';
99
import { getSearchPathEnvVarNames } from '../utils/exec';
10-
import { Architecture, getArchitecture, getOSType, OSType } from '../utils/platform';
10+
import { Architecture, getArchitecture, getOSType, isWindows, OSType } from '../utils/platform';
1111
import { parseSemVerSafe } from '../utils/version';
1212
import { IPlatformService } from './types';
1313

@@ -73,7 +73,3 @@ export class PlatformService implements IPlatformService {
7373
return getArchitecture() === Architecture.x64;
7474
}
7575
}
76-
77-
export function isWindows(): boolean {
78-
return getOSType() === OSType.Windows;
79-
}

src/client/common/terminal/service.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import {
2121
} from './types';
2222
import { traceVerbose } from '../../logging';
2323
import { getConfiguration } from '../vscodeApis/workspaceApis';
24-
import { isWindows } from '../platform/platformService';
24+
import { isWindows } from '../utils/platform';
2525

2626
@injectable()
2727
export class TerminalService implements ITerminalService, Disposable {

src/client/common/utils/platform.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,3 +67,7 @@ export function getUserHomeDir(): string | undefined {
6767
}
6868
return getEnvironmentVariable('HOME') || getEnvironmentVariable('HOMEPATH');
6969
}
70+
71+
export function isWindows(): boolean {
72+
return getOSType() === OSType.Windows;
73+
}

src/test/common/terminals/service.unit.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ import { IServiceContainer } from '../../../client/ioc/types';
2424
import { ITerminalAutoActivation } from '../../../client/terminals/types';
2525
import { createPythonInterpreter } from '../../utils/interpreters';
2626
import * as workspaceApis from '../../../client/common/vscodeApis/workspaceApis';
27+
import * as platform from '../../../client/common/utils/platform';
2728

2829
suite('Terminal Service', () => {
2930
let service: TerminalService;
@@ -95,7 +96,7 @@ suite('Terminal Service', () => {
9596
mockServiceContainer.setup((c) => c.get(ITerminalActivator)).returns(() => terminalActivator.object);
9697
mockServiceContainer.setup((c) => c.get(ITerminalAutoActivation)).returns(() => terminalAutoActivator.object);
9798
getConfigurationStub = sinon.stub(workspaceApis, 'getConfiguration');
98-
isWindowsStub = sinon.stub(platformService.object, 'isWindows');
99+
isWindowsStub = sinon.stub(platform, 'isWindows');
99100
pythonConfig = TypeMoq.Mock.ofType<WorkspaceConfiguration>();
100101
editorConfig = TypeMoq.Mock.ofType<WorkspaceConfiguration>();
101102
getConfigurationStub.callsFake((section: string) => {

0 commit comments

Comments
 (0)