Skip to content

Commit

Permalink
Revert debugging
Browse files Browse the repository at this point in the history
  • Loading branch information
kenodegard committed Jan 28, 2025
1 parent 7e70f02 commit 8c80f63
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 23 deletions.
29 changes: 10 additions & 19 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,19 +76,17 @@ jobs:
# test lower version (w/ defaults) and upper version (w/ defaults and conda-forge)
python-version: ['3.9', '3.12']
default-channel: [defaults, conda-forge]
test-type: [integration]
# test-type: [unit, integration]
test-group: [1]
# test-group: [1, 2, 3]
test-type: [unit, integration]
test-group: [1, 2, 3]
exclude:
- default-channel: conda-forge
python-version: '3.9'
# - test-type: unit
# test-group: 3
- test-type: unit
test-group: 3
env:
ErrorActionPreference: Stop # powershell exit immediately on error
PYTEST_MARKER: ${{ matrix.test-type == 'unit' && 'not integration' || 'integration' }}
PYTEST_SPLITS: '1' # ${{ matrix.test-type == 'unit' && '2' || '3' }}
PYTEST_SPLITS: ${{ matrix.test-type == 'unit' && '2' || '3' }}
REQUIREMENTS_TRUSTSTORE: ${{ contains('3.10|3.11|3.12', matrix.python-version) && '--file tests\requirements-truststore.txt' || '' }}
CONDA_TEST_SOLVERS: ${{ github.event_name == 'pull_request' && 'libmamba' || 'libmamba,classic' }}

Expand Down Expand Up @@ -140,17 +138,11 @@ jobs:
run: conda list --show-channel-urls

- name: Setup Powershell
shell: pwsh
run: |
Invoke-Expression "& { $(Invoke-RestMethod https://aka.ms/install-powershell.ps1) }"
"PWSHPATH=$env:LOCALAPPDATA\Microsoft\powershell" | Out-File -FilePath $env:GITHUB_ENV -Append
- name: Powershell Info
shell: pwsh
run: $PSVersionTable

- name: Run Tests
shell: pwsh
# Windows is sensitive to long paths, using `--basetemp=${{ runner.temp }} to
# keep the test directories shorter
run: >
Expand All @@ -161,7 +153,6 @@ jobs:
--group=${{ matrix.test-group }}
--splits=${{ env.PYTEST_SPLITS }}
-m "${{ env.PYTEST_MARKER }}"
-k test_powershell_basic_integration
- name: Upload Coverage
uses: codecov/codecov-action@1e68e06f1dbfde0e4cefc87efeba9e4643565303 # v5.1.2
Expand All @@ -185,7 +176,7 @@ jobs:
linux:
# only run test suite if there are code changes
needs: changes
if: false # github.event_name == 'schedule' || needs.changes.outputs.code == 'true'
if: github.event_name == 'schedule' || needs.changes.outputs.code == 'true'

runs-on: ubuntu-latest
defaults:
Expand Down Expand Up @@ -288,7 +279,7 @@ jobs:
linux-benchmarks:
# only run test suite if there are code changes
needs: changes
if: false # needs.changes.outputs.code == 'true'
if: needs.changes.outputs.code == 'true'

runs-on: ubuntu-22.04
defaults:
Expand Down Expand Up @@ -355,7 +346,7 @@ jobs:
linux-qemu:
# only run test suite if there are code changes
needs: changes
if: false # github.event_name == 'schedule' || needs.changes.outputs.code == 'true'
if: github.event_name == 'schedule' || needs.changes.outputs.code == 'true'

# Run one single fast test per docker+qemu emulated linux platform to test that
# test execution is possible there (container+tools+dependencies work). Can be
Expand Down Expand Up @@ -418,7 +409,7 @@ jobs:
linux-memray:
# only run test suite if there are code changes
needs: changes
if: false # needs.changes.outputs.code == 'true'
if: needs.changes.outputs.code == 'true'

runs-on: ubuntu-latest
defaults:
Expand Down Expand Up @@ -480,7 +471,7 @@ jobs:
macos:
# only run test suite if there are code changes
needs: changes
if: false # github.event_name == 'schedule' || needs.changes.outputs.code == 'true'
if: github.event_name == 'schedule' || needs.changes.outputs.code == 'true'

runs-on: ${{ (matrix.arch == 'osx-64' && 'macos-13') || 'macos-14' }}
defaults:
Expand Down
6 changes: 2 additions & 4 deletions conda/base/context.py
Original file line number Diff line number Diff line change
Expand Up @@ -835,12 +835,10 @@ def conda_exe_vars_dict(self):
}
else:
exe = "conda.exe" if on_win else "conda"
# I was going to use None to indicate a variable to unset, but that gets tricky with
# error-on-undefined.
return {
"CONDA_EXE": os.path.join(sys.prefix, BIN_DIRECTORY, exe),
"_CE_M": "",
"_CE_CONDA": "",
"_CE_M": None,
"_CE_CONDA": None,
"CONDA_PYTHON_EXE": sys.executable,
}

Expand Down

0 comments on commit 8c80f63

Please sign in to comment.