iframe-proxy | Sunbelt Computer Software

236 lines (197 loc) · 7.78 KB
# 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 hacking import core
Guidelines for writing new hacking checks
 - Use only for python-openstackclient specific tests. OpenStack general tests
   should be submitted to the common 'hacking' module.
 - Pick numbers in the range O4xx. Find the current test with the highest
   allocated number and then pick the next value.
@core.flake8ext
def assert_no_oslo(logical_line):
    """Check for use of oslo libraries.
    Okay: import os
    Okay: from os import path
    O400: import oslo_messaging
    O400: from oslo_log import log
    if match := re.match(r'(from|import) (oslo_.*)', logical_line):
        if match.group(2) == 'oslo_i18n':
            return
        yield (0, "O400: oslo libraries should not be used in SDK projects")
@core.flake8ext
def assert_no_duplicated_setup(logical_line, filename):
    """Check for use of various unnecessary test duplications.
    This check only applies to files under openstackclient/tests/unit/.
    Okay: self.app = fakes.FakeShell()
    Okay: self.app.client_manager.auth_ref = mock.Mock()
    O401: self.app = Namespace(self.app, self.namespace)
    O401: self.network_client = self.app.client_manager.network
    O401: self.app.client_manager.network = mock.Mock()
    if os.path.join('openstackclient', 'tests', 'unit') not in filename:
        return
    if re.match(r'self.app = .*\(self.app, self.namespace\)', logical_line):
        yield (
            'O401: It is not necessary to create dummy Namespace objects',
    if os.path.basename(filename) != 'fakes.py':
        if re.match(
            r'self.[a-zA-Z0-9_]+ = self.app.client_manager.*', logical_line
            yield (
                "O401: Aliases for mocks of the service client are already "
                "provided by the respective service's FakeClientMixin class",
        if match := re.match(
            r'self.app.client_manager.([a-z_]+) = mock.Mock', logical_line
            service = match.group(1)
            if service == 'auth_ref':
                return
            yield (
                f"O401: client_manager.{service} mocks are already provided "
                f"by the {service} service's FakeClientMixin class",
@core.flake8ext
def assert_use_of_client_aliases(logical_line, filename):
    """Ensure we use $service_client instead of $sdk_connection.service.
    Okay: self.compute_client.find_server(foo)
    O402: self.app.client_manager.sdk_connnection.compute.find_server(foo)
    The following checks only apply to files under openstackclient/tests/unit/:
    O402: self.app.client_manager.compute.find_server.return_value = server
    O402: self.app.client_manager.compute.find_server = mock.Mock()
    O402: self.compute_client.find_server = mock.Mock()
    # we should expand the list of services as we drop legacy clients
    if match := re.match(
        r'self\.app\.client_manager\.sdk_connnection\.(compute|network|image)',
        logical_line,
        service = match.group(1)
        yield (0, f"O402: prefer {service}_client to sdk_connection.{service}")
    # everything from here down only affects unit tests
    if os.path.join('openstackclient', 'tests', 'unit') not in filename:
        return
    if match := re.match(
        r'(self\.app\.client_manager\.(compute|network|image)+\.[a-z_]+)\.(return_value|side_effect) = ',  # noqa: E501
        logical_line,
        service = match.group(1)
        yield (
            f"O402: prefer {service}_client to "
            f"self.app.client_manager.{service}",
    if match := re.match(
        r'(self\.app\.client_manager\.(compute|network|image)+\.[a-z_]+) = mock.Mock',  # noqa: E501
        logical_line,
        yield (
            f"O402: {match.group(1)} is already a mock: there's no need to "
            f"assign a new mock.Mock instance.",
    if match := re.match(
        r'(self\.(compute|network|image)_client\.[a-z_]+) = mock.Mock',
        logical_line,
        yield (
            f"O402: {match.group(1)} is already a mock: there's no need to "
            f"assign a new mock.Mock instance.",
class SDKProxyFindChecker(ast.NodeVisitor):
    """NodeVisitor to find ``*_client.find_*`` statements."""
    def __init__(self):
        self.error = False
    def visit_Call(self, node):
        # No need to keep visiting the AST if we already found something.
        if self.error:
            return
        self.generic_visit(node)
        if not (
            isinstance(node.func, ast.Attribute)
            and node.func.attr.startswith('find_')  # and
            # isinstance(node.func.value, ast.Attribute) and
            # node.func.value.attr.endswith('_client')
            # print(f'skipping: got {node.func}')
            return
        if not (
                # handle calls like 'identity_client.find_project'
                isinstance(node.func.value, ast.Name)
                and node.func.value.id.endswith('client')
                # handle calls like 'self.compute_client.find_server'
                isinstance(node.func.value, ast.Attribute)
                and node.func.value.attr.endswith('_client')
                # handle calls like 'self.app.client_manager.image.find_image'
                isinstance(node.func.value, ast.Attribute)
                and node.func.value.attr
                in ('identity', 'network', 'image', 'compute')
            return
        if not any(kw.arg == 'ignore_missing' for kw in node.keywords):
            self.error = True
@core.flake8ext
def assert_find_ignore_missing_kwargs(logical_line, filename):
    """Ensure ignore_missing is always used for ``find_*`` SDK proxy calls.
    Okay: self.compute_client.find_server(foo, ignore_missing=True)
    Okay: self.image_client.find_server(foo, ignore_missing=False)
    Okay: self.volume_client.volumes.find(name='foo')
    O403: self.network_client.find_network(parsed_args.network)
    O403: self.compute_client.find_flavor(flavor_id, get_extra_specs=True)
    if 'tests' in filename:
        return
    checker = SDKProxyFindChecker()
        parsed_logical_line = ast.parse(logical_line)
    except SyntaxError:
        # let flake8 catch this itself
        # https://github.com/PyCQA/flake8/issues/1948
        return
    checker.visit(parsed_logical_line)
    if checker.error:
        yield (
            'O403: Calls to find_* proxy methods must explicitly set '
            'ignore_missing',
@core.flake8ext
def assert_use_of_osc_command(logical_line, filename):
    """Ensure we use openstackclient.command instead of osc_lib.command.
    Okay: from openstackclient.command import command
    Okay: import openstackclient.command
    O404: from osc_lib.command import command
    if filename == 'openstackclient/command.py':
        return
    if re.match(r'^from osc_lib\.command import command$', logical_line):
        yield (
            'O404: Import Command classes from openstackclient.command, not '
            'osc_lib.command.command',
Sunbelt Computer Software

PL/B Language Development and Support

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Sunbelt Computer Software

PL/B Language Development and Support

Uh oh!

FilesExpand file tree

checks.py

Latest commit

History

checks.py

File metadata and controls
