[ARVADOS] updated: 1.1.4-140-g4b5517d
Git user
git at public.curoverse.com
Mon Apr 23 11:31:07 EDT 2018
Summary of changes:
.../nodemanager/tests/test_computenode_dispatch_slurm.py | 12 ++++++++++++
1 file changed, 12 insertions(+)
via 4b5517dfc0f3126edf85a2701e1ee218ab4f0ab0 (commit)
from 64c74f24fcdfbbef0e08b72c09d85c53390ca3b3 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
commit 4b5517dfc0f3126edf85a2701e1ee218ab4f0ab0
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date: Mon Apr 23 11:29:55 2018 -0400
Fix nodemanager tests. no issue #
Stub out subprocess.check_output to avoid calling the real "scontrol".
Fixes test_failed_arvados_calls_retried, test_subscribe and
test_creation_with_arvados_node in SLURMComputeNodeSetupActorTestCase.
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>
diff --git a/services/nodemanager/tests/test_computenode_dispatch_slurm.py b/services/nodemanager/tests/test_computenode_dispatch_slurm.py
index b61db5c..840d0a5 100644
--- a/services/nodemanager/tests/test_computenode_dispatch_slurm.py
+++ b/services/nodemanager/tests/test_computenode_dispatch_slurm.py
@@ -141,3 +141,15 @@ class SLURMComputeNodeSetupActorTestCase(ComputeNodeSetupActorTestCase):
self.make_actor()
self.wait_for_assignment(self.setup_actor, 'cloud_node')
check_output.assert_called_with(['scontrol', 'update', 'NodeName=compute99', 'Weight=1000', 'Features=instancetype=z1.test'])
+
+ @mock.patch('subprocess.check_output')
+ def test_failed_arvados_calls_retried(self, check_output):
+ super(SLURMComputeNodeSetupActorTestCase, self).test_failed_arvados_calls_retried()
+
+ @mock.patch('subprocess.check_output')
+ def test_subscribe(self, check_output):
+ super(SLURMComputeNodeSetupActorTestCase, self).test_subscribe()
+
+ @mock.patch('subprocess.check_output')
+ def test_creation_with_arvados_node(self, check_output):
+ super(SLURMComputeNodeSetupActorTestCase, self).test_creation_with_arvados_node()
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list