diff options
author | Akihiro Motoki <amotoki@gmail.com> | 2019-07-18 16:31:38 +0900 |
---|---|---|
committer | Akihiro Motoki <amotoki@gmail.com> | 2019-07-18 20:28:55 +0900 |
commit | 2af19d55d6029712f3888a37c019be5a25eba1fe (patch) | |
tree | 34df5792055c4895724cf730a65f8184f86d3a19 | |
parent | cf95be3ea5e4396f26564a76e5ab67342afe6de5 (diff) | |
download | python-neutronclient-2af19d55d6029712f3888a37c019be5a25eba1fe.tar.gz |
Run functional test correctly
legacy-neutronclient-test-dsvm-functional job actually runs
neutronclient.tests.unit because .stestr.conf does not honor
OS_TEST_PATH. This commit fixes .stestr.conf to honor OS_TEST_PATH
specified in tox.ini.
Also fixes the logic of is_extension_enabled() in ClientTestBase
to check whether FWaaS v1 (fwaas) is enabled correctly.
Previously the logic checks a substring of a specified extension,
so 'fwaas' (FWaaS v1) matches 'fwaas_v2' (FWaaS v2),
which leads to a failure of FWaaS v1 CLI tests.
Change-Id: I958ad496b16cca8d03a7b84ebf5f8f031e4248ea
-rw-r--r-- | .stestr.conf | 2 | ||||
-rw-r--r-- | neutronclient/tests/functional/base.py | 6 |
2 files changed, 3 insertions, 5 deletions
diff --git a/.stestr.conf b/.stestr.conf index 6a6b6f1..875bb14 100644 --- a/.stestr.conf +++ b/.stestr.conf @@ -1,3 +1,3 @@ [DEFAULT] -test_path=./neutronclient/tests/unit +test_path=${OS_TEST_PATH:-./neutronclient/tests/unit} top_dir=./ diff --git a/neutronclient/tests/functional/base.py b/neutronclient/tests/functional/base.py index d92d693..655c6c4 100644 --- a/neutronclient/tests/functional/base.py +++ b/neutronclient/tests/functional/base.py @@ -76,7 +76,5 @@ class ClientTestBase(base.ClientTestBase): def is_extension_enabled(self, extension_alias): extensions = self.parser.listing(self.neutron('ext-list')) - for extension in extensions: - if extension_alias in extension['alias']: - return True - return False + aliases = [e['alias'] for e in extensions] + return extension_alias in aliases |