HBASE-26920 Fix missing braces warnings in TestProcedureMember (apach…

…e#4315)

Signed-off-by: Xiaolin Ha <haxiaolin@apache.org>
(cherry picked from commit 78676bb)

hbase-server/src/test/java/org/apache/hadoop/hbase/procedure/TestProcedureMember.java

@@ -48,6 +48,8 @@
 import org.mockito.invocation.InvocationOnMock;
 import org.mockito.stubbing.Answer;
 
+import org.apache.hbase.thirdparty.com.google.common.io.Closeables;
+
 /**
  * Test the procedure member, and it's error handling mechanisms.
  */
@@ -77,14 +79,9 @@ public class TestProcedureMember {
    * Reset all the mock objects
    */
   @After
-  public void resetTest() {
+  public void resetTest() throws IOException {
     reset(mockListener, mockBuilder, mockMemberComms);
-    if (member != null)
-      try {
-        member.close();
-      } catch (IOException e) {
-        e.printStackTrace();
-      }
+    Closeables.close(member, true);
   }
 
   /**
@@ -273,7 +270,7 @@ public Void answer(InvocationOnMock invocation) throws Throwable {
 
   /**
    * Handle failures if a member's commit phase fails.
-   *
+   * <p/>
    * NOTE: This is the core difference that makes this different from traditional 2PC.  In true
    * 2PC the transaction is committed just before the coordinator sends commit messages to the
    * member.  Members are then responsible for reading its TX log.  This implementation actually
@@ -314,7 +311,7 @@ public Void answer(InvocationOnMock invocation) throws Throwable {
 
   /**
    * Handle Failures if a member's commit phase succeeds but notification to coordinator fails
-   *
+   * <p/>
    * NOTE: This is the core difference that makes this different from traditional 2PC.  In true
    * 2PC the transaction is committed just before the coordinator sends commit messages to the
    * member.  Members are then responsible for reading its TX log.  This implementation actually