Skip to content

Commit 159e50e

Browse files
authored
Merge pull request #71 from jayvdb/tests-win
Fix tests on Windows
2 parents 47dfa4e + 1d9db3a commit 159e50e

File tree

2 files changed

+11
-7
lines changed

2 files changed

+11
-7
lines changed

.github/workflows/build.yml

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,15 +11,19 @@ on:
1111
- master
1212

1313
jobs:
14-
build-linux:
15-
runs-on: ubuntu-latest
14+
build:
15+
16+
strategy:
17+
matrix:
18+
os:
19+
- ubuntu-22.04
20+
- macos-12
21+
- windows-2022
22+
runs-on: ${{ matrix.os }}
1623

1724
steps:
1825
- uses: actions/checkout@v2
19-
- name: Add conda to system path
20-
run: |
21-
# $CONDA is an environment variable pointing to the root of the miniconda directory
22-
echo $CONDA/bin >> $GITHUB_PATH
26+
- uses: s-weigand/setup-conda@v1
2327
- name: Install mamba
2428
run: |
2529
conda install -c conda-forge mamba=0.20.0

tests/test_managed.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ def resolve_managed(endpoint, cache_dir, m2_repo):
3535

3636
def find_jar_matching(jars, pattern):
3737
for jar in jars:
38-
lastindex = jar.rindex('/')
38+
lastindex = jar.rindex(os.sep)
3939
if jar[lastindex:].find(pattern) != -1:
4040
return jar
4141
return None

0 commit comments

Comments
 (0)