diff options
| author | Akihiro Motoki <amotoki@gmail.com> | 2017-07-15 16:58:08 +0000 |
|---|---|---|
| committer | Akihiro Motoki <amotoki@gmail.com> | 2017-08-17 06:55:00 +0000 |
| commit | f3bbf52b3c7e731796aaf38294d08109859babe5 (patch) | |
| tree | a73c02ce01812683f538a084c2cfeb486229a9d2 /openstackclient/tests/unit/common | |
| parent | 53bea90a75774cb7468884da43b55381b491dc46 (diff) | |
| download | python-openstackclient-f3bbf52b3c7e731796aaf38294d08109859babe5.tar.gz | |
Use flake8-import-order plugin
In reviews we usually check import grouping but it is boring.
By using flake8-import-order plugin, we can avoid this.
It enforces loose checking so it sounds good to use it.
This flake8 plugin is already used in tempest.
Note that flake8-import-order version is pinned to avoid unexpected
breakage of pep8 job.
Setup for unit tests of hacking rules is tweaked to disable
flake8-import-order checks. This extension assumes an actual file exists
and causes hacking rule unit tests.
Change-Id: I12b596820727aeeb379bee16c2bc993dee9eb637
Diffstat (limited to 'openstackclient/tests/unit/common')
| -rw-r--r-- | openstackclient/tests/unit/common/test_logs.py | 1 | ||||
| -rw-r--r-- | openstackclient/tests/unit/common/test_quota.py | 2 |
2 files changed, 2 insertions, 1 deletions
diff --git a/openstackclient/tests/unit/common/test_logs.py b/openstackclient/tests/unit/common/test_logs.py index 4842c8d4..b1e4d612 100644 --- a/openstackclient/tests/unit/common/test_logs.py +++ b/openstackclient/tests/unit/common/test_logs.py @@ -15,6 +15,7 @@ # or Jun 2017. import logging + import mock from openstackclient.common import logs diff --git a/openstackclient/tests/unit/common/test_quota.py b/openstackclient/tests/unit/common/test_quota.py index 482653f4..1a3da31d 100644 --- a/openstackclient/tests/unit/common/test_quota.py +++ b/openstackclient/tests/unit/common/test_quota.py @@ -11,8 +11,8 @@ # under the License. import copy -import mock +import mock from osc_lib import exceptions from openstackclient.common import quota |