summaryrefslogtreecommitdiff
path: root/openstackclient/tests/functional/network
diff options
context:
space:
mode:
authorGlenn Van de Water <glenn.van_de_water@nuagenetworks.net>2019-03-08 16:31:23 +0100
committerGlenn Van de Water <glenn.van_de_water@nuagenetworks.net>2019-03-13 16:16:24 +0100
commit7741347041b078ca4d687597897194d7797d202d (patch)
treea8f7d26920c0b2d00f55676b90a816663131d77e /openstackclient/tests/functional/network
parent28c06d06885b3ae93da07eb14411d92c3df7e792 (diff)
downloadpython-openstackclient-7741347041b078ca4d687597897194d7797d202d.tar.gz
Fix service discovery in functional tests
If a required service is not enabled then we skip the test. The discovery is done by tests/functional/base.py:is_service_enabled but this method is broken, credentials are not passed to the 'openstack service show' command so every call will fail and every test that relies on it will be skipped. This commit fixed that method and the issues that popped up when re-enabling tests. Network segment range: - issue where we assumed network-segment-range extension is always present - issue where we compare integers and string representations of numbers Subnet: - issue where we try to deepcopy an uncopyable object in UnsetSubnet Change-Id: Id3cc907c1ed2a25b49cf6f4a7233e0401a02383a Story: 2005169 Task: 29908
Diffstat (limited to 'openstackclient/tests/functional/network')
-rw-r--r--openstackclient/tests/functional/network/v2/common.py2
-rw-r--r--openstackclient/tests/functional/network/v2/test_network_segment_range.py14
2 files changed, 8 insertions, 8 deletions
diff --git a/openstackclient/tests/functional/network/v2/common.py b/openstackclient/tests/functional/network/v2/common.py
index a18bc48f..5243ecd0 100644
--- a/openstackclient/tests/functional/network/v2/common.py
+++ b/openstackclient/tests/functional/network/v2/common.py
@@ -22,7 +22,7 @@ class NetworkTests(base.TestCase):
@classmethod
def setUpClass(cls):
super(NetworkTests, cls).setUpClass()
- cls.haz_network = base.is_service_enabled('network')
+ cls.haz_network = cls.is_service_enabled('network')
class NetworkTagTests(NetworkTests):
diff --git a/openstackclient/tests/functional/network/v2/test_network_segment_range.py b/openstackclient/tests/functional/network/v2/test_network_segment_range.py
index 95402dcb..c42f8744 100644
--- a/openstackclient/tests/functional/network/v2/test_network_segment_range.py
+++ b/openstackclient/tests/functional/network/v2/test_network_segment_range.py
@@ -28,6 +28,8 @@ class NetworkSegmentRangeTests(common.NetworkTests):
# Nothing in this class works with Nova Network
if not self.haz_network:
self.skipTest("No Network service present")
+ if not self.is_extension_enabled('network-segment-range'):
+ self.skipTest("No network-segment-range extension present")
self.PROJECT_NAME = uuid.uuid4().hex
def test_network_segment_range_create_delete(self):
@@ -83,7 +85,7 @@ class NetworkSegmentRangeTests(common.NetworkTests):
)
json_output = json.loads(self.openstack(
- 'network segment list -f json'
+ 'network segment range list -f json'
))
item_map = {
item.get('ID'): item.get('Name') for item in json_output
@@ -117,13 +119,11 @@ class NetworkSegmentRangeTests(common.NetworkTests):
json_output["project_id"],
)
- new_minimum = '2010'
- new_maximum = '2060'
+ new_minimum = 2010
+ new_maximum = 2060
cmd_output = self.openstack(
- 'network segment range set ' +
- '--minimum ' + new_minimum + ' ' +
- '--maximum ' + new_maximum + ' ' +
- name
+ 'network segment range set --minimum {min} --maximum {max} {name}'
+ .format(min=new_minimum, max=new_maximum, name=name)
)
self.assertOutput('', cmd_output)