diff options
author | wevsty <11432483+wevsty@users.noreply.github.com> | 2020-07-13 20:31:14 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-07-13 19:31:14 +0700 |
commit | 4e1eb97056001d1da4f23dc3cfe63e671f15590b (patch) | |
tree | a463305d85d23f24fc40b7c396965818b50b019c /tests/test_worker.py | |
parent | 5d199df85de0706d6cc04ac9e64d7d78b2b76333 (diff) | |
download | rq-4e1eb97056001d1da4f23dc3cfe63e671f15590b.tar.gz |
Split kill_house() fix issues #1234 (#1300)
* Split kill_house() fix issues #1234
Details View issues #1234
* Removing the catch finally
* rename wait_horse() to wait_for_horse()
* rename wait_horse() to wait_for_horse()
* update test_handle_shutdown_request()
Change test_handle_shutdown_request() exitcode assert
* Restore kill_horse() output
* optimization wait_for_horse()
Diffstat (limited to 'tests/test_worker.py')
-rw-r--r-- | tests/test_worker.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/tests/test_worker.py b/tests/test_worker.py index 8e8564f..df902ac 100644 --- a/tests/test_worker.py +++ b/tests/test_worker.py @@ -1212,9 +1212,8 @@ class HerokuWorkerShutdownTestCase(TimeoutTestCase, RQTestCase): w._horse_pid = p.pid w.handle_warm_shutdown_request() p.join(2) - # would expect p.exitcode to be -34 but for some reason os.waitpid is setting it to None, even though - # the process has ended - self.assertEqual(p.exitcode, None) + # would expect p.exitcode to be -34 + self.assertEqual(p.exitcode, -34) self.assertFalse(os.path.exists(path)) mock_logger_info.assert_called_with('Killed horse pid %s', p.pid) |