diff options
| -rw-r--r-- | NEWS | 2 | ||||
| -rw-r--r-- | lib/fixtures/_fixtures/popen.py | 3 | ||||
| -rw-r--r-- | lib/fixtures/tests/_fixtures/test_popen.py | 4 |
3 files changed, 9 insertions, 0 deletions
@@ -12,6 +12,8 @@ CHANGES * ``FakePopen`` now supports being called under a context manager (IE: with). (Steve Kowalik) +* ``FakeProcess`` now supports kill(). (Steve Kowalik) + * ``MonkeyPatch`` now preserves ``staticmethod`` functions. (Dan Kenigsberg) diff --git a/lib/fixtures/_fixtures/popen.py b/lib/fixtures/_fixtures/popen.py index 80629fd..67f20e3 100644 --- a/lib/fixtures/_fixtures/popen.py +++ b/lib/fixtures/_fixtures/popen.py @@ -54,6 +54,9 @@ class FakeProcess(object): def __exit__(self, exc_type, exc_value, traceback): self.wait() + def kill(self): + pass + def wait(self): if self.returncode is None: self.communicate() diff --git a/lib/fixtures/tests/_fixtures/test_popen.py b/lib/fixtures/tests/_fixtures/test_popen.py index 8eb0174..6d47291 100644 --- a/lib/fixtures/tests/_fixtures/test_popen.py +++ b/lib/fixtures/tests/_fixtures/test_popen.py @@ -94,3 +94,7 @@ class TestFakeProcess(testtools.TestCase): proc = FakeProcess({}, {'stdout': BytesIO(_b('foo'))}) self.assertEqual((_b('foo'), ''), proc.communicate()) self.assertEqual(0, proc.returncode) + + def test_kill(self): + proc = FakeProcess({}, {}) + self.assertIs(None, proc.kill()) |