YARN-2899. Run TestDockerContainerExecutorWithMocks on Linux only. Co…

…ntributed by Ming Ma.
apache · Feb 16, 2015 · a3fc81f · a3fc81f
1 parent b6fc5b8
commit a3fc81f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/hadoop-yarn-project/CHANGES.txt b/hadoop-yarn-project/CHANGES.txt
@@ -596,6 +596,9 @@ Release 2.7.0 - UNRELEASED
     YARN-2847. Linux native container executor segfaults if default banned
     user detected (Olaf Flebbe via jlowe)
 
+    YARN-2899. Run TestDockerContainerExecutorWithMocks on Linux only.
+    (Ming Ma via cnauroth)
+
 Release 2.6.0 - 2014-11-18
 
   INCOMPATIBLE CHANGES

diff --git a/.../java/org/apache/hadoop/yarn/server/nodemanager/TestDockerContainerExecutorWithMocks.java b/.../java/org/apache/hadoop/yarn/server/nodemanager/TestDockerContainerExecutorWithMocks.java
@@ -69,7 +69,7 @@ public class TestDockerContainerExecutorWithMocks {
 
   @Before
   public void setup() {
-    assumeTrue(!Path.WINDOWS);
+    assumeTrue(Shell.LINUX);
     File f = new File("./src/test/resources/mock-container-executor");
     if(!FileUtil.canExecute(f)) {
       FileUtil.setExecutable(f, true);
@@ -101,7 +101,9 @@ public void setup() {
   @After
   public void tearDown() {
     try {
-      lfs.delete(workDir, true);
+      if (lfs != null) {
+        lfs.delete(workDir, true);
+      }
     } catch (IOException e) {
       throw new RuntimeException(e);
     }