diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 10bd0449de12..66530c11c042 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -311,7 +311,7 @@ jobs: if: matrix.test-suite == 'venv' run: | python -m pip install virtualenv - python -m virtualenv .virtualenv/ + python -m virtualenv --no-venv-redirect .virtualenv/ if ('${{ matrix.os }}' -match 'windows-latest') { & ".virtualenv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} virtualEnvPath } else { @@ -326,10 +326,11 @@ jobs: if: matrix.test-suite == 'venv' && startsWith(matrix.python, 3.) run: | python -m venv .venv + if ($LASTEXITCODE -ne 0) { exit $LASTEXITCODE } if ('${{ matrix.os }}' -match 'windows-latest') { - & ".venv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath + & "./.venv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath } else { - & ".venv/bin/python" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath + & "./.venv/bin/python" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath } - name: Prepare conda for venv tests diff --git a/.github/workflows/pr-check.yml b/.github/workflows/pr-check.yml index 34869bfcd0cb..45079075899f 100644 --- a/.github/workflows/pr-check.yml +++ b/.github/workflows/pr-check.yml @@ -304,7 +304,7 @@ jobs: if: matrix.test-suite == 'venv' run: | python -m pip install virtualenv - python -m virtualenv .virtualenv/ + python -m virtualenv --no-venv-redirect .virtualenv/ if ('${{ matrix.os }}' -match 'windows-latest') { & ".virtualenv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} virtualEnvPath } else { @@ -319,10 +319,11 @@ jobs: if: matrix.test-suite == 'venv' && startsWith(matrix.python, 3.) run: | python -m venv .venv + if ($LASTEXITCODE -ne 0) { exit $LASTEXITCODE } if ('${{ matrix.os }}' -match 'windows-latest') { - & ".venv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath + & "./.venv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath } else { - & ".venv/bin/python" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath + & "./.venv/bin/python" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath } - name: Prepare conda for venv tests @@ -584,7 +585,7 @@ jobs: shell: pwsh run: | python -m pip install virtualenv - python -m virtualenv .virtualenv/ + python -m virtualenv --no-venv-redirect .virtualenv/ if ('${{ matrix.os }}' -match 'windows-latest') { & ".virtualenv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} virtualEnvPath } else { @@ -598,10 +599,11 @@ jobs: shell: pwsh run: | python -m venv .venv + if ($LASTEXITCODE -ne 0) { exit $LASTEXITCODE } if ('${{ matrix.os }}' -match 'windows-latest') { - & ".venv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath + & "./.venv/Scripts/python.exe" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath } else { - & ".venv/bin/python" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath + & "./.venv/bin/python" ./build/ci/addEnvPath.py ${{ env.PYTHON_VIRTUAL_ENVS_LOCATION }} venvPath } - name: Prepare conda for venv tests diff --git a/src/client/pythonEnvironments/base/locators/common/nativePythonFinder.ts b/src/client/pythonEnvironments/base/locators/common/nativePythonFinder.ts index ea0d63cd7552..7d0f7d103d76 100644 --- a/src/client/pythonEnvironments/base/locators/common/nativePythonFinder.ts +++ b/src/client/pythonEnvironments/base/locators/common/nativePythonFinder.ts @@ -120,7 +120,6 @@ class NativePythonFinderImpl extends DisposableBase implements NativePythonFinde ? getGlobalStorage(this.context, DONT_SHOW_SPAWN_ERROR_AGAIN, false) : ({ get: () => false, set: async () => {} } as IPersistentStorage); this.connection = this.start(); - void this.configure(); this.firstRefreshResults = this.refreshFirstTime(); } diff --git a/src/test/pythonEnvironments/nativePythonFinder.unit.test.ts b/src/test/pythonEnvironments/nativePythonFinder.unit.test.ts index b6182da8111f..597b99ecd0ea 100644 --- a/src/test/pythonEnvironments/nativePythonFinder.unit.test.ts +++ b/src/test/pythonEnvironments/nativePythonFinder.unit.test.ts @@ -21,10 +21,11 @@ suite('Native Python Finder', () => { let getConfigurationStub: sinon.SinonStub; let configMock: typemoq.IMock; let getWorkspaceFolderPathsStub: sinon.SinonStub; + const locatorOutput = new MockOutputChannel('locator'); setup(() => { createLogOutputChannelStub = sinon.stub(windowsApis, 'createLogOutputChannel'); - createLogOutputChannelStub.returns(new MockOutputChannel('locator')); + createLogOutputChannelStub.returns(locatorOutput); getWorkspaceFolderPathsStub = sinon.stub(workspaceApis, 'getWorkspaceFolderPaths'); getWorkspaceFolderPathsStub.returns([]); @@ -55,7 +56,7 @@ suite('Native Python Finder', () => { } // typically all test envs should have at least one environment - assert.isNotEmpty(envs); + assert.isNotEmpty(envs, `Python Locator output:\n${locatorOutput.output || '(none)'}`); }); test('Resolve should return python environments with version', async () => {