Remove integrated dashboard tests
Horizon tempest tests are now in a plugin openstack/tempest-horizon This patch removes the scenario test for the dashboard and the corresponding configuration values. Change-Id: I2a69ebed2947a5ab5e5ca79557130bd093e168dd
This commit is contained in:
David Lyle
parent
484748e8b5
commit
672ebd32ba
@ -0,0 +1,7 @@
|
||||
---
|
||||
upgrade:
|
||||
- The integrated dashboard scenario test has been
|
||||
removed and is now in a separate tempest plugin
|
||||
tempest-horizon. The removed test coverage can be
|
||||
used by installing tempest-horizon on the server
|
||||
where you run tempest.
|
||||
@ -841,21 +841,6 @@ OrchestrationGroup = [
|
||||
help="Value must match heat configuration of the same name."),
|
||||
]
|
||||
|
||||
|
||||
dashboard_group = cfg.OptGroup(name="dashboard",
|
||||
title="Dashboard options")
|
||||
|
||||
DashboardGroup = [
|
||||
cfg.StrOpt('dashboard_url',
|
||||
default='http://localhost/',
|
||||
help="Where the dashboard can be found"),
|
||||
cfg.StrOpt('login_url',
|
||||
default='http://localhost/auth/login/',
|
||||
help="Login page for the dashboard",
|
||||
deprecated_for_removal=True),
|
||||
]
|
||||
|
||||
|
||||
data_processing_group = cfg.OptGroup(name="data-processing",
|
||||
title="Data Processing options")
|
||||
|
||||
@ -986,9 +971,6 @@ ServiceAvailableGroup = [
|
||||
cfg.BoolOpt('heat',
|
||||
default=False,
|
||||
help="Whether or not Heat is expected to be available"),
|
||||
cfg.BoolOpt('horizon',
|
||||
default=True,
|
||||
help="Whether or not Horizon is expected to be available"),
|
||||
cfg.BoolOpt('sahara',
|
||||
default=False,
|
||||
help="Whether or not Sahara is expected to be available"),
|
||||
@ -1131,7 +1113,6 @@ _opts = [
|
||||
(object_storage_feature_group, ObjectStoreFeaturesGroup),
|
||||
(database_group, DatabaseGroup),
|
||||
(orchestration_group, OrchestrationGroup),
|
||||
(dashboard_group, DashboardGroup),
|
||||
(data_processing_group, DataProcessingGroup),
|
||||
(data_processing_feature_group, DataProcessingFeaturesGroup),
|
||||
(stress_group, StressGroup),
|
||||
@ -1199,7 +1180,6 @@ class TempestConfigPrivate(object):
|
||||
'object-storage-feature-enabled']
|
||||
self.database = _CONF.database
|
||||
self.orchestration = _CONF.orchestration
|
||||
self.dashboard = _CONF.dashboard
|
||||
self.data_processing = _CONF['data-processing']
|
||||
self.data_processing_feature_enabled = _CONF[
|
||||
'data-processing-feature-enabled']
|
||||
|
||||
@ -1,120 +0,0 @@
|
||||
# All Rights Reserved.
|
||||
#
|
||||
# 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.
|
||||
|
||||
from six.moves import html_parser as HTMLParser
|
||||
from six.moves.urllib import parse
|
||||
from six.moves.urllib import request
|
||||
|
||||
from tempest import config
|
||||
from tempest.scenario import manager
|
||||
from tempest import test
|
||||
|
||||
CONF = config.CONF
|
||||
|
||||
|
||||
class HorizonHTMLParser(HTMLParser.HTMLParser):
|
||||
csrf_token = None
|
||||
region = None
|
||||
login = None
|
||||
|
||||
def _find_name(self, attrs, name):
|
||||
for attrpair in attrs:
|
||||
if attrpair[0] == 'name' and attrpair[1] == name:
|
||||
return True
|
||||
return False
|
||||
|
||||
def _find_value(self, attrs):
|
||||
for attrpair in attrs:
|
||||
if attrpair[0] == 'value':
|
||||
return attrpair[1]
|
||||
return None
|
||||
|
||||
def _find_attr_value(self, attrs, attr_name):
|
||||
for attrpair in attrs:
|
||||
if attrpair[0] == attr_name:
|
||||
return attrpair[1]
|
||||
return None
|
||||
|
||||
def handle_starttag(self, tag, attrs):
|
||||
if tag == 'input':
|
||||
if self._find_name(attrs, 'csrfmiddlewaretoken'):
|
||||
self.csrf_token = self._find_value(attrs)
|
||||
if self._find_name(attrs, 'region'):
|
||||
self.region = self._find_value(attrs)
|
||||
if tag == 'form':
|
||||
self.login = self._find_attr_value(attrs, 'action')
|
||||
|
||||
|
||||
class TestDashboardBasicOps(manager.ScenarioTest):
|
||||
|
||||
"""The test suite for dashboard basic operations
|
||||
|
||||
This is a basic scenario test:
|
||||
* checks that the login page is available
|
||||
* logs in as a regular user
|
||||
* checks that the user home page loads without error
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def skip_checks(cls):
|
||||
super(TestDashboardBasicOps, cls).skip_checks()
|
||||
if not CONF.service_available.horizon:
|
||||
raise cls.skipException("Horizon support is required")
|
||||
|
||||
@classmethod
|
||||
def setup_credentials(cls):
|
||||
cls.set_network_resources()
|
||||
super(TestDashboardBasicOps, cls).setup_credentials()
|
||||
|
||||
def check_login_page(self):
|
||||
response = request.urlopen(CONF.dashboard.dashboard_url)
|
||||
self.assertIn("id_username", response.read())
|
||||
|
||||
def user_login(self, username, password):
|
||||
self.opener = request.build_opener(request.HTTPCookieProcessor())
|
||||
response = self.opener.open(CONF.dashboard.dashboard_url).read()
|
||||
|
||||
# Grab the CSRF token and default region
|
||||
parser = HorizonHTMLParser()
|
||||
parser.feed(response)
|
||||
|
||||
# construct login url for dashboard, discovery accommodates non-/ web
|
||||
# root for dashboard
|
||||
login_url = parse.urljoin(CONF.dashboard.dashboard_url, parser.login)
|
||||
|
||||
# Prepare login form request
|
||||
req = request.Request(login_url)
|
||||
req.add_header('Content-type', 'application/x-www-form-urlencoded')
|
||||
req.add_header('Referer', CONF.dashboard.dashboard_url)
|
||||
|
||||
# Pass the default domain name regardless of the auth version in order
|
||||
# to test the scenario of when horizon is running with keystone v3
|
||||
params = {'username': username,
|
||||
'password': password,
|
||||
'region': parser.region,
|
||||
'domain': CONF.auth.default_credentials_domain_name,
|
||||
'csrfmiddlewaretoken': parser.csrf_token}
|
||||
self.opener.open(req, parse.urlencode(params))
|
||||
|
||||
def check_home_page(self):
|
||||
response = self.opener.open(CONF.dashboard.dashboard_url)
|
||||
self.assertIn('Overview', response.read())
|
||||
|
||||
@test.idempotent_id('4f8851b1-0e69-482b-b63b-84c6e76f6c80')
|
||||
@test.services('dashboard')
|
||||
def test_basic_scenario(self):
|
||||
creds = self.os.credentials
|
||||
self.check_login_page()
|
||||
self.user_login(creds.username, creds.password)
|
||||
self.check_home_page()
|
||||
@ -77,7 +77,6 @@ def get_service_list():
|
||||
'network': True,
|
||||
'identity': True,
|
||||
'object_storage': CONF.service_available.swift,
|
||||
'dashboard': CONF.service_available.horizon,
|
||||
'data_processing': CONF.service_available.sahara,
|
||||
'database': CONF.service_available.trove
|
||||
}
|
||||
@ -92,8 +91,8 @@ def services(*args):
|
||||
"""
|
||||
def decorator(f):
|
||||
services = ['compute', 'image', 'baremetal', 'volume', 'orchestration',
|
||||
'network', 'identity', 'object_storage', 'dashboard',
|
||||
'data_processing', 'database']
|
||||
'network', 'identity', 'object_storage', 'data_processing',
|
||||
'database']
|
||||
for service in args:
|
||||
if service not in services:
|
||||
raise exceptions.InvalidServiceTag('%s is not a valid '
|
||||
|
||||
Loading…
Reference in New Issue
Block a user