Make socket.error with errno EBADF get raised as OSError.

Fixes #17
python · itsmekarthikreddy · Jan 14, 2015 · Jan 14, 2015 · Jan 14, 2015 · Jan 14, 2015
commit a74449ad5e7de34ca6cadedd062d9ee4307db911
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -0,0 +1,8 @@
+=========
+ Changes
+=========
+
+2.2.1 (unreleased)
+==================
+
+- Properly reraise socket.error with an errno of EBADF as an OSError.
diff --git a/MANIFEST.in b/MANIFEST.in
@@ -14,3 +14,5 @@ include examples/*.py
 
 include tests/*.crt tests/*.pem tests/*.key
 include tests/*.py
+
+include *.rst
diff --git a/runtests.py b/runtests.py
@@ -151,7 +151,7 @@ def randomize_tests(tests, seed):
     random.shuffle(tests._tests)
 
 
-class TestsFinder:
+class TestsFinder(object):
 
     def __init__(self, testsdir, includes=(), excludes=()):
         self._testsdir = testsdir
@@ -167,7 +167,7 @@ def find_available_tests(self):
         mods = [mod for mod, _ in load_modules(self._testsdir)]
         for mod in mods:
             for name in set(dir(mod)):
-                if name.endswith('Tests'):
+                if name.endswith('Tests') or name.startswith('Test'):
                     self._test_factories.append(getattr(mod, name))
 
     def load_tests(self):

diff --git a/setup.py b/setup.py
@@ -5,6 +5,9 @@
 with open("README.rst") as fp:
     long_description = fp.read()
 
+with open('CHANGES.rst') as fp:
+    long_description += '\n\n' + fp.read()
+
 extensions = []
 if os.name == 'nt':
     ext = Extension(
@@ -18,7 +21,7 @@
 
 setup(
     name="trollius",
-    version="2.2.post1",
+    version="2.2.1",
     license="Apache License 2.0",
     author='Victor Stinner',
     author_email='victor.stinner@gmail.com',

diff --git a/tests/test_py33_exceptions.py b/tests/test_py33_exceptions.py
@@ -0,0 +1,35 @@
+# -*- coding: utf-8 -*-
+"""
+Tests for py33_exceptions.
+
+"""
+from __future__ import absolute_import
+from __future__ import division
+from __future__ import print_function
+
+
+import unittest
+from trollius import py33_exceptions
+
+class TestWrapErrors(unittest.TestCase):
+
+    def test_ebadf_wrapped_to_OSError(self):
+        # https://github.com/jamadden/trollius/issues/17
+        import socket
+        import os
+        import errno
+        s = socket.socket()
+        os.close(s.fileno())
+
+        with self.assertRaises(socket.error) as exc:
+            s.send(b'abc')
+
+        self.assertEqual(exc.exception.errno, errno.EBADF)
+
+        with self.assertRaises(OSError) as exc:
+            py33_exceptions.wrap_error(s.send, b'abc')
+
+        self.assertEqual(exc.exception.errno, errno.EBADF)
+
+if __name__ == '__main__':
+    unittest.main()
diff --git a/trollius/py33_exceptions.py b/trollius/py33_exceptions.py
@@ -79,6 +79,9 @@ class ProcessLookupError(OSError):
     errno.ESRCH: ProcessLookupError,
 }
 
+if hasattr(errno, 'EBADF') and errno.EBADF not in _MAP_ERRNO:
+    _MAP_ERRNO[errno.EBADF] = OSError
+
 if sys.platform == 'win32':
     from trollius import _overlapped
     _MAP_ERRNO.update({
Original file line number	Diff line number	Diff line change
Expand Up		@@ -14,3 +14,5 @@ include examples/*.py

		include tests/.crt tests/.pem tests/*.key
		include tests/*.py

		include *.rst