commit ebd5e419d70b8515fbe98d1be865dcbb510a9f5f
author zhufl <zhu.fanglei@zte.com.cn> Mon Mar 13 16:42:32 2017 +0800
committer zhufl <zhu.fanglei@zte.com.cn> Tue Apr 25 16:13:12 2017 +0800
tree 077c021287c63dffdc7e12049c617844ad4ab780
parent e1d9fd3a698ce8d09d019f38dc8926dd3603f32c

Skip when force_tenant_isolation=true and no admin credentials

AbsoluteLimitsTests requires force_tenant_isolation=True,
and if admin credentials are not present, the testcase will
fail instead of being skipped.

This is to let AbsoluteLimitsTests inherit from BaseVolumeAdminTest
so it can skip if admin credentials are not present.

Change-Id: Id71a705cf9b1dd0c0d41a2fb45ab77c95430a123
Closes-Bug: #1671256

tempest/api/volume/test_volume_absolute_limits.py

diff --git a/tempest/api/volume/test_volume_absolute_limits.py b/tempest/api/volume/test_volume_absolute_limits.py
index 836e489..870b9f0 100644
--- a/tempest/api/volume/test_volume_absolute_limits.py
+++ b/tempest/api/volume/test_volume_absolute_limits.py
@@ -21,7 +21,10 @@
 CONF = config.CONF
 
 
-class AbsoluteLimitsTests(base.BaseVolumeTest):
+# NOTE(zhufl): This inherits from BaseVolumeAdminTest because
+# it requires force_tenant_isolation=True, which need admin
+# credentials to create non-admin users for the tests.
+class AbsoluteLimitsTests(base.BaseVolumeAdminTest):
 
     # avoid existing volumes of pre-defined tenant
     force_tenant_isolation = True