openstack
/
manila-tempest-plugin
Code Issues Proposed changes

Fix usage of tenant_id in tests 

Some tests assume that the base test classes
set a tenant_id parameter at the class level.
This is not the case because base test classes
most often are capable of dealing with multiple
tenants via tempest's test credential framework.

Change-Id: If9b57f0f78c43d5f17738f13cc86b0594d3d2994
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
This commit is contained in:
Goutham Pacha Ravi 2020-07-14 21:34:34 -07:00
parent e52b78d3d8
commit 38544589c4
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manila_tempest_tests/tests/api/test_quotas.py
View File

@ -35,8 +35,8 @@ class SharesQuotasTest(base.BaseSharesTest):
msg = "Quota tests are disabled."
raise cls.skipException(msg)
super(SharesQuotasTest, cls).resource_setup()
cls.user_id = cls.shares_v2_client.user_id or cls.user_id
cls.tenant_id = cls.shares_v2_client.tenant_id or cls.tenant_id
cls.user_id = cls.shares_v2_client.user_id
cls.tenant_id = cls.shares_v2_client.tenant_id
@tc.attr(base.TAG_POSITIVE, base.TAG_API)
@ddt.data('shares_client', 'shares_v2_client')

2
manila_tempest_tests/tests/api/test_rules.py
View File

@ -505,7 +505,7 @@ class ShareCephxRulesForCephFSTest(base.BaseSharesMixedTest):
# Create a new user in the current project
project = self.os_admin.projects_client.show_project(
self.tenant_id)['project']
self.shares_v2_client.tenant_id)['project']
user_client = self.create_user_and_get_client(project)
# Create second share by the new user