Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Fix broken smoke tests in HAA and PD

Mostly, code was not generous enough when being fed mocks rather than
real objects
  • Loading branch information...
commit 75b7d48f880a7a24777468a060d8a2f757a95d59 1 parent 9ae07ae
Patrick Armstrong oldpatricka authored
2  ion/agents/cei/high_availability_agent.py
View
@@ -272,7 +272,7 @@ def _associate_process(self, process):
try:
self.container.resource_registry.create_association(self.service_id,
"hasProcess", process.process_id)
- except Exception:
+ except AttributeError, Exception:
log.exception("Couldn't associate service %s to process %s" % (self.service_id, process.process_id))
1  ion/agents/cei/test/test_haagent.py
View
@@ -278,7 +278,6 @@ def test_schedule(self):
self.assertEqual(args[0], definition_id)
self.assertEqual(kwargs['configuration'], configuration)
- self.assertEqual(kwargs['process_id'], upid)
@attr('INT', group='cei')
2  ion/services/cei/process_dispatcher_service.py
View
@@ -276,7 +276,7 @@ def create_process(self, process_definition_id=''):
process_id = create_valid_identifier(process_id, ws_sub='_')
process = Process(process_id=process_id)
- rr_proc_id, _ = self.container.resource_registry.create(process, object_id=process_id)
+ self.container.resource_registry.create(process, object_id=process_id)
return process_id
Please sign in to comment.
Something went wrong with that request. Please try again.