Skip to content

Commit

Permalink
tests: fix for release candidates (and test on more versions of Python)
Browse files Browse the repository at this point in the history
Fixes #146

Includes loosening the constraints on versions used to run tests in CI (e.g.
pytest, coverage) since the older versions don't seem to work on newer Python.
Specifically, coverage 6.2 bombs out with an error on Python 3.11)
  • Loading branch information
michalc committed Sep 20, 2024
1 parent 8dd1382 commit a936d5d
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 7 deletions.
10 changes: 8 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,11 +24,17 @@ jobs:
os: "ubuntu-20.04"
- python-version: "3.10.0"
os: "ubuntu-20.04"
- python-version: "3.11.0"
os: "ubuntu-20.04"
- python-version: "3.12.0"
os: "ubuntu-20.04"
- python-version: "3.13.0-rc.2"
os: "ubuntu-20.04"
runs-on: ${{ matrix.os }}
steps:
- name: "Checkout"
uses: "actions/checkout@v3"
- uses: "actions/setup-python@v4"
- uses: "actions/setup-python@v5"
with:
python-version: '${{ matrix.python-version }}'
- name: "Install bsdcpio"
Expand All @@ -47,7 +53,7 @@ jobs:
)
- name: "Install python dependencies"
run: |
pip install ".[ci]"
pip install ".[dev,ci]"
- name: "Run type checking"
run: |
mypy stream_zip --strict
Expand Down
3 changes: 0 additions & 3 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,6 @@ dev = [
]
ci = [
"pycryptodome==3.10.1",
"coverage==6.2",
"pytest==7.0.1",
"pytest-cov==3.0.0",
"stream-unzip==0.0.86",
"pyzipper==0.3.6",
# Type checking
Expand Down
4 changes: 2 additions & 2 deletions test_stream_zip.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
import asyncio
import contextlib
import os
import platform
import secrets
import sys
import stat
import subprocess
import zlib
Expand Down Expand Up @@ -1388,7 +1388,7 @@ async def test():


@pytest.mark.skipif(
tuple(int(v) for v in platform.python_version().split('.')) < (3,7,0),
sys.version_info[:2] < (3,7,0),
reason="contextvars are not supported before Python 3.7.0",
)
def test_copy_of_context_variable_available_in_iterable():
Expand Down

0 comments on commit a936d5d

Please sign in to comment.