Skip to content

Commit

Permalink
tests: fix dependencies for tests
Browse files Browse the repository at this point in the history
* Remove the limitation on prospector as the upstream bug was fixed.
* Exclude flake8 from the minimum requirements as we just run the unit
  tests with the minimum requirements.
* This will require the removal of python3-flake8 from the Build-Depends
  on debian/control when doing the next release as flake8 is not needed
  when building the package.

Bug: T270795
Change-Id: I6abfc42bfd70b5c68e476884cf453aca35c6a72f
  • Loading branch information
volans- committed Dec 24, 2020
1 parent 5c18e5c commit 670dc5a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
3 changes: 3 additions & 0 deletions cumin/tests/vulture_whitelist.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ def __getattr__(self, _):
whitelist_cli = Whitelist()
whitelist_cli.run.h

whitelist_Config = Whitelist() # noqa: N816
whitelist_Config.__new__

whitelist_transports_clustershell = Whitelist()
whitelist_transports_clustershell.BaseEventHandler.kwargs

Expand Down
6 changes: 3 additions & 3 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,7 @@
'bandit>=1.5.1',
'flake8>=3.6.0',
'flake8-import-order>=0.18.1',
# Temporary upper constraint until https://github.com/PyCQA/prospector/issues/389 is fixed
'prospector[with_everything]>=1.1.7,<=1.2.0',
'prospector[with_everything]>=1.3.1',
'pytest-cov>=2.6.0',
'pytest-xdist>=1.26.1',
'pytest>=3.10.1',
Expand All @@ -49,7 +48,8 @@
extras_require['tests'].extend(extras_require['with-openstack'])

# Generate minimum dependencies
extras_require['tests-min'] = [dep.replace('>=', '==') for dep in extras_require['tests']]
extras_require['tests-min'] = [dep.replace('>=', '==') for dep in extras_require['tests']
if not dep.startswith('flake8')]
if os.getenv('CUMIN_MIN_DEPS', False):
install_requires = [dep.split(',')[0].replace('>=', '==') for dep in install_requires]

Expand Down

0 comments on commit 670dc5a

Please sign in to comment.