Skip to content

Commit 5b241fe

Browse files
committed
Extend ci coverage to ms-win/macos, fix ns issue in test for py>3.10
1 parent fee7f41 commit 5b241fe

File tree

2 files changed

+53
-1
lines changed

2 files changed

+53
-1
lines changed

.github/workflows/test.yml

Lines changed: 52 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,52 @@
1+
name: test
2+
3+
on:
4+
push:
5+
branches:
6+
- master
7+
paths-ignore:
8+
- '**/README.md'
9+
- '**/CHANGELOG.md'
10+
pull_request:
11+
12+
jobs:
13+
unit-test:
14+
runs-on: ${{matrix.os}}
15+
strategy:
16+
fail-fast: false
17+
matrix:
18+
os: [macos-latest, ubuntu-latest, windows-latest]
19+
jdk: [8, 17, 19]
20+
python-version: ["3.9", "3.11"]
21+
22+
steps:
23+
- uses: actions/checkout@v3
24+
25+
- name: Set up JDK ${{ matrix.jdk }}
26+
uses: actions/setup-java@v1
27+
with:
28+
java-version: ${{ matrix.jdk }}
29+
30+
- name: Install Clojure
31+
uses: DeLaGuardo/setup-clojure@11.0
32+
with:
33+
cli: 1.11.1.1347
34+
35+
- name: Set up Python ${{ matrix.python-version }}
36+
uses: actions/setup-python@v4
37+
with:
38+
python-version: ${{ matrix.python-version }}
39+
40+
- name: Install dependencies
41+
run: |
42+
python -m pip install --upgrade pip
43+
pip install numpy
44+
45+
- name: Run tests (jdk<17)
46+
if: ${{ matrix.jdk < 17 }}
47+
run: |
48+
clojure -M:test
49+
- name: Run tests (jdk>=17)
50+
if: ${{ matrix.jdk >= 17 }}
51+
run: |
52+
clojure -M:jdk-${{matrix.jdk}}:test

test/libpython_clj2/python_test.clj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -365,7 +365,7 @@ class Foo:
365365
bridged-dict (py/as-python {"a" 1 "b" 2})
366366
bridged-iter (py/as-python (repeat 5 1))
367367
bridged-list (py/as-python (vec (range 10)))
368-
pycol (py/import-module "collections")
368+
pycol (py/import-module "collections.abc")
369369
mapping-type (py/get-attr pycol "Mapping")
370370
iter-type (py/get-attr pycol "Iterable")
371371
sequence-type (py/get-attr pycol "Sequence")]

0 commit comments

Comments
 (0)