Skip to content

Commit

Permalink
Do not run tests on Python 3.8, they fail on all platforms
Browse files Browse the repository at this point in the history
  • Loading branch information
sjaensch committed Oct 2, 2020
1 parent 21ee18a commit 1a49dde
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest, windows-latest]
python-version: ['3.6', '3.7', '3.8']
python-version: ['3.6', '3.7']

name: Python ${{ matrix.python-version }} on ${{ matrix.os }}
runs-on: ${{ matrix.os }}
Expand Down
5 changes: 5 additions & 0 deletions tests/future_adapter_test.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import asyncio
import concurrent.futures
import sys
import time

import mock
Expand Down Expand Up @@ -57,6 +58,10 @@ def test_future_adapter(mock_future, mock_response):
assert 0 < result.remaining_timeout < 5


@pytest.mark.skipif(
sys.version_info >= (3, 8),
reason="This does not seem to hold true for Python 3.8+ anymore",
)
def test_future_adapter_timeout_error_class():
"""Let's make sure refactors never break timeout errors"""
assert concurrent.futures.TimeoutError in AsyncioFutureAdapter.timeout_errors
Expand Down

0 comments on commit 1a49dde

Please sign in to comment.