Merge pull request #2039 from graingert/fix-ssl-tests-sys-platform

fix sys.platform/sys.implementation.name confusion
python-trio · Jun 14, 2021 · 6c0428b · 6c0428b
2 parents efab757 + c575703
commit 6c0428b
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/trio/tests/test_ssl.py b/trio/tests/test_ssl.py
@@ -111,7 +111,10 @@ def ssl_echo_serve_sync(sock, *, expect_fail=False):
                         # it's fixed in 'recent' CPython versions so we fail
                         # the test for those and ignore it for earlier
                         # versions.
-                        if sys.platform != "cpython" or sys.version_info >= (3, 8):
+                        if (
+                            sys.implementation.name != "cpython"
+                            or sys.version_info >= (3, 8)
+                        ):
                             pytest.fail(
                                 "still an issue on recent python versions "
                                 "add a comment to "