Skip to content

Commit c0e0398

Browse files
authored
Merge pull request #76 from OpenNTI/fixing-tests
Use tox to run the tests; avoid an editable install because we're getting namespace errors
2 parents 29afdbe + 8acfa9d commit c0e0398

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

.github/workflows/tests.yml

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,15 +31,13 @@ jobs:
3131
cache-dependency-path: 'setup.py'
3232
- name: Install dependencies
3333
run: |
34-
python -m pip install -U pip setuptools wheel
35-
python -m pip install -U coverage zope.testrunner
36-
python -m pip install -U -e ".[test]"
34+
python -m pip install -U pip
35+
python -m pip install -U tox
3736
- name: Test
3837
run: |
39-
coverage run -m zope.testrunner --test-path=src
40-
coverage report
38+
tox -v -e py
4139
- name: Lint
4240
if: matrix.python-version == '3.12'
4341
run: |
44-
python -m pip install -U pylint
42+
python -m pip install -U pylint ".[test]"
4543
pylint src

tox.ini

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@ deps =
1010
zope.testrunner
1111
commands =
1212
coverage run -p -m zope.testrunner --test-path=src
13+
coverage combine
14+
coverage report
1315
passenv = HOME
1416

1517
[testenv:docs]

0 commit comments

Comments
 (0)