x/mogan
Code Issues Proposed changes

Remove unneeded get_node_list mock

Browse Source 
Change-Id: I73f20bf5b5cbc4b4e14fff80b924f84e749d458e
This commit is contained in:
Zhenguo Niu 2017-03-08 11:50:54 +08:00
parent fbacbd51ec
commit 370d45f388
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
mogan/tests/unit/engine/test_base_manager.py
View File

@ -16,7 +16,6 @@ import eventlet
import mock
from oslo_config import cfg
from mogan.engine.baremetal import ironic
from mogan.tests.unit.db import base as tests_db_base
from mogan.tests.unit.engine import mgr_utils
@ -24,15 +23,14 @@ from mogan.tests.unit.engine import mgr_utils
CONF = cfg.CONF
@mock.patch.object(ironic, 'get_node_list')
class StartStopTestCase(mgr_utils.ServiceSetUpMixin, tests_db_base.DbTestCase):
def test_prevent_double_start(self, mock_node_list):
def test_prevent_double_start(self):
self._start_service()
self.assertRaisesRegex(RuntimeError, 'already running',
self.service.init_host)
@mock.patch.object(eventlet.greenpool.GreenPool, 'waitall')
def test_del_host_waits_on_workerpool(self, wait_mock, mock_node_list):
def test_del_host_waits_on_workerpool(self, wait_mock):
self._start_service()
self.service.del_host()
self.assertTrue(wait_mock.called)