Akihiro Motoki e5d09edc20 Use python3-style super()
In python3, super() does not always require a class and self reference.
In other words, super() is enough for most cases.
This is much simpler and it is time to switch it to the newer style.

pylint provides a check for this.
Let's enable 'super-with-arguments' check.

NOTE: _prepare_mappings() method of FormRegion in
openstack_dashboard/test/integration_tests/regions/forms.py is refactored.
super() (without explicit class and self referece) does not work when
a subclass method calls a same method in a parent class multiple times.
It looks better to prepare a separate method to provide a common logic.

Change-Id: Id9512a14be9f20dbd5ebd63d446570c7b7c825ff
2020-10-15 14:37:20 +09:00

82 lines
3.1 KiB
Python

# 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 os import listdir
from os.path import isfile
from os.path import join
from re import search
from openstack_dashboard.test.integration_tests.pages import basepage
from openstack_dashboard.test.integration_tests.regions import tables
class ApiAccessTable(tables.TableRegion):
name = "endpoints"
dropdown = 'Download OpenStack'
def download_rcfile_dropdown(self):
self.driver.find_element_by_partial_link_text(
self.dropdown).click()
@tables.bind_table_action('download_openrc_v2')
def download_openstack_rc_v2(self, download_button):
self.download_rcfile_dropdown()
download_button.click()
@tables.bind_table_action('download_openrc')
def download_openstack_rc_v3(self, download_button):
self.download_rcfile_dropdown()
download_button.click()
class ApiaccessPage(basepage.BaseNavigationPage):
def __init__(self, driver, conf):
super().__init__(driver, conf)
self._page_title = "API Access"
@property
def apiaccess_table(self):
return ApiAccessTable(self.driver, self.conf)
def download_openstack_rc_file(self, version, directory, template):
if version == 2:
self.apiaccess_table.download_openstack_rc_v2()
elif version == 3:
self.apiaccess_table.download_openstack_rc_v3()
def list_of_files(self, directory, template):
return [f for f in listdir(directory)
if (isfile(join(directory, f)) and
f.endswith(template))]
def get_credentials_from_file(self, version, directory, template):
self._wait_until(
lambda _: len(self.list_of_files(directory, template)) > 0)
file_name = self.list_of_files(directory, template)[0]
with open(join(directory, file_name)) as cred_file:
content = cred_file.read()
username_re = r'export OS_USERNAME="([^"]+)"'
if version == 2:
tenant_name_re = r'export OS_TENANT_NAME="([^"]+)"'
tenant_id_re = r'export OS_TENANT_ID=(.+)'
elif version == 3:
tenant_name_re = r'export OS_PROJECT_NAME="([^"]+)"'
tenant_id_re = r'export OS_PROJECT_ID=(.+)'
username = search(username_re, content).group(1)
tenant_name = search(tenant_name_re, content).group(1)
tenant_id = search(tenant_id_re, content).group(1)
cred_dict = {'OS_USERNAME': username,
'OS_TENANT_NAME': tenant_name,
'OS_TENANT_ID': tenant_id}
return cred_dict