Commit 78fd521f authored by Florent Xicluna's avatar Florent Xicluna

Fix an oversight in r78508: p.wait() should be compared to 0

parent 98e3fc39
...@@ -643,7 +643,7 @@ class POSIXProcessTestCase(unittest.TestCase): ...@@ -643,7 +643,7 @@ class POSIXProcessTestCase(unittest.TestCase):
self.assertIs(p.poll(), None) self.assertIs(p.poll(), None)
p.send_signal(signal.SIGINT) p.send_signal(signal.SIGINT)
self.assertIsNot(p.wait(), None) self.assertNotEqual(p.wait(), 0)
@unittest.skip("See issue #2777") @unittest.skip("See issue #2777")
def test_kill(self): def test_kill(self):
...@@ -741,7 +741,7 @@ class Win32ProcessTestCase(unittest.TestCase): ...@@ -741,7 +741,7 @@ class Win32ProcessTestCase(unittest.TestCase):
self.assertIs(p.poll(), None) self.assertIs(p.poll(), None)
p.send_signal(signal.SIGTERM) p.send_signal(signal.SIGTERM)
self.assertIsNot(p.wait(), None) self.assertNotEqual(p.wait(), 0)
@unittest.skip("See issue #2777") @unittest.skip("See issue #2777")
def test_kill(self): def test_kill(self):
...@@ -749,7 +749,7 @@ class Win32ProcessTestCase(unittest.TestCase): ...@@ -749,7 +749,7 @@ class Win32ProcessTestCase(unittest.TestCase):
self.assertIs(p.poll(), None) self.assertIs(p.poll(), None)
p.kill() p.kill()
self.assertIsNot(p.wait(), None) self.assertNotEqual(p.wait(), 0)
@unittest.skip("See issue #2777") @unittest.skip("See issue #2777")
def test_terminate(self): def test_terminate(self):
...@@ -757,7 +757,7 @@ class Win32ProcessTestCase(unittest.TestCase): ...@@ -757,7 +757,7 @@ class Win32ProcessTestCase(unittest.TestCase):
self.assertIs(p.poll(), None) self.assertIs(p.poll(), None)
p.terminate() p.terminate()
self.assertIsNot(p.wait(), None) self.assertNotEqual(p.wait(), 0)
@unittest.skipUnless(getattr(subprocess, '_has_poll', False), @unittest.skipUnless(getattr(subprocess, '_has_poll', False),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment