Repository: hadoop
Updated Branches:
  refs/heads/branch-2-jhung-test be705b531 -> 78030ccb3


corrupt stdin


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/78030ccb
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/78030ccb
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/78030ccb

Branch: refs/heads/branch-2-jhung-test
Commit: 78030ccb3b0bd7a2cf47cf1cd8957220ebc24ca5
Parents: be705b5
Author: Jonathan Hung <[email protected]>
Authored: Tue Dec 18 14:05:55 2018 -0800
Committer: Jonathan Hung <[email protected]>
Committed: Tue Dec 18 14:05:55 2018 -0800

----------------------------------------------------------------------
 .../apache/hadoop/hdfs/server/namenode/TestClusterId.java   | 4 +++-
 .../java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java   | 9 +++++++++
 2 files changed, 12 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/78030ccb/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestClusterId.java
----------------------------------------------------------------------
diff --git 
a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestClusterId.java
 
b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestClusterId.java
index 549487e..3796a58 100644
--- 
a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestClusterId.java
+++ 
b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestClusterId.java
@@ -51,6 +51,7 @@ import org.apache.hadoop.util.ExitUtil;
 import org.apache.hadoop.util.ExitUtil.ExitException;
 import org.junit.After;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 
 public class TestClusterId {
@@ -391,6 +392,7 @@ public class TestClusterId {
    * @throws InterruptedException
    */
   @Test
+  @Ignore
   public void testFormatWithoutForceEnterYes() throws IOException,
       InterruptedException {
 
@@ -486,4 +488,4 @@ public class TestClusterId {
     config.setBoolean(DFSConfigKeys.DFS_REFORMAT_DISABLED, true);
     DFSTestUtil.formatNameNode(config);
   }
-}
\ No newline at end of file
+}

http://git-wip-us.apache.org/repos/asf/hadoop/blob/78030ccb/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
----------------------------------------------------------------------
diff --git 
a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
 
b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
index f1f5793..5f77f91 100644
--- 
a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
+++ 
b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
@@ -25,6 +25,7 @@ import static org.junit.Assert.fail;
 import java.io.ByteArrayInputStream;
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
+import java.io.InputStream;
 import java.io.PrintStream;
 
 import org.apache.commons.logging.Log;
@@ -227,6 +228,8 @@ public class TestDFSHAAdmin {
     assertEquals(0, runTool("-transitionToActive", "-forcemanual", "nn1"));
     setupConfirmationOnSystemIn();
     assertEquals(0, runTool("-transitionToStandby", "-forcemanual", "nn1"));
+    undoSystemIn();
+    
 
     Mockito.verify(mockProtocol, Mockito.times(1)).transitionToActive(
         reqInfoCaptor.capture());
@@ -243,10 +246,16 @@ public class TestDFSHAAdmin {
    * Setup System.in with a stream that feeds a "yes" answer on the
    * next prompt.
    */
+  private static InputStream oldIn;
   private static void setupConfirmationOnSystemIn() {
    // Answer "yes" to the prompt about transition to active
+    oldIn = System.in;
    System.setIn(new ByteArrayInputStream("yes\n".getBytes()));
   }
+  
+  private static void undoSystemIn() {
+    System.setIn(oldIn);
+  }
 
   /**
    * Test that, even if automatic HA is enabled, the monitoring operations


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to