X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/f4ca9ad94a6bb006d1f3c7ba207837f1736d1247..12bfd7a65c6635a882cb2e5e419321db100b9d56:/services/nodemanager/tests/test_failure.py diff --git a/services/nodemanager/tests/test_failure.py b/services/nodemanager/tests/test_failure.py index afebb9ca32..35605fcd8c 100644 --- a/services/nodemanager/tests/test_failure.py +++ b/services/nodemanager/tests/test_failure.py @@ -12,9 +12,9 @@ import pykka from . import testutil -import arvnodeman.fullstopactor +import arvnodeman.baseactor -class BogusActor(arvnodeman.fullstopactor.FullStopActor): +class BogusActor(arvnodeman.baseactor.BaseNodeManagerActor): def __init__(self, e): super(BogusActor, self).__init__() self.exp = e @@ -23,26 +23,17 @@ class BogusActor(arvnodeman.fullstopactor.FullStopActor): raise self.exp class ActorUnhandledExceptionTest(unittest.TestCase): - def test1(self): + def test_fatal_error(self): for e in (MemoryError(), threading.ThreadError(), OSError(errno.ENOMEM, "")): with mock.patch('os.killpg') as killpg_mock: - act = BogusActor.start(e) - act.tell({ - 'command': 'pykka_call', - 'attr_path': ("doStuff",), - 'args': [], - 'kwargs': {} - }) - act.stop(block=True) + act = BogusActor.start(e).tell_proxy() + act.doStuff() + act.actor_ref.stop(block=True) self.assertTrue(killpg_mock.called) + def test_nonfatal_error(self): with mock.patch('os.killpg') as killpg_mock: - act = BogusActor.start(OSError(errno.ENOENT, "")) - act.tell({ - 'command': 'pykka_call', - 'attr_path': ("doStuff",), - 'args': [], - 'kwargs': {} - }) - act.stop(block=True) + act = BogusActor.start(OSError(errno.ENOENT, "")).tell_proxy() + act.doStuff() + act.actor_ref.stop(block=True) self.assertFalse(killpg_mock.called)