Add missing baremetal API tests
This adds some missing API test coverage for listing ports by
node id, node by instance uuid, node by chassis id, and listing
nodes by instance association status. In addition to the required
client changes, this adds a common waiter for baremetal nodes which
can be leveraged in the future by the baremetal scenario tests.
Change-Id: I175da30516d697a2b3115353f405e0fe3e1ccc76
diff --git a/tempest/services/baremetal/base.py b/tempest/services/baremetal/base.py
index 0b97f74..4933300 100644
--- a/tempest/services/baremetal/base.py
+++ b/tempest/services/baremetal/base.py
@@ -95,9 +95,13 @@
for ch in get_change(value, path + '%s/' % name):
yield ch
else:
- yield {'path': path + name,
- 'value': value,
- 'op': 'replace'}
+ if value is None:
+ yield {'path': path + name,
+ 'op': 'remove'}
+ else:
+ yield {'path': path + name,
+ 'value': value,
+ 'op': 'replace'}
patch = [ch for ch in get_change(kw)
if ch['path'].lstrip('/') in allowed_attributes]