summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorlei zhang <shleiz@cn.ibm.com>2015-11-28 23:44:14 +0800
committerlei zhang <shleiz@cn.ibm.com>2015-11-30 15:10:17 +0800
commitf2f496b656481b22885b071ce233cb978e63ac83 (patch)
tree2fc334db58967d8f06774e5b1f47eaceccda924f
parentd88f3a34ddb50db305c201f393874e21dc89a010 (diff)
downloadtempest-f2f496b656481b22885b071ce233cb978e63ac83.tar.gz
[V3] Rename get_policy to show_policy
[GET /resources] methods should be "list_<resource name>s" or "show_<resource name>", so this patch applies the rule to policy client. Partially implements blueprint consistent-service-method-names Change-Id: Ic4cfe4a5ec5695d943f7dca9579c545285fb2719
-rw-r--r--tempest/api/identity/admin/v3/test_policies.py2
-rw-r--r--tempest/hacking/ignored_list_T110.txt1
-rw-r--r--tempest/services/identity/v3/json/policy_client.py2
3 files changed, 2 insertions, 3 deletions
diff --git a/tempest/api/identity/admin/v3/test_policies.py b/tempest/api/identity/admin/v3/test_policies.py
index 44e5c7b5c..f38d25d1e 100644
--- a/tempest/api/identity/admin/v3/test_policies.py
+++ b/tempest/api/identity/admin/v3/test_policies.py
@@ -64,7 +64,7 @@ class PoliciesTestJSON(base.BaseIdentityV3AdminTest):
policy['id'], type=update_type)['policy']
self.assertIn('type', data)
# Assertion for updated value with fetched value
- fetched_policy = self.policy_client.get_policy(policy['id'])['policy']
+ fetched_policy = self.policy_client.show_policy(policy['id'])['policy']
self.assertIn('id', fetched_policy)
self.assertIn('blob', fetched_policy)
self.assertIn('type', fetched_policy)
diff --git a/tempest/hacking/ignored_list_T110.txt b/tempest/hacking/ignored_list_T110.txt
index dd3e48913..79dc062e5 100644
--- a/tempest/hacking/ignored_list_T110.txt
+++ b/tempest/hacking/ignored_list_T110.txt
@@ -1,6 +1,5 @@
./tempest/services/database/json/flavors_client.py
./tempest/services/identity/v3/json/identity_client.py
-./tempest/services/identity/v3/json/policy_client.py
./tempest/services/messaging/json/messaging_client.py
./tempest/services/object_storage/object_client.py
./tempest/services/telemetry/json/alarming_client.py
diff --git a/tempest/services/identity/v3/json/policy_client.py b/tempest/services/identity/v3/json/policy_client.py
index ecc9df70a..7927ed517 100644
--- a/tempest/services/identity/v3/json/policy_client.py
+++ b/tempest/services/identity/v3/json/policy_client.py
@@ -44,7 +44,7 @@ class PolicyClient(service_client.ServiceClient):
body = json.loads(body)
return service_client.ResponseBody(resp, body)
- def get_policy(self, policy_id):
+ def show_policy(self, policy_id):
"""Lists out the given policy."""
url = 'policies/%s' % policy_id
resp, body = self.get(url)