diff options
| author | David Hurst | 2017-10-14 21:43:49 -0600 |
|---|---|---|
| committer | GitHub | 2017-10-14 21:43:49 -0600 |
| commit | c170ffb61daa34399312762067c379d107918a96 (patch) | |
| tree | ea8f40409a29b2ae639b85421eaa77b700a7bd47 | |
| parent | caaac2a469cd8d97bd01541b2d172614efe8cbfb (diff) | |
| parent | 9ddd1de0e8ee13bff98097890b43d33e49b5f38e (diff) | |
| download | eventmq-c170ffb61daa34399312762067c379d107918a96.tar.gz eventmq-c170ffb61daa34399312762067c379d107918a96.zip | |
Merge pull request #56 from sideshowdave7/python3-test-hang
mock manager call
| -rw-r--r-- | eventmq/tests/test_jobmanager.py | 3 | ||||
| -rw-r--r-- | eventmq/tests/test_worker.py | 12 |
2 files changed, 2 insertions, 13 deletions
diff --git a/eventmq/tests/test_jobmanager.py b/eventmq/tests/test_jobmanager.py index 566c583..777e69c 100644 --- a/eventmq/tests/test_jobmanager.py +++ b/eventmq/tests/test_jobmanager.py | |||
| @@ -44,7 +44,7 @@ class TestCase(unittest.TestCase): | |||
| 44 | 44 | ||
| 45 | sndmsg_mock.assert_called_with(jm.outgoing, 'READY') | 45 | sndmsg_mock.assert_called_with(jm.outgoing, 'READY') |
| 46 | 46 | ||
| 47 | @mock.patch('multiprocessing.pool.Pool.close') | 47 | @mock.patch('multiprocessing.Manager') |
| 48 | @mock.patch('eventmq.jobmanager.JobManager.process_message') | 48 | @mock.patch('eventmq.jobmanager.JobManager.process_message') |
| 49 | @mock.patch('eventmq.jobmanager.Sender.recv_multipart') | 49 | @mock.patch('eventmq.jobmanager.Sender.recv_multipart') |
| 50 | @mock.patch('eventmq.jobmanager.Poller.poll') | 50 | @mock.patch('eventmq.jobmanager.Poller.poll') |
| @@ -56,6 +56,7 @@ class TestCase(unittest.TestCase): | |||
| 56 | maybe_send_hb_mock.return_value = False | 56 | maybe_send_hb_mock.return_value = False |
| 57 | poll_mock.return_value = {jm.outgoing: jobmanager.POLLIN} | 57 | poll_mock.return_value = {jm.outgoing: jobmanager.POLLIN} |
| 58 | sender_mock.return_value = [1, 2, 3] | 58 | sender_mock.return_value = [1, 2, 3] |
| 59 | pool_close_mock.return_value = None | ||
| 59 | 60 | ||
| 60 | jm._start_event_loop() | 61 | jm._start_event_loop() |
| 61 | 62 | ||
diff --git a/eventmq/tests/test_worker.py b/eventmq/tests/test_worker.py index 66808fd..5e72f39 100644 --- a/eventmq/tests/test_worker.py +++ b/eventmq/tests/test_worker.py | |||
| @@ -14,24 +14,13 @@ | |||
| 14 | # along with eventmq. If not, see <http://www.gnu.org/licenses/>. | 14 | # along with eventmq. If not, see <http://www.gnu.org/licenses/>. |
| 15 | 15 | ||
| 16 | import logging | 16 | import logging |
| 17 | from multiprocessing import Pool | ||
| 18 | import time | 17 | import time |
| 19 | 18 | ||
| 20 | from nose import with_setup | ||
| 21 | |||
| 22 | from .. import worker | 19 | from .. import worker |
| 23 | 20 | ||
| 24 | ADDR = 'inproc://pour_the_rice_in_the_thing' | 21 | ADDR = 'inproc://pour_the_rice_in_the_thing' |
| 25 | 22 | ||
| 26 | 23 | ||
| 27 | def setup_func(): | ||
| 28 | global pool | ||
| 29 | global out | ||
| 30 | pool = Pool() | ||
| 31 | out = pool.map(job, range(1)) | ||
| 32 | |||
| 33 | |||
| 34 | @with_setup(setup_func) | ||
| 35 | def test_run_with_timeout(): | 24 | def test_run_with_timeout(): |
| 36 | payload = { | 25 | payload = { |
| 37 | 'path': 'eventmq.tests.test_worker', | 26 | 'path': 'eventmq.tests.test_worker', |
| @@ -44,7 +33,6 @@ def test_run_with_timeout(): | |||
| 44 | assert msgid | 33 | assert msgid |
| 45 | 34 | ||
| 46 | 35 | ||
| 47 | @with_setup(setup_func) | ||
| 48 | def test_run_setup(): | 36 | def test_run_setup(): |
| 49 | setup_callable = 'pre_hook' | 37 | setup_callable = 'pre_hook' |
| 50 | setup_path = 'eventmq.tests.test_worker' | 38 | setup_path = 'eventmq.tests.test_worker' |