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

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 4a0e29c
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 5 deletions.
8 changes: 7 additions & 1 deletion .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 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
2 changes: 1 addition & 1 deletion test_stream_zip.py
Original file line number Diff line number Diff line change
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),
tuple(int(v) for v in platform.python_version().split('.')[: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 4a0e29c

Please sign in to comment.