Merge "Make magnum manage_template read config file and increase coverage"
This commit is contained in:
commit
0beb06cb67
|
@ -110,4 +110,7 @@ class TemplateManager(app.App):
|
|||
def main(args=None):
|
||||
if args is None:
|
||||
args = sys.argv[1:]
|
||||
CONF([],
|
||||
project='magnum',
|
||||
version=version.version_info.release_string())
|
||||
return TemplateManager().run(args)
|
||||
|
|
|
@ -0,0 +1,99 @@
|
|||
# Copyright 2016 - Fujitsu, Ltd.
|
||||
#
|
||||
# Licensed under the Apache License, Version 2.0 (the "License");
|
||||
# you may not use this file except in compliance with the License.
|
||||
# You may obtain a copy of the License at
|
||||
#
|
||||
# http://www.apache.org/licenses/LICENSE-2.0
|
||||
#
|
||||
# Unless required by applicable law or agreed to in writing, software
|
||||
# distributed under the License is distributed on an "AS IS" BASIS,
|
||||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
# See the License for the specific language governing permissions and
|
||||
# limitations under the License.
|
||||
|
||||
import mock
|
||||
|
||||
from magnum.cmd import template_manage
|
||||
from magnum.tests import base
|
||||
|
||||
|
||||
class TestMagnumTemplateManage(base.TestCase):
|
||||
|
||||
# Fake entrypoints method
|
||||
@staticmethod
|
||||
def _fake_entry(num_of_entries):
|
||||
while num_of_entries:
|
||||
fake_entry = mock.MagicMock()
|
||||
fake_entry.name = 'magnum_' + 'test_' + \
|
||||
'foo_' + 'bar'*num_of_entries
|
||||
fake_cls = mock.MagicMock()
|
||||
fake_definition = fake_cls()
|
||||
fake_definition.provides = [{'coe': 'foo', 'os': 'bar',
|
||||
'server_type': 'test'}]
|
||||
fake_definition.template_path = 'fake_path'
|
||||
yield fake_entry, fake_cls
|
||||
num_of_entries -= 1
|
||||
|
||||
@mock.patch.object(template_manage.TemplateManager, 'run')
|
||||
@mock.patch('sys.argv', ['foo', 'bar'])
|
||||
def test_none_arg(self, mock_run):
|
||||
args = None
|
||||
template_manage.main(args)
|
||||
mock_run.assert_called_once_with(['bar'])
|
||||
|
||||
# NOTE(hieulq): we fake the entrypoints then we need to mock the cliff
|
||||
# produce_output in order to assert with fake value
|
||||
@mock.patch('magnum.cmd.template_manage.TemplateList.produce_output')
|
||||
@mock.patch('magnum.drivers.common.template_def.TemplateDefinition')
|
||||
def test_correct_arg_with_details_and_path(self, mock_tdef, mock_produce):
|
||||
args = ['list-templates', '-d', '-p']
|
||||
mock_tdef.load_entry_points.return_value = self._fake_entry(1)
|
||||
template_manage.main(args)
|
||||
mock_tdef.load_entry_points.assert_called_once_with()
|
||||
mock_produce.assert_called_once_with(mock.ANY, mock.ANY,
|
||||
[('magnum_test_foo_bar',
|
||||
False, 'test',
|
||||
'bar', 'foo', 'fake_path')])
|
||||
|
||||
# NOTE(hieulq): we fake the entrypoints then we need to mock the cliff
|
||||
# produce_output in order to assert with fake value
|
||||
@mock.patch('magnum.cmd.template_manage.TemplateList.produce_output')
|
||||
@mock.patch('magnum.drivers.common.template_def.TemplateDefinition')
|
||||
def test_correct_arg_without_details_and_path(self, mock_tdef,
|
||||
mock_produce):
|
||||
args = ['list-templates']
|
||||
mock_tdef.load_entry_points.return_value = self._fake_entry(1)
|
||||
template_manage.main(args)
|
||||
mock_tdef.load_entry_points.assert_called_once_with()
|
||||
mock_produce.assert_called_once_with(mock.ANY, mock.ANY,
|
||||
[('magnum_test_foo_bar', False)])
|
||||
|
||||
# NOTE(hieulq): we fake the entrypoints then we need to mock the cliff
|
||||
# produce_output in order to assert with fake value
|
||||
@mock.patch('magnum.cmd.template_manage.TemplateList.produce_output')
|
||||
@mock.patch('magnum.drivers.common.template_def.TemplateDefinition')
|
||||
def test_correct_arg_with_enabled_template(self, mock_tdef, mock_produce):
|
||||
args = ['list-templates', '--enabled']
|
||||
self.config(enabled_definitions={'magnum_test_foo_bar'},
|
||||
group='cluster')
|
||||
mock_tdef.load_entry_points.return_value = self._fake_entry(2)
|
||||
template_manage.main(args)
|
||||
mock_tdef.load_entry_points.assert_called_once_with()
|
||||
mock_produce.assert_called_once_with(mock.ANY, mock.ANY,
|
||||
[('magnum_test_foo_bar', True)])
|
||||
|
||||
# NOTE(hieulq): we fake the entrypoints then we need to mock the cliff
|
||||
# produce_output in order to assert with fake value
|
||||
@mock.patch('magnum.cmd.template_manage.TemplateList.produce_output')
|
||||
@mock.patch('magnum.drivers.common.template_def.TemplateDefinition')
|
||||
def test_correct_arg_with_disabled_template(self, mock_tdef, mock_produce):
|
||||
args = ['list-templates', '--disable']
|
||||
self.config(enabled_definitions={'magnum_test_foo_bar'},
|
||||
group='cluster')
|
||||
mock_tdef.load_entry_points.return_value = self._fake_entry(2)
|
||||
template_manage.main(args)
|
||||
mock_tdef.load_entry_points.assert_called_once_with()
|
||||
mock_produce.assert_called_once_with(mock.ANY, mock.ANY,
|
||||
[('magnum_test_foo_barbar',
|
||||
False)])
|
Loading…
Reference in New Issue