From: Ihar Hrachyshka <ihrachys@redhat.com>
Date: Wed, 29 Jul 2015 18:34:49 +0000 (+0200)
Subject: Enable rule delete test
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=80ff953069dc096383e04350a7013971214e1e5d;p=openstack-build%2Fneutron-build.git

Enable rule delete test

Change-Id: Ic950db35aec66fc0f81070a0641e0473f70d765c
Partially-Implements: blueprint quantum-qos-api
---

diff --git a/neutron/tests/api/test_qos.py b/neutron/tests/api/test_qos.py
index e4b05321d..845ef61cc 100644
--- a/neutron/tests/api/test_qos.py
+++ b/neutron/tests/api/test_qos.py
@@ -277,25 +277,24 @@ class QosBandwidthLimitRuleTestJSON(base.BaseAdminNetworkTest):
         self.assertEqual(200, retrieved_policy['max_kbps'])
         self.assertEqual(1337, retrieved_policy['max_burst_kbps'])
 
-    #TODO(QoS): Uncomment once the rule-delete logic is fixed.
-#    @test.attr(type='smoke')
-#    @test.idempotent_id('67ee6efd-7b33-4a68-927d-275b4f8ba958')
-#    def test_rule_delete(self):
-#        policy = self.create_qos_policy(name='test-policy',
-#                                        description='test policy',
-#                                        shared=False)
-#        rule = self.admin_client.create_bandwidth_limit_rule(
-#            policy['id'], 200, 1337)['bandwidth_limit_rule']
-#
-#        retrieved_policy = self.admin_client.show_bandwidth_limit_rule(
-#            policy['id'], rule['id'])
-#        retrieved_policy = retrieved_policy['bandwidth_limit_rule']
-#        self.assertEqual(rule['id'], retrieved_policy['id'])
-#
-#        self.admin_client.delete_bandwidth_limit_rule(policy['id'], rule['id']
-#        self.assertRaises(exceptions.ServerFault,
-#                          self.admin_client.show_bandwidth_limit_rule,
-#                          policy['id'], rule['id'])
+    @test.attr(type='smoke')
+    @test.idempotent_id('67ee6efd-7b33-4a68-927d-275b4f8ba958')
+    def test_rule_delete(self):
+        policy = self.create_qos_policy(name='test-policy',
+                                        description='test policy',
+                                        shared=False)
+        rule = self.admin_client.create_bandwidth_limit_rule(
+            policy['id'], 200, 1337)['bandwidth_limit_rule']
+
+        retrieved_policy = self.admin_client.show_bandwidth_limit_rule(
+            policy['id'], rule['id'])
+        retrieved_policy = retrieved_policy['bandwidth_limit_rule']
+        self.assertEqual(rule['id'], retrieved_policy['id'])
+
+        self.admin_client.delete_bandwidth_limit_rule(policy['id'], rule['id'])
+        self.assertRaises(exceptions.ServerFault,
+                          self.admin_client.show_bandwidth_limit_rule,
+                          policy['id'], rule['id'])
 
     #TODO(QoS): create several bandwidth-limit rules (not sure it makes sense,
     #           but to test more than one rule)