[jira] [Commented] (HBASE-9058) deleteSnapshot() call may be skipped in TestFlushSnapshotFromClient tests

["Matteo Bertozzi (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HBASE-9058?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13721639#comment-13721639 ] 

Matteo Bertozzi commented on HBASE-9058:
----------------------------------------

instead of adding a finally block in every test, the foreach snapshot delete should be added to the tearDown() method.. since every test is doing that anyway.
                
> deleteSnapshot() call may be skipped in TestFlushSnapshotFromClient tests
> -------------------------------------------------------------------------
>
>                 Key: HBASE-9058
>                 URL: https://issues.apache.org/jira/browse/HBASE-9058
>             Project: HBase
>          Issue Type: Test
>            Reporter: Ted Yu
>            Priority: Minor
>
> At the end of each test, we have this call:
>     admin.deleteSnapshot(snapshotName);
> However it is not placed in finally block. When assertion in earlier part of the test fails (See [1]), snapshot cleanup would be skipped, leading to SnapshotTestingUtils.assertNoSnapshots() failing for subsequent tests.
> [1]: https://builds.apache.org/job/hbase-0.95-on-hadoop2/197/testReport/org.apache.hadoop.hbase.snapshot/TestFlushSnapshotFromClient/testFlushCreateListDestroy/

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira