diff options
author | Michael Hudson-Doyle <michael.hudson@canonical.com> | 2017-06-07 13:52:17 -0700 |
---|---|---|
committer | Michael Hudson-Doyle <michael.hudson@canonical.com> | 2017-06-07 13:52:17 -0700 |
commit | 4b0ed8b6030450e6d99909a7c683e9642e546387 (patch) | |
tree | 6c1b0acc8a28d2015c8561e6383b11d860af125f /urwid | |
parent | fee50e77e85fb0fc5f7d664ae13c0e139d711a00 (diff) | |
download | urwid-4b0ed8b6030450e6d99909a7c683e9642e546387.tar.gz |
fix test_remove_watch_file flakiness
pass a known-good file descriptor to watch_file rather than hard-coding 5
Fixes #164
Diffstat (limited to 'urwid')
-rw-r--r-- | urwid/tests/test_event_loops.py | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/urwid/tests/test_event_loops.py b/urwid/tests/test_event_loops.py index c85bbed..b01212d 100644 --- a/urwid/tests/test_event_loops.py +++ b/urwid/tests/test_event_loops.py @@ -30,9 +30,14 @@ class EventLoopTestMixin(object): def test_remove_watch_file(self): evl = self.evl - handle = evl.watch_file(5, lambda: None) - self.assertTrue(evl.remove_watch_file(handle)) - self.assertFalse(evl.remove_watch_file(handle)) + fd_r, fd_w = os.pipe() + try: + handle = evl.watch_file(fd_r, lambda: None) + self.assertTrue(evl.remove_watch_file(handle)) + self.assertFalse(evl.remove_watch_file(handle)) + finally: + os.close(fd_r) + os.close(fd_w) _expected_idle_handle = 1 |