Updates for py312 compatibility
- Removed pins to old version of setuptools. - Added py312 section to tox.ini [testenv] - Updated flake8 to 7.1.1 instead of 3.9.2 - use assertEqual in unit tests instead of the removed assertEquals Change-Id: Ia87ff66df41854f30586f341f7d481dea775edb5
This commit is contained in:
parent
d35ce67d9e
commit
c3c8db7497
@ -3,10 +3,6 @@
|
||||
# choices of *requirements.txt files for OpenStack Charms:
|
||||
# https://github.com/openstack-charmers/release-tools
|
||||
#
|
||||
# NOTE(lourot): This might look like a duplication of test-requirements.txt but
|
||||
# some tox targets use only test-requirements.txt whereas charm-build uses only
|
||||
# requirements.txt
|
||||
setuptools<50.0.0 # https://github.com/pypa/setuptools/commit/04e3df22df840c6bb244e9b27bc56750c44b7c85
|
||||
|
||||
# NOTE: newer versions of cryptography require a Rust compiler to build,
|
||||
# see
|
||||
|
@ -4,7 +4,6 @@
|
||||
# https://github.com/openstack-charmers/release-tools
|
||||
#
|
||||
pyparsing<3.0.0 # aodhclient is pinned in zaza and needs pyparsing < 3.0.0, but cffi also needs it, so pin here.
|
||||
setuptools<50.0.0 # https://github.com/pypa/setuptools/commit/04e3df22df840c6bb244e9b27bc56750c44b7c85
|
||||
|
||||
stestr>=2.2.0
|
||||
|
||||
|
9
tox.ini
9
tox.ini
@ -73,9 +73,16 @@ deps =
|
||||
-r{toxinidir}/test-requirements.txt
|
||||
commands = stestr run --slowest {posargs}
|
||||
|
||||
[testenv:py312]
|
||||
basepython = python3.12
|
||||
deps =
|
||||
-c {env:TEST_CONSTRAINTS_FILE:https://raw.githubusercontent.com/openstack-charmers/zaza-openstack-tests/master/constraints/constraints-2024.1.txt}
|
||||
-r{toxinidir}/test-requirements.txt
|
||||
commands = stestr run --slowest {posargs}
|
||||
|
||||
[testenv:pep8]
|
||||
basepython = python3
|
||||
deps = flake8==3.9.2
|
||||
deps = flake8==7.1.1
|
||||
git+https://github.com/juju/charm-tools.git
|
||||
commands = flake8 {posargs} src unit_tests
|
||||
|
||||
|
@ -85,9 +85,9 @@ class TestBarbicanVaultHandlers(test_utils.PatchHelper):
|
||||
endpoint_from_flag.all_unit_tokens = ['token1']
|
||||
endpoint_from_flag.vault_url = 'https://foo.fl:8200'
|
||||
retrieve_secret_id.return_value = 'big-secret'
|
||||
self.assertEquals(handlers.get_secret_id(endpoint_from_flag,
|
||||
'old-secret'),
|
||||
'big-secret')
|
||||
self.assertEqual(handlers.get_secret_id(endpoint_from_flag,
|
||||
'old-secret'),
|
||||
'big-secret')
|
||||
|
||||
@mock.patch.object(handlers.vault_utils, 'retrieve_secret_id')
|
||||
@mock.patch.object(handlers.reactive, 'endpoint_from_flag')
|
||||
@ -100,9 +100,9 @@ class TestBarbicanVaultHandlers(test_utils.PatchHelper):
|
||||
raise self.fake_hvac.exceptions.InvalidRequest
|
||||
|
||||
retrieve_secret_id.side_effect = fail
|
||||
self.assertEquals(handlers.get_secret_id(endpoint_from_flag,
|
||||
'old-secret'),
|
||||
'old-secret')
|
||||
self.assertEqual(handlers.get_secret_id(endpoint_from_flag,
|
||||
'old-secret'),
|
||||
'old-secret')
|
||||
|
||||
def test_plugin_info_barbican_publish(self):
|
||||
barbican_vault_charm = self.patch_charm()
|
||||
|
Loading…
Reference in New Issue
Block a user