YARN-9476. [YARN-9473] Create unit tests for VE plugin. Contributed by Peter Bacsko.

Author: tangzhankun

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/resourceplugin/com/nec/NECVEPlugin.java

@@ -74,7 +74,7 @@ public NECVEPlugin() throws ResourceHandlerException {
     if (envScriptName != null) {
       binaryName = envScriptName;
     }
-    LOG.info("Use {} as script name.", envScriptName);
+    LOG.info("Use {} as script name.", binaryName);
 
     // Try to find the script based on an environment variable, if set
     boolean found = false;
@@ -115,11 +115,13 @@ public NECVEPlugin() throws ResourceHandlerException {
     }
   }
 
+  @Override
   public DeviceRegisterRequest getRegisterRequestInfo() {
     return DeviceRegisterRequest.Builder.newInstance()
         .setResourceName("nec.com/ve").build();
   }
 
+  @Override
   public Set<Device> getDevices() {
     Set<Device> devices = null;
 
@@ -135,6 +137,7 @@ public Set<Device> getDevices() {
     return devices;
   }
 
+  @Override
   public DeviceRuntimeSpec onDevicesAllocated(Set<Device> set,
       YarnRuntimeType yarnRuntimeType) {
     return null;
@@ -151,6 +154,7 @@ private Set<Device> parseOutput(String output) {
 
     LOG.info("Parsing output: {}", output);
     String[] lines = output.split("\n");
+    outer:
     for (String line : lines) {
       Device.Builder builder = Device.Builder.newInstance();
 
@@ -163,7 +167,7 @@ private Set<Device> parseOutput(String output) {
         String[] tokens = keyValue.trim().split("=");
         if (tokens.length != 2) {
           LOG.error("Unknown format of script output! Skipping this line");
-          continue;
+          continue outer;
         }
 
         final String key = tokens[0];