The mock third party library was needed for mock support in py2
runtimes. Since we now only support py36 and later, we don't need it
in lower-constraints and requirements.
These changes will help us to drop `mock` from openstack/requirements
Change-Id: I1a31c6c5a04e0e883cf956a5ae74f444b1673332
flake8 new release 3.8.0 added new checks and gate pep8
job start failing. hacking 3.0.1 fix the pinning of flake8 to
avoid bringing in a new version with new checks.
Though it is fixed in latest hacking but 2.0 and 3.0 has cap for
flake8 as <4.0.0 which mean flake8 new version 3.9.0 can also
break the pep8 job if new check are added.
To avoid similar gate break in future, we need to bump the hacking min
version.
- http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014828.html
Change-Id: If3e913f5251ecbdcc996199329ec27daa80486ee
The repo is Python 3 now, so update hacking to version 3.0 which
supports Python 3.
Fix problems found.
Update local hacking checks for new flake8.
Change-Id: Iad5e3891c59713f7dd2cdb0a691a964ff37108cf
Ignore W504 and F405 by configurations.
Reasons of disabling them are explained as comments in tox.ini.
They are already ignored in the horizon repository and
AFAICT it is not easy to avoid them in horizon plugins as well.
Fixes the following new error:
* E123 closing bracket does not match indentation of opening bracket's line
Change-Id: Ic2154fff3b90cebbfba86deb8cf832232682d347
nose is not actively maintained now and Django default test runner
seems to have enough features.
Also drops unnecessary python dependencies so that
the maintenance of requirements files easily.
There is no need to install Django explicitly as it is a horizon dependency.
testscenarios, testrepository, testtools are not used actually.
Depends-On: https://review.openstack.org/#/c/544296/
Change-Id: Ic360c181d1f3278957007fc7188560893cc5ed03
create_stubs in test.helpers is also dropped in this commit
because this creates a stub using mox.
This is part of mox-removal community goal in Rocky.
Depends-On: https://review.openstack.org/549842/
Change-Id: I150d654b40dd14bb738e0da991e5db7f311cf7b6
Closes-Bug: #1753504
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.
application-import-names needs to be specified in tox.ini
to ensure application imports are placed after third-party imports.
Change-Id: If2067b950737ae7ebc65b2311955754be795758f
In openstackdocstheme 0.11.0, the way to configure openstackdocstheme
was changed and the new way is really simple. Let's use it.
Change-Id: I3f8775cc54ece19915ab7b06285a0e7d2995eb8b
* Adopt openstackdocstheme
* Document structure follows doc-migration spec
https://review.openstack.org/#/c/472275/
* Release notes is generated as part of doc build to make
doc publishing easy. neutron-vpnaas is not an official project
and cannot publish it to docs.o.o.
Change-Id: I4fb1058fc078b32d549db3c66d91c85e0bdbe7a5