Skip to content

Commit

Permalink
Now that I enabled c-ares tests on all Linux builds, there is one get…
Browse files Browse the repository at this point in the history
…addrinfo test in 3.12 that has to be disabled; ares doesn't like that syntax.
  • Loading branch information
jamadden committed Feb 13, 2024
1 parent a0b3caf commit 989c2c0
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/gevent/resolver/ares.py
Expand Up @@ -94,6 +94,8 @@ class Resolver(AbstractResolver):
- ``getaddrinfo`` may return canonical names even without the ``AI_CANONNAME``
being set.
- ``getaddrinfo`` does not appear to support IPv6 symbolic scope IDs.
.. caution::
This module is considered extremely experimental on PyPy, and
Expand Down
6 changes: 6 additions & 0 deletions src/gevent/testing/patched_tests_setup.py
Expand Up @@ -1298,6 +1298,12 @@ def test(*args, **kwargs):
'test_ssl.TestPreHandshakeClose.test_preauth_data_to_tls_server',
]

if ARES:
disabled_tests += [
# c-ares doesn't like the IPv6 syntax it uses here.
'test_socket.GeneralModuleTeststest_getaddrinfo_ipv6_scopeid_symbolic',
]

if TRAVIS:
disabled_tests += [
# These tests frequently break when we try to use newer Travis CI images,
Expand Down

0 comments on commit 989c2c0

Please sign in to comment.