[Python-checkins] closes bpo-37252: Fix devpoll tests. (GH-14017)

[Benjamin Peterson]

https://github.com/python/cpython/commit/95da826db9ed4acbc81f32296f14429c06bd2124
commit: 95da826db9ed4acbc81f32296f14429c06bd2124
branch: master
author: Jakub Kulík <Kulikjak at gmail.com>
committer: Benjamin Peterson <benjamin at python.org>
date: 2019-09-11T14:42:04+01:00
summary:

closes bpo-37252: Fix devpoll tests. (GH-14017)

files:
A Misc/NEWS.d/next/Tests/2019-06-12-14-30-29.bpo-37252.4o-uLs.rst
M Lib/test/test_devpoll.py

diff --git a/Lib/test/test_devpoll.py b/Lib/test/test_devpoll.py
index c133c811b197..110c00686273 100644
--- a/Lib/test/test_devpoll.py
+++ b/Lib/test/test_devpoll.py
@@ -109,7 +109,7 @@ def test_close(self):
         # operations must fail with ValueError("I/O operation on closed ...")
         self.assertRaises(ValueError, devpoll.modify, fd, select.POLLIN)
         self.assertRaises(ValueError, devpoll.poll)
-        self.assertRaises(ValueError, devpoll.register, fd, fd, select.POLLIN)
+        self.assertRaises(ValueError, devpoll.register, fd, select.POLLIN)
         self.assertRaises(ValueError, devpoll.unregister, fd)
 
     def test_fd_non_inheritable(self):
@@ -122,9 +122,9 @@ def test_events_mask_overflow(self):
         w, r = os.pipe()
         pollster.register(w)
         # Issue #17919
-        self.assertRaises(OverflowError, pollster.register, 0, -1)
+        self.assertRaises(ValueError, pollster.register, 0, -1)
         self.assertRaises(OverflowError, pollster.register, 0, 1 << 64)
-        self.assertRaises(OverflowError, pollster.modify, 1, -1)
+        self.assertRaises(ValueError, pollster.modify, 1, -1)
         self.assertRaises(OverflowError, pollster.modify, 1, 1 << 64)
 
     @cpython_only
diff --git a/Misc/NEWS.d/next/Tests/2019-06-12-14-30-29.bpo-37252.4o-uLs.rst b/Misc/NEWS.d/next/Tests/2019-06-12-14-30-29.bpo-37252.4o-uLs.rst
new file mode 100644
index 000000000000..1bd7d2872f13
--- /dev/null
+++ b/Misc/NEWS.d/next/Tests/2019-06-12-14-30-29.bpo-37252.4o-uLs.rst
@@ -0,0 +1,2 @@
+Fix assertions in ``test_close`` and ``test_events_mask_overflow`` devpoll
+tests.