Skip to content

fix: Skip flaky pager test on macOS with free-threaded Python 3.14t#3470

Merged
davidism merged 1 commit into
pallets:stablefrom
Jazzcort:skip-flaky-tests
May 18, 2026
Merged

fix: Skip flaky pager test on macOS with free-threaded Python 3.14t#3470
davidism merged 1 commit into
pallets:stablefrom
Jazzcort:skip-flaky-tests

Conversation

@Jazzcort

@Jazzcort Jazzcort commented May 18, 2026

Copy link
Copy Markdown
Contributor

Temporarily fixes flaky test failure for test_echo_via_pager on macOS with 3.14t (free-threading) by skipping them for macOS + Python 3.14t combination until we figure out how to make the test result consistently pass when test_echo_via_pager and test_stream_lifecycle are running in parallel.

fixes #2899

@Jazzcort Jazzcort force-pushed the skip-flaky-tests branch 3 times, most recently from 351484d to aa9442c Compare May 18, 2026 20:15
Add a MAC platform constant in `_compat.py` and use it to skip
`test_echo_via_pager` on macOS when running free-threaded Python,
where generator exception tests are flaky in parallel execution.
@Jazzcort Jazzcort force-pushed the skip-flaky-tests branch from aa9442c to d15f3c2 Compare May 18, 2026 21:01
@davidism davidism merged commit bee3314 into pallets:stable May 18, 2026
12 checks passed
@Jazzcort

Copy link
Copy Markdown
Contributor Author

@davidism @Rowlando13
Thank you so much for the guidance! 😁

@Jazzcort Jazzcort deleted the skip-flaky-tests branch May 18, 2026 21:13
@Rowlando13 Rowlando13 added this to the 8.4.1 milestone May 18, 2026
@kdeldycke kdeldycke added bug f:test runner feature: cli test runner labels May 21, 2026
@davidism

Copy link
Copy Markdown
Member

Oops, got this backwards, should be not sys._is_gil_enabled(). I'll fix in a second.

@davidism

Copy link
Copy Markdown
Member

6cb3477

@Jazzcort

Copy link
Copy Markdown
Contributor Author

@davidism My bad! Should have caught it earlier. 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug f:test runner feature: cli test runner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

running tests in parallel with tox p fails

4 participants