gh-116622: Test updates for Android#117299
Conversation
mhsmith
requested review from
brettcannon,
ericsnowcurrently,
gpshead,
ncoghlan and
warsaw
as code owners
| @unittest.skipIf( | ||
| sys.platform.startswith(("freebsd", "wasi")) | ||
| or (sys.platform == "android" and platform.machine() == "x86_64"), | ||
| f"this platform doesn't implement IEE 754-2008 properly") | ||
| def test_fma_zero_result(self): |
|
@erlend-aasland: Are you able to review this PR? It's a continuation of the previous Android test updates from #115918. |
erlend-aasland
left a comment
erlend-aasland
left a comment
There was a problem hiding this comment.
LGTM, with some nits. BTW, last time I checked, the sys.platform.startswith idiom was still recommended practice. I'd prefer if we could follow that.
Lib/test/test_os.py
Outdated
| # The Android testbed redirects the native stdout to a pipe, | ||
| # which returns a different error code. | ||
| known_errnos.append(errno.EACCES) | ||
| if (sys.platform == "win32") or (e.errno in known_errnos): |
There was a problem hiding this comment.
The parentheses are not needed.
Fixed.
I actually updated the |
|
@mhsmith, thanks for the heads-up, I missed that docs update (and I've must have looked at the 3.12 docs). Perhaps we should consider backporting that doc patch. |
- re-enable test_fcntl_64_bit on Linux aarch64, but disable it on all Android ABIs - use support.setswitchinterval in all relevant tests - skip test_fma_zero_result on Android x86_64 - accept EACCES when calling os.get_terminal_size on Android
2 participants
This PR fixes various Android test issues which I've discovered over the last few weeks.