Update unit tests
This commit is contained in:
parent
5367fa957d
commit
cca3937327
|
@ -12,7 +12,7 @@ from test_utils import (
|
|||
|
||||
TO_PATCH = [
|
||||
'config_changed',
|
||||
'do_action_openstack_upgrade',
|
||||
'do_openstack_upgrade',
|
||||
'register_configs',
|
||||
'uuid'
|
||||
]
|
||||
|
@ -26,20 +26,30 @@ class TestCinderUpgradeActions(CharmTestCase):
|
|||
|
||||
@patch('charmhelpers.contrib.openstack.utils.config')
|
||||
@patch('charmhelpers.contrib.openstack.utils.action_set')
|
||||
def test_openstack_upgrade_true(self, action_set, config):
|
||||
self.do_action_openstack_upgrade.return_value = True
|
||||
@patch('charmhelpers.contrib.openstack.utils.git_install_requested')
|
||||
@patch('charmhelpers.contrib.openstack.utils.openstack_upgrade_available')
|
||||
def test_openstack_upgrade_true(self, upgrade_avail, git_requested,
|
||||
action_set, config):
|
||||
git_requested.return_value = False
|
||||
upgrade_avail.return_value = True
|
||||
config.return_value = True
|
||||
|
||||
openstack_upgrade.openstack_upgrade()
|
||||
|
||||
self.assertTrue(self.do_action_openstack_upgrade.called)
|
||||
self.assertTrue(self.do_openstack_upgrade.called)
|
||||
self.assertTrue(self.config_changed.called)
|
||||
|
||||
@patch('charmhelpers.contrib.openstack.utils.config')
|
||||
@patch('charmhelpers.contrib.openstack.utils.action_set')
|
||||
def test_openstack_upgrade_false(self, action_set, config):
|
||||
self.do_action_openstack_upgrade.return_value = False
|
||||
@patch('charmhelpers.contrib.openstack.utils.git_install_requested')
|
||||
@patch('charmhelpers.contrib.openstack.utils.openstack_upgrade_available')
|
||||
def test_openstack_upgrade_true(self, upgrade_avail, git_requested,
|
||||
action_set, config):
|
||||
git_requested.return_value = False
|
||||
upgrade_avail.return_value = True
|
||||
config.return_value = False
|
||||
|
||||
openstack_upgrade.openstack_upgrade()
|
||||
|
||||
self.assertTrue(self.do_action_openstack_upgrade.called)
|
||||
self.assertFalse(self.do_openstack_upgrade.called)
|
||||
self.assertFalse(self.config_changed.called)
|
||||
|
|
Loading…
Reference in New Issue