Merge pull request #11 from armfergom/JENKINS-34378

[JENKINS-34378] Validation for the test result filename field.
jenkinsci · May 5, 2016 · 1bd02f6 · 1bd02f6
2 parents e877ea9 + 7335ad6
commit 1bd02f6
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/MsTestBuilder.java b/src/main/java/org/jenkinsci/plugins/MsTestBuilder.java
@@ -15,6 +15,7 @@
 import hudson.tasks.BuildStepDescriptor;
 import hudson.tasks.Builder;
 import hudson.tools.ToolInstallation;
+import hudson.util.FormValidation;
 
 import java.io.File;
 import java.io.IOException;
@@ -25,7 +26,9 @@
 import java.util.Set;
 import java.util.StringTokenizer;
 
+import org.apache.commons.lang.StringUtils;
 import org.kohsuke.stapler.DataBoundConstructor;
+import org.kohsuke.stapler.QueryParameter;
 
 /**
  * @author Ido Ran
@@ -294,5 +297,12 @@ public void setInstallations(MsTestInstallation... antInstallations) {
         public MsTestInstallation.DescriptorImpl getToolDescriptor() {
             return ToolInstallation.all().get(MsTestInstallation.DescriptorImpl.class);
         }
+
+        public FormValidation doCheckResultFile(@QueryParameter String value) {
+            if (StringUtils.isBlank(value)) {
+                return FormValidation.warning("Result File Name must be provided.");
+            }
+            return FormValidation.ok();
+        }
     }
 }