netutil_test: Disable error tests on TwistedResolver

These tests are flaky with the new "messages to stderr are errors"
check.

diff --git a/tornado/test/netutil_test.py b/tornado/test/netutil_test.py
index 00fc6eaf46a8a4bcf380ebfc1e326f9b4b7689bb..f9879946fb6a4c1e1512578d4c32c29cb359d101 100644 (file)
--- a/tornado/test/netutil_test.py
+++ b/tornado/test/netutil_test.py
@@ -184,12 +184,16 @@ class CaresResolverTest(AsyncTestCase, _ResolverTestMixin):
 
 
 # TwistedResolver produces consistent errors in our test cases so we
-# can test the regular and error cases in the same class.
+# could test the regular and error cases in the same class. However,
+# in the error cases it appears that cleanup of socket objects is
+# handled asynchronously and occasionally results in "unclosed socket"
+# warnings if not given time to shut down (and there is no way to
+# explicitly shut it down). This makes the test flaky, so we do not
+# test error cases here.
 @skipIfNoNetwork
 @unittest.skipIf(twisted is None, "twisted module not present")
 @unittest.skipIf(getattr(twisted, '__version__', '0.0') < "12.1", "old version of twisted")
-class TwistedResolverTest(AsyncTestCase, _ResolverTestMixin,
-                          _ResolverErrorTestMixin):
+class TwistedResolverTest(AsyncTestCase, _ResolverTestMixin):
     def setUp(self):
         super(TwistedResolverTest, self).setUp()
         self.resolver = TwistedResolver()