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

• Type: Test
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Hadoop Flags: Reviewed
• Assignee: Ted Yu
• Reporter: Ted Yu
• Created: 27/Jul/13 14:40
• Updated: 16/Jun/22 17:47
• Resolved: 29/Jul/13 16:50

Description

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/