c1b51aa5e0
Enables the ability for default config files to be overriden for each entry point type (simple, uwsgi, and nginx). Additionally, align unit tests with keystone's snap-openstack.yaml, as features tend to land in the keystone snap first. Change-Id: I9fe54f1dae2b94d2f78783f38ced48ebe7c66f0f
246 lines
10 KiB
Python
246 lines
10 KiB
Python
# -*- coding: utf-8 -*-
|
|
|
|
# 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.
|
|
|
|
"""
|
|
test_snap_openstack
|
|
----------------------------------
|
|
|
|
Tests for `snap_openstack` module.
|
|
"""
|
|
|
|
import os
|
|
import sys
|
|
|
|
from mock import mock_open
|
|
from mock import patch
|
|
|
|
from snap_openstack import base
|
|
from snap_openstack.tests import base as test_base
|
|
|
|
TEST_DIR = os.path.join(os.path.dirname(os.path.realpath(__file__)),
|
|
'data')
|
|
|
|
MOCK_SNAP_ENV = {
|
|
'snap_common': '/var/snap/keystone/common',
|
|
'snap': '/snap/keystone/current',
|
|
}
|
|
|
|
|
|
class TestOpenStackSnapExecute(test_base.TestCase):
|
|
|
|
@classmethod
|
|
def mock_exists(cls, path):
|
|
'''Test helper for os.path.exists'''
|
|
paths = {
|
|
'/snap/keystone/current/etc/keystone/keystone.conf': True,
|
|
'/var/snap/keystone/common/etc/keystone/conf.d': True,
|
|
'/var/snap/keystone/common/etc/nginx/nginx-snap.conf': True,
|
|
}
|
|
return paths.get(path, False)
|
|
|
|
def mock_exists_overrides(cls, path):
|
|
'''Test helper for os.path.exists'''
|
|
paths = {
|
|
'/snap/keystone/current/etc/keystone/keystone.conf': True,
|
|
'/var/snap/keystone/common/etc/keystone/keystone.conf': True,
|
|
'/var/snap/keystone/common/etc/keystone/conf.d': True,
|
|
'/var/snap/keystone/common/etc/nginx/nginx-snap.conf': True,
|
|
'/var/snap/keystone/common/etc/nginx/nginx.conf': True,
|
|
}
|
|
return paths.get(path, False)
|
|
|
|
def mock_snap_utils(self, mock_utils):
|
|
snap_utils = mock_utils.return_value
|
|
snap_utils.snap_env = MOCK_SNAP_ENV
|
|
snap_utils.drop_privileges.return_value = None
|
|
|
|
@patch.object(base, 'SnapFileRenderer')
|
|
@patch('snap_openstack.base.SnapUtils')
|
|
@patch.object(base, 'os')
|
|
def test_base_snap_config(self, mock_os, mock_utils,
|
|
mock_renderer):
|
|
'''Ensure wrapped binary called with full args list'''
|
|
self.mock_snap_utils(mock_utils)
|
|
snap = base.OpenStackSnap(os.path.join(TEST_DIR,
|
|
'snap-openstack.yaml'))
|
|
mock_os.path.exists.side_effect = self.mock_exists
|
|
snap.execute(['snap-openstack',
|
|
'keystone-manage'])
|
|
mock_os.execvp.assert_called_with(
|
|
'/snap/keystone/current/bin/keystone-manage',
|
|
['/snap/keystone/current/bin/keystone-manage',
|
|
'--config-file=/snap/keystone/current/etc/keystone/keystone.conf',
|
|
'--config-dir=/var/snap/keystone/common/etc/keystone/conf.d']
|
|
)
|
|
|
|
@patch.object(base, 'SnapFileRenderer')
|
|
@patch('snap_openstack.base.SnapUtils')
|
|
@patch.object(base, 'os')
|
|
def test_base_snap_config_override(self, mock_os, mock_utils,
|
|
mock_renderer):
|
|
'''Ensure wrapped binary called with full args list'''
|
|
self.mock_snap_utils(mock_utils)
|
|
snap = base.OpenStackSnap(os.path.join(TEST_DIR,
|
|
'snap-openstack.yaml'))
|
|
mock_os.path.exists.side_effect = self.mock_exists_overrides
|
|
snap.execute(['snap-openstack',
|
|
'keystone-manage'])
|
|
mock_os.execvp.assert_called_with(
|
|
'/snap/keystone/current/bin/keystone-manage',
|
|
['/snap/keystone/current/bin/keystone-manage',
|
|
'--config-file=/var/snap/keystone/common/etc/keystone/'
|
|
'keystone.conf',
|
|
'--config-dir=/var/snap/keystone/common/etc/keystone/conf.d']
|
|
)
|
|
|
|
@patch.object(base, 'SnapFileRenderer')
|
|
@patch('snap_openstack.base.SnapUtils')
|
|
@patch.object(base, 'os')
|
|
def test_base_snap_config_no_logging(self, mock_os, mock_utils,
|
|
mock_renderer):
|
|
'''Ensure wrapped binary called correctly with no logfile'''
|
|
self.mock_snap_utils(mock_utils)
|
|
snap = base.OpenStackSnap(os.path.join(TEST_DIR,
|
|
'snap-openstack.yaml'))
|
|
mock_os.path.exists.side_effect = self.mock_exists
|
|
snap.execute(['snap-openstack',
|
|
'keystone-manage',
|
|
'db', 'sync'])
|
|
mock_os.execvp.assert_called_with(
|
|
'/snap/keystone/current/bin/keystone-manage',
|
|
['/snap/keystone/current/bin/keystone-manage',
|
|
'--config-file=/snap/keystone/current/etc/keystone/keystone.conf',
|
|
'--config-dir=/var/snap/keystone/common/etc/keystone/conf.d',
|
|
'db', 'sync']
|
|
)
|
|
|
|
@patch.object(base, 'SnapFileRenderer')
|
|
@patch('snap_openstack.base.SnapUtils')
|
|
@patch.object(base, 'os')
|
|
def test_base_snap_config_missing_entry_point(self, mock_os, mock_utils,
|
|
mock_renderer):
|
|
'''Ensure ValueError raised for missing entry_point'''
|
|
self.mock_snap_utils(mock_utils)
|
|
snap = base.OpenStackSnap(os.path.join(TEST_DIR,
|
|
'snap-openstack.yaml'))
|
|
mock_os.path.exists.side_effect = self.mock_exists
|
|
self.assertRaises(ValueError,
|
|
snap.execute,
|
|
['snap-openstack',
|
|
'keystone-api'])
|
|
|
|
@patch.object(base, 'SnapFileRenderer')
|
|
@patch('snap_openstack.base.SnapUtils')
|
|
@patch.object(base, 'os')
|
|
def test_base_snap_config_uwsgi(self, mock_os, mock_utils,
|
|
mock_renderer):
|
|
'''Ensure wrapped binary of uwsgi called with correct arguments'''
|
|
self.mock_snap_utils(mock_utils)
|
|
snap = base.OpenStackSnap(os.path.join(TEST_DIR,
|
|
'snap-openstack.yaml'))
|
|
mock_os.path.exists.side_effect = self.mock_exists
|
|
builtin = '__builtin__'
|
|
if sys.version_info > (3, 0):
|
|
builtin = 'builtins'
|
|
with patch('{}.open'.format(builtin), mock_open(), create=True):
|
|
snap.execute(['snap-openstack',
|
|
'keystone-uwsgi'])
|
|
mock_os.execvp.assert_called_with(
|
|
'/snap/keystone/current/bin/uwsgi',
|
|
['/snap/keystone/current/bin/uwsgi', '--master',
|
|
'--die-on-term', '-H', '/snap/keystone/current/usr',
|
|
'--emperor', '/var/snap/keystone/common/etc/uwsgi/snap',
|
|
'--logto', '/var/snap/keystone/common/log/uwsgi.log']
|
|
)
|
|
|
|
@patch.object(base, 'SnapFileRenderer')
|
|
@patch('snap_openstack.base.SnapUtils')
|
|
@patch.object(base, 'os')
|
|
def test_base_snap_config_uwsgi_override(self, mock_os, mock_utils,
|
|
mock_renderer):
|
|
'''Ensure wrapped binary of uwsgi called with correct arguments'''
|
|
self.mock_snap_utils(mock_utils)
|
|
snap = base.OpenStackSnap(os.path.join(TEST_DIR,
|
|
'snap-openstack.yaml'))
|
|
mock_os.path.exists.side_effect = self.mock_exists_overrides
|
|
mock_os.listdir.side_effect = (
|
|
'/var/snap/keystone/common/etc/uwsgi/config.ini'
|
|
)
|
|
builtin = '__builtin__'
|
|
if sys.version_info > (3, 0):
|
|
builtin = 'builtins'
|
|
with patch('{}.open'.format(builtin), mock_open(), create=True):
|
|
snap.execute(['snap-openstack',
|
|
'keystone-uwsgi'])
|
|
mock_os.execvp.assert_called_with(
|
|
'/snap/keystone/current/bin/uwsgi',
|
|
['/snap/keystone/current/bin/uwsgi', '--master',
|
|
'--die-on-term', '-H', '/snap/keystone/current/usr',
|
|
'--emperor', '/var/snap/keystone/common/etc/uwsgi',
|
|
'--logto', '/var/snap/keystone/common/log/uwsgi.log']
|
|
)
|
|
|
|
@patch.object(base, 'SnapFileRenderer')
|
|
@patch('snap_openstack.base.SnapUtils')
|
|
@patch.object(base, 'os')
|
|
def test_base_snap_config_nginx(self, mock_os, mock_utils,
|
|
mock_renderer):
|
|
'''Ensure wrapped binary of nginx called with correct arguments'''
|
|
self.mock_snap_utils(mock_utils)
|
|
snap = base.OpenStackSnap(os.path.join(TEST_DIR,
|
|
'snap-openstack.yaml'))
|
|
mock_os.path.exists.side_effect = self.mock_exists
|
|
snap.execute(['snap-openstack',
|
|
'keystone-nginx'])
|
|
mock_os.execvp.assert_called_with(
|
|
'/snap/keystone/current/usr/sbin/nginx',
|
|
['/snap/keystone/current/usr/sbin/nginx', '-g',
|
|
'daemon on; master_process on;',
|
|
'-c', '/var/snap/keystone/common/etc/nginx/nginx-snap.conf']
|
|
)
|
|
|
|
@patch.object(base, 'SnapFileRenderer')
|
|
@patch('snap_openstack.base.SnapUtils')
|
|
@patch.object(base, 'os')
|
|
def test_base_snap_config_nginx_override(self, mock_os, mock_utils,
|
|
mock_renderer):
|
|
'''Ensure wrapped binary of nginx called with correct arguments'''
|
|
self.mock_snap_utils(mock_utils)
|
|
snap = base.OpenStackSnap(os.path.join(TEST_DIR,
|
|
'snap-openstack.yaml'))
|
|
mock_os.path.exists.side_effect = self.mock_exists_overrides
|
|
snap.execute(['snap-openstack',
|
|
'keystone-nginx'])
|
|
mock_os.execvp.assert_called_with(
|
|
'/snap/keystone/current/usr/sbin/nginx',
|
|
['/snap/keystone/current/usr/sbin/nginx', '-g',
|
|
'daemon on; master_process on;',
|
|
'-c', '/var/snap/keystone/common/etc/nginx/nginx.conf']
|
|
)
|
|
|
|
@patch.object(base, 'SnapFileRenderer')
|
|
@patch('snap_openstack.base.SnapUtils')
|
|
@patch.object(base, 'os')
|
|
def test_base_snap_config_invalid_ep_type(self, mock_os, mock_utils,
|
|
mock_renderer):
|
|
'''Ensure endpoint types are correctly validated'''
|
|
self.mock_snap_utils(mock_utils)
|
|
snap = base.OpenStackSnap(os.path.join(TEST_DIR,
|
|
'snap-openstack.yaml'))
|
|
mock_os.path.exists.side_effect = self.mock_exists
|
|
self.assertRaises(ValueError,
|
|
snap.execute,
|
|
['snap-openstack',
|
|
'keystone-broken'])
|