• Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • gradle-jpi-plugin
    • None

      Maven plugin does this to check the syntax of Jelly and other things. This is good to have.

          [JENKINS-12193] Hook up injected tests

          https://github.com/jenkinsci/gradle-jpi-plugin/pull/42

          For now injected tests will be disabled by default because of JENKINS-21977, but that may change in the future.

          New options:

          ...
          jenkinsPlugin {
              ...
          
              disabledTestInjection = false     // disabled by default
              injectedTestName = 'InjectedTest' // that's the default
              requirePI = true                  // enabled by default
          
              ...
          }
          ...
          

          Daniel Spilker added a comment - https://github.com/jenkinsci/gradle-jpi-plugin/pull/42 For now injected tests will be disabled by default because of JENKINS-21977 , but that may change in the future. New options: ... jenkinsPlugin { ... disabledTestInjection = false // disabled by default injectedTestName = 'InjectedTest' // that's the default requirePI = true // enabled by default ... } ...

          Code changed in jenkins
          User: Daniel Spilker
          Path:
          CHANGELOG.md
          src/main/groovy/org/jenkinsci/gradle/plugins/jpi/JpiExtension.groovy
          src/main/groovy/org/jenkinsci/gradle/plugins/jpi/JpiPlugin.groovy
          src/main/groovy/org/jenkinsci/gradle/plugins/jpi/TestInsertionTask.groovy
          src/test/groovy/org/jenkinsci/gradle/plugins/jpi/JpiPluginSpec.groovy
          http://jenkins-ci.org/commit/gradle-jpi-plugin/fc956747d8ea5150984e0e86b811dd9113b77582
          Log:
          JENKINS-12193 hook up injected tests, but disabled by default because of JENKINS-21977

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Spilker Path: CHANGELOG.md src/main/groovy/org/jenkinsci/gradle/plugins/jpi/JpiExtension.groovy src/main/groovy/org/jenkinsci/gradle/plugins/jpi/JpiPlugin.groovy src/main/groovy/org/jenkinsci/gradle/plugins/jpi/TestInsertionTask.groovy src/test/groovy/org/jenkinsci/gradle/plugins/jpi/JpiPluginSpec.groovy http://jenkins-ci.org/commit/gradle-jpi-plugin/fc956747d8ea5150984e0e86b811dd9113b77582 Log: JENKINS-12193 hook up injected tests, but disabled by default because of JENKINS-21977

          Code changed in jenkins
          User: Daniel Spilker
          Path:
          CHANGELOG.md
          src/main/groovy/org/jenkinsci/gradle/plugins/jpi/JpiExtension.groovy
          src/main/groovy/org/jenkinsci/gradle/plugins/jpi/JpiPlugin.groovy
          src/main/groovy/org/jenkinsci/gradle/plugins/jpi/TestInsertionTask.groovy
          src/test/groovy/org/jenkinsci/gradle/plugins/jpi/JpiPluginSpec.groovy
          http://jenkins-ci.org/commit/gradle-jpi-plugin/0031ffab8800923fc38bdc123edc1742ee3438dc
          Log:
          Merge pull request #42 from daspilker/JENKINS-12193

          JENKINS-12193 Hook up injected tests

          Compare: https://github.com/jenkinsci/gradle-jpi-plugin/compare/9f6de01dd6ad...0031ffab8800

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Spilker Path: CHANGELOG.md src/main/groovy/org/jenkinsci/gradle/plugins/jpi/JpiExtension.groovy src/main/groovy/org/jenkinsci/gradle/plugins/jpi/JpiPlugin.groovy src/main/groovy/org/jenkinsci/gradle/plugins/jpi/TestInsertionTask.groovy src/test/groovy/org/jenkinsci/gradle/plugins/jpi/JpiPluginSpec.groovy http://jenkins-ci.org/commit/gradle-jpi-plugin/0031ffab8800923fc38bdc123edc1742ee3438dc Log: Merge pull request #42 from daspilker/ JENKINS-12193 JENKINS-12193 Hook up injected tests Compare: https://github.com/jenkinsci/gradle-jpi-plugin/compare/9f6de01dd6ad...0031ffab8800

          Will be released in 0.8.2.

          Daniel Spilker added a comment - Will be released in 0.8.2.

          Released in 0.9.0.

          Daniel Spilker added a comment - Released in 0.9.0.

            daspilker Daniel Spilker
            kohsuke Kohsuke Kawaguchi
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: