Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-34378

Would be convenient to have validation in the step configuration

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      The following fields are mandatory so it would be convenient to have the plugin warn us if they are blank:

      • Test files
      • Result File Name

      Just a minor and easy improvement that would speed up plugin configuration (not having to look at logs when values are accidentally left blank)

        Attachments

          Issue Links

            Activity

            Hide
            armfergom Armando Fernandez added a comment - - edited

            Test Files is an expandabletextbox that does not support form validation at the moment. See JENKINS-26407. So, only validation for result file name will be added.

            Show
            armfergom Armando Fernandez added a comment - - edited Test Files is an expandabletextbox that does not support form validation at the moment. See JENKINS-26407 . So, only validation for result file name will be added.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Armando Fernandez
            Path:
            src/main/java/org/jenkinsci/plugins/MsTestBuilder.java
            http://jenkins-ci.org/commit/mstestrunner-plugin/7335ad6368d089838cd8890a236e6dfac2c1e8dd
            Log:
            JENKINS-34378 Validation for the test result filename field.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Armando Fernandez Path: src/main/java/org/jenkinsci/plugins/MsTestBuilder.java http://jenkins-ci.org/commit/mstestrunner-plugin/7335ad6368d089838cd8890a236e6dfac2c1e8dd Log: JENKINS-34378 Validation for the test result filename field.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Ido Ran
            Path:
            src/main/java/org/jenkinsci/plugins/MsTestBuilder.java
            http://jenkins-ci.org/commit/mstestrunner-plugin/1bd02f687c2d7992c154981d7d9b5ea424c1bace
            Log:
            Merge pull request #11 from armfergom/JENKINS-34378

            JENKINS-34378 Validation for the test result filename field.

            Compare: https://github.com/jenkinsci/mstestrunner-plugin/compare/e877ea9d93a2...1bd02f687c2d

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ido Ran Path: src/main/java/org/jenkinsci/plugins/MsTestBuilder.java http://jenkins-ci.org/commit/mstestrunner-plugin/1bd02f687c2d7992c154981d7d9b5ea424c1bace Log: Merge pull request #11 from armfergom/ JENKINS-34378 JENKINS-34378 Validation for the test result filename field. Compare: https://github.com/jenkinsci/mstestrunner-plugin/compare/e877ea9d93a2...1bd02f687c2d

              People

              Assignee:
              armfergom Armando Fernandez
              Reporter:
              armfergom Armando Fernandez
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: