Fix list-all-project tests for pagination limit
Designate has a fairly small default pagination limit of 20.
Some tests were making list API calls with the all-projects header,
which with parallel test execution could cause the list to exceed the
default pagination limit of 20. This would cause the tests to intermittently
fail.
This patch sets the request parameter "limit" to 1000 for these tests
to allow for parallel test execution which may cause lists to include
more than twenty entries.
Change-Id: Ic074c2c4dc5a5f97c609aac5992f78557722bb71
diff --git a/designate_tempest_plugin/tests/api/v2/test_recordset.py b/designate_tempest_plugin/tests/api/v2/test_recordset.py
index 70e74c6..96179f1 100644
--- a/designate_tempest_plugin/tests/api/v2/test_recordset.py
+++ b/designate_tempest_plugin/tests/api/v2/test_recordset.py
@@ -281,10 +281,14 @@
LOG.info('Re-Fetch Recordsets as Admin tenant for a Primary project '
'using "x-auth-all-projects" HTTP header.')
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
primary_recordsets_ids = [
item['id'] for item in self.admin_client.list_recordset(
self.zone['id'],
- headers={'x-auth-all-projects': True})[1]['recordsets']]
+ headers={'x-auth-all-projects': True},
+ params={'limit': 1000})[1]['recordsets']]
for recordset_id in [body_pr_1['id'], body_pr_2['id']]:
self.assertIn(
@@ -702,8 +706,11 @@
project_ids_used = [
item['project_id'] for item in self._create_client_recordset(
['primary', 'alt']).values()]
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
recordsets = self.admin_client.list_owned_recordsets(
- headers={'x-auth-all-projects': True})
+ headers={'x-auth-all-projects': True}, params={'limit': 1000})
LOG.info('Received by API recordsets are {} '.format(recordsets))
project_ids_api = set([item['project_id'] for item in recordsets])
for prj_id in project_ids_used:
diff --git a/designate_tempest_plugin/tests/api/v2/test_transfer_accepts.py b/designate_tempest_plugin/tests/api/v2/test_transfer_accepts.py
index 831619e..53147a8 100644
--- a/designate_tempest_plugin/tests/api/v2/test_transfer_accepts.py
+++ b/designate_tempest_plugin/tests/api/v2/test_transfer_accepts.py
@@ -184,11 +184,14 @@
# As Admin list all accepted zone transfers, expected:
# each previously transferred zone is listed.
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
LOG.info('Use Admin client to list all "accepted zone transfers"')
admin_client_accept_ids = [
item['id'] for item in
self.admin_accept_client.list_transfer_accept(
- headers={'x-auth-all-projects': True})]
+ headers={'x-auth-all-projects': True}, params={'limit': 1000})]
for tr_id in transfer_request_ids:
self.assertIn(
tr_id, admin_client_accept_ids,
diff --git a/designate_tempest_plugin/tests/api/v2/test_transfer_request.py b/designate_tempest_plugin/tests/api/v2/test_transfer_request.py
index 666fd79..94a1480 100644
--- a/designate_tempest_plugin/tests/api/v2/test_transfer_request.py
+++ b/designate_tempest_plugin/tests/api/v2/test_transfer_request.py
@@ -228,9 +228,13 @@
LOG.info('List transfer_requests for all projects using Admin tenant '
'and "x-auth-all-projects" HTTP header.')
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
request_ids = [
item['id'] for item in self.admin_client.list_transfer_requests(
- headers={'x-auth-all-projects': True})[1]['transfer_requests']]
+ headers={'x-auth-all-projects': True},
+ params={'limit': 1000})[1]['transfer_requests']]
for request_id in [primary_transfer_request['id'],
alt_transfer_request['id']]:
diff --git a/designate_tempest_plugin/tests/api/v2/test_zones.py b/designate_tempest_plugin/tests/api/v2/test_zones.py
index 78b8d4a..c2337fe 100644
--- a/designate_tempest_plugin/tests/api/v2/test_zones.py
+++ b/designate_tempest_plugin/tests/api/v2/test_zones.py
@@ -270,8 +270,12 @@
self.admin_client, admin_zone['id'], 'ACTIVE')
LOG.info('As admin user list all projects zones')
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
body = self.admin_client.list_zones(
- headers={'x-auth-all-projects': True})[1]['zones']
+ headers={'x-auth-all-projects': True},
+ params={'limit': 1000})[1]['zones']
listed_zone_ids = [item['id'] for item in body]
LOG.info('Ensure the fetched response includes all zone '
diff --git a/designate_tempest_plugin/tests/api/v2/test_zones_exports.py b/designate_tempest_plugin/tests/api/v2/test_zones_exports.py
index ef9ba6d..a87e105 100644
--- a/designate_tempest_plugin/tests/api/v2/test_zones_exports.py
+++ b/designate_tempest_plugin/tests/api/v2/test_zones_exports.py
@@ -146,9 +146,13 @@
self.addCleanup(self.alt_client.delete_zone_export, alt_export['id'])
LOG.info('As admin user list zone exports for all projects')
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
listed_exports_ids = [
item['id'] for item in self.admin_client.list_zone_exports(
- headers={'x-auth-all-projects': True})[1]['exports']]
+ headers={'x-auth-all-projects': True},
+ params={'limit': 1000})[1]['exports']]
LOG.info('Make sure that all previously created zone '
'export IDs are listed')
diff --git a/designate_tempest_plugin/tests/api/v2/test_zones_imports.py b/designate_tempest_plugin/tests/api/v2/test_zones_imports.py
index 1d5f89e..332899d 100644
--- a/designate_tempest_plugin/tests/api/v2/test_zones_imports.py
+++ b/designate_tempest_plugin/tests/api/v2/test_zones_imports.py
@@ -199,8 +199,12 @@
headers={'x-auth-all-projects': True}))
LOG.info('As Admin tenant list import zones for all projects')
+ # Note: This is an all-projects list call, so other tests running
+ # in parallel will impact the list result set. Since the default
+ # pagination limit is only 20, we set a param limit of 1000 here.
body = self.admin_client.list_zone_imports(headers={
- 'x-auth-all-projects': True})[1]['imports']
+ 'x-auth-all-projects': True},
+ params={'limit': 1000})[1]['imports']
LOG.info('Ensure the fetched response includes previously '
'created import ID')