openstack
/
ironic-tempest-plugin
Code Issues Proposed changes

Swap maintenance fix to cleanup step 

Follow-up to I89839ecb0e08555ddcfee0289350fac1923b3958 to
use a cleanup step so the maintenance state is rolled back
regardless.

Change-Id: I52a5c36d8ca1965d866409fee4590cc1d151f238
This commit is contained in:
Julia Kreger 2022-07-20 06:59:04 -07:00
parent 8d176bd321
commit a766d8e426
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
ironic_tempest_plugin/tests/api/admin/test_nodes.py
View File

@ -911,13 +911,13 @@ class TestNodeProtected(base.BaseBaremetalTest):
self.deploy_node(self.node['uuid'])
self.client.update_node(self.node['uuid'], protected=True,
maintenance=True)
# undo maintenance because we can't teardown something
# in maintenance.
self.addCleanup(self.client.update_node, self.node['uuid'],
maintenance=False)
self.assertRaises(lib_exc.Forbidden,
self.client.delete_node,
self.node['uuid'])
# undo maintenance because we can't teardown something
# in maintenance.
self.client.update_node(self.node['uuid'], maintenance=False)
@decorators.attr(type='negative')
@decorators.idempotent_id('1c819f4c-6c1d-4150-ba4a-3b0dcb3c8694')