Murano

[tests] Need to rewrite cleanup mechanism in tempest api tests

Bug #1512770 reported by Anastasia Kuznetsova
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Murano
Invalid
Medium
Unassigned
Murano mitaka-rc1 "RC1"

Bug Description

Current implementation of tempest api functional tests uses incorrect, ugly cleanup mechanism:
all created test objects save their id/name in the array and in the teardown method there is a 'for' loop that goes by created array with and call delete method for appopriate object.

Need to use standard addCleanup method.

Nikolay Starodubtsev (starodubcevna)
Changed in murano:
importance: Undecided → Medium
milestone: none → mitaka-2
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to murano (master)

Fix proposed to branch: master
Review: https://review.openstack.org/242875

Changed in murano:
assignee: Anastasia Kuznetsova (akuznetsova) → Akanksha Srivastava (akanksha-dlf)
status: Confirmed → In Progress
Revision history for this message
Kirill Zaitsev (kzaitsev) wrote :

It is usually not a good idea to try and fix a bug, that is assigned to someone else. Especially after she confirms, that she is working on the fix. Even if you believe, that the fix is easy. Please contact the person before taking ownership of the bug. There are plenty of other things to do and it would be wise to not duplicate someone else's work.

Changed in murano:
assignee: Akanksha Srivastava (akanksha-dlf) → Anastasia Kuznetsova (akuznetsova)
Kirill Zaitsev (kzaitsev)
Changed in murano:
milestone: mitaka-2 → mitaka-3
Kirill Zaitsev (kzaitsev)
Changed in murano:
milestone: mitaka-3 → mitaka-rc1
Kirill Zaitsev (kzaitsev)
Changed in murano:
assignee: Anastasia Kuznetsova (akuznetsova) → nobody
Revision history for this message
Kirill Zaitsev (kzaitsev) wrote :

already released/updated

Changed in murano:
status: In Progress → Confirmed
status: Confirmed → Fix Committed
Kirill Zaitsev (kzaitsev)
Changed in murano:
status: Fix Committed → Invalid
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on murano (master)

Change abandoned by Akanksha Srivastava (<email address hidden>) on branch: master
Review: https://review.openstack.org/242875
Reason: Abandoning because the test logic has changed since this patch and this work becomes irrelevant.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.