commit cfa1638457068537a0f99ee165611184e7e60e8b
author Dennis Dmitriev <ddmitriev@mirantis.com> Wed Apr 26 16:09:59 2017 +0300
committer Dennis Dmitriev <ddmitriev@mirantis.com> Wed Apr 26 16:09:59 2017 +0300
tree f8d2f51184c56ce967ba646f10a1d1ef304a4a97
parent 45396dbd63e79f8d4955c8ddec767568c11b7751

Fix test finalizer regression

tcp_tests/fixtures/underlay_fixtures.py

diff --git a/tcp_tests/fixtures/underlay_fixtures.py b/tcp_tests/fixtures/underlay_fixtures.py
index 43024e3..d7351bf 100644
--- a/tcp_tests/fixtures/underlay_fixtures.py
+++ b/tcp_tests/fixtures/underlay_fixtures.py
@@ -137,16 +137,17 @@
                 and snapshot_needed:
             snapshot_name = utils.extract_name_from_mark(snapshot_needed) or \
                 "{}_passed".format(default_snapshot_name)
+            hardware.create_snapshot(snapshot_name)
 
         elif hasattr(request.node, 'rep_setup') and \
                 request.node.rep_setup.failed and fail_snapshot:
             snapshot_name = "{0}_prep_failed".format(default_snapshot_name)
+            hardware.create_snapshot(snapshot_name)
 
         elif hasattr(request.node, 'rep_call') and \
                 request.node.rep_call.failed and fail_snapshot:
             snapshot_name = "{0}_failed".format(default_snapshot_name)
-
-        hardware.create_snapshot(snapshot_name)
+            hardware.create_snapshot(snapshot_name)
 
     request.addfinalizer(test_fin)