ass messages to Junit4 assumptions

Author: stoty

hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSShell.java

@@ -599,9 +599,7 @@ public void checkPermission(Permission perm) {
         }
       });
     } catch (UnsupportedOperationException e) {
-      // Test is skipped because SecurityManager cannot be set (JEP411)
-      // TODO add message when migrating to Junit 5
-      assumeTrue(false);
+      assumeTrue("Test is skipped because SecurityManager cannot be set (JEP 411)", false);
     }
     show("copy local " + f1 + " to remote " + dst);
     dfs.copyFromLocalFile(false, false, new Path(f1.getPath()), dst);

hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDatanodeRegistration.java

@@ -83,9 +83,7 @@ public void testDNSLookups() throws Exception {
     try {
       System.setSecurityManager(sm);
     } catch (UnsupportedOperationException e) {
-      // Test is skipped because SecurityManager cannot be set (JEP411)
-      // TODO add message when migrating to Junit 5
-      assumeTrue(false);
+      assumeTrue("Test is skipped because SecurityManager cannot be set (JEP 411)", false);
     }
 
     MiniDFSCluster cluster = null;

hadoop-tools/hadoop-distcp/src/test/java/org/apache/hadoop/tools/TestExternalCall.java

@@ -70,8 +70,7 @@ public void setup() {
     try {
       System.setSecurityManager(new NoExitSecurityManager());
     } catch (UnsupportedOperationException e) {
-      assumeTrue(false,
-          "Test is skipped because SecurityManager cannot be set (JEP411)");
+      assumeTrue(false, "Test is skipped because SecurityManager cannot be set (JEP 411)");
     }
     try {
       fs = FileSystem.get(getConf());

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/resources/TestCGroupsHandlerImpl.java

@@ -375,8 +375,7 @@ private void testPreMountedControllerInitialization(String myHierarchy)
       try {
         System.setSecurityManager(new MockSecurityManagerDenyWrite());
       } catch (UnsupportedOperationException e) {
-        assumeTrue(false,
-            "Test is skipped because SecurityManager cannot be set (JEP411)");
+        assumeTrue(false, "Test is skipped because SecurityManager cannot be set (JEP 411)");
       }
       try {
         cGroupsHandler.initializeCGroupController(

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/TestCapacitySchedulerAsyncScheduling.java

@@ -1094,9 +1094,7 @@ public void testAsyncScheduleThreadExit() throws Exception {
     try {
       System.setSecurityManager(noExitSecurityManager);
     } catch (UnsupportedOperationException e) {
-      // Test is skipped because SecurityManager cannot be set (JEP411)
-      // TODO add message when migrating to Junit 5
-      Assume.assumeTrue(false);
+      Assume.assumeTrue("Test is skipped because SecurityManager cannot be set (JEP 411)", false);
     }
 
     // test async-scheduling thread exit