Cleaned up some TODO comments for feature/qos that do not apply anymore
1. Removed comment to add tests for invalid input for attach/detach
methods for QosPolicy. Those tests are already implemented in the test
class, so the TODO became obsolete.
2. Removed TODO to use a constant for rule 'type' field. There is no
'type' field in rules anymore, so it does not apply any more.
Change-Id: I205cbc2d9a2eeee1a0a9fb5794efc063de6f326d
Partially-Implements: blueprint quantum-qos-api
diff --git a/neutron/tests/tempest/services/network/json/network_client.py b/neutron/tests/tempest/services/network/json/network_client.py
index c01c83c..daf563f 100644
--- a/neutron/tests/tempest/services/network/json/network_client.py
+++ b/neutron/tests/tempest/services/network/json/network_client.py
@@ -656,7 +656,6 @@
def create_bandwidth_limit_rule(self, policy_id, max_kbps, max_burst_kbps):
uri = '%s/qos/policies/%s/bandwidth_limit_rules' % (
self.uri_prefix, policy_id)
- #TODO(QoS): 'bandwidth_limit' should not be a magic string.
post_data = self.serialize(
{'bandwidth_limit_rule': {
'max_kbps': max_kbps,