Fix tests on windows

bdarnell · bdarnell · commit b76e79611c58 · 2012-08-28T15:24:55.000-04:00
diff --git a/tornado/test/options_test.py b/tornado/test/options_test.py
@@ -63,6 +63,7 @@ def setUp(self):
         self.logger.addHandler(self.handler)
 
     def tearDown(self):
+        self.handler.close()
         os.unlink(self.filename)
         os.rmdir(self.tempdir)
 
diff --git a/tornado/test/simple_httpclient_test.py b/tornado/test/simple_httpclient_test.py
@@ -2,10 +2,13 @@
 
 import collections
 from contextlib import closing
+import errno
 import gzip
 import logging
+import os
 import re
 import socket
+import sys
 
 from tornado.httpclient import AsyncHTTPClient
 from tornado.httputil import HTTPHeaders
@@ -284,10 +287,20 @@ def test_host_header(self):
         self.assertTrue(host_re.match(response.body), response.body)
 
     def test_connection_refused(self):
-        self.http_client.fetch("http://localhost:1/", self.stop)
+        port = get_unused_port()
+        self.http_client.fetch("http://localhost:%d/" % port, self.stop)
         response = self.wait()
         self.assertEqual(599, response.code)
-        self.assertTrue("Connection refused" in str(response.error))
+
+        if sys.platform != 'cygwin':
+            # cygwin returns EPERM instead of ECONNREFUSED here
+            self.assertTrue(str(errno.ECONNREFUSED) in str(response.error),
+                            response.error)
+            # This is usually "Connection refused".
+            # On windows, strerror is broken and returns "Unknown error".
+            expected_message = os.strerror(errno.ECONNREFUSED)
+            self.assertTrue(expected_message in str(response.error),
+                            response.error)
 
 
 class CreateAsyncHTTPClientTestCase(AsyncTestCase, LogTrapTestCase):
