Skip to content

Commit 45ef0e1

Browse files
author
Kartik Raj
committed
Why are conda tests getting skipped
1 parent 75ef9f8 commit 45ef0e1

2 files changed

Lines changed: 22 additions & 23 deletions

File tree

.github/workflows/pr-check.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,6 @@ jobs:
195195
- name: Run venv tests
196196
env:
197197
TEST_FILES_SUFFIX: testvirtualenvs
198-
VSC_PYTHON_CI_TEST_GREP: 'Activation of Environments in Terminal'
199198
CI_PYTHON_VERSION: ${{matrix.python}}
200199
uses: GabrielBB/xvfb-action@v1.5
201200
with:

src/test/common/terminals/environmentActivationProviders/terminalActivation.testvirtualenvs.ts

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -163,29 +163,29 @@ suite('Activation of Environments in Terminal', () => {
163163
expect(fileSystem.arePathsSame(content, envPath)).to.equal(true, 'Environment not activated');
164164
}
165165

166-
// test('Should not activate', async () => {
167-
// await updateSetting(
168-
// 'terminal.activateEnvironment',
169-
// false,
170-
// vscode.workspace.workspaceFolders![0].uri,
171-
// vscode.ConfigurationTarget.WorkspaceFolder,
172-
// );
173-
// const content = await openTerminalAndAwaitCommandContent(waitTimeForActivation, file, outputFile, 5_000);
174-
// expect(fileSystem.arePathsSame(content, PYTHON_PATH)).to.equal(false, 'Environment not activated');
175-
// });
166+
test('Should not activate', async () => {
167+
await updateSetting(
168+
'terminal.activateEnvironment',
169+
false,
170+
vscode.workspace.workspaceFolders![0].uri,
171+
vscode.ConfigurationTarget.WorkspaceFolder,
172+
);
173+
const content = await openTerminalAndAwaitCommandContent(waitTimeForActivation, file, outputFile, 5_000);
174+
expect(fileSystem.arePathsSame(content, PYTHON_PATH)).to.equal(false, 'Environment not activated');
175+
});
176176

177-
// test('Should activate with venv', async function () {
178-
// if (process.env.CI_PYTHON_VERSION && process.env.CI_PYTHON_VERSION.startsWith('2.')) {
179-
// this.skip();
180-
// }
181-
// await testActivation(envPaths.venvPath);
182-
// });
183-
// test('Should activate with pipenv', async () => {
184-
// await testActivation(envPaths.pipenvPath);
185-
// });
186-
// test('Should activate with virtualenv', async () => {
187-
// await testActivation(envPaths.virtualEnvPath);
188-
// });
177+
test('Should activate with venv', async function () {
178+
if (process.env.CI_PYTHON_VERSION && process.env.CI_PYTHON_VERSION.startsWith('2.')) {
179+
this.skip();
180+
}
181+
await testActivation(envPaths.venvPath);
182+
});
183+
test('Should activate with pipenv', async () => {
184+
await testActivation(envPaths.pipenvPath);
185+
});
186+
test('Should activate with virtualenv', async () => {
187+
await testActivation(envPaths.virtualEnvPath);
188+
});
189189
test('Should activate with conda', async () => {
190190
await terminalSettings.update(
191191
'integrated.shell.windows',

0 commit comments

Comments
 (0)