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

Add support for maven-android-plugin integration test reports

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Blocker Blocker
    • maven-plugin
    • None
    • Jenkins 1.428 on Ubuntu 10.04 with Windows and Linux slaves performing Android builds.

      A recent addition to the maven-android-plugin generates junit test output xml files. Currently the jenkins maven plugin does not recognise these and hence does not report the test results along with other junit test results.

      SurefireArchiver.java in maven-plugin needs to recognise the
      com.jayway.maven.plugins.android.generation2:maven-android-plugin/internal-integration-test mojo as producing test results.

          [JENKINS-10913] Add support for maven-android-plugin integration test reports

          Richard Mortimer created issue -

          See pull request https://github.com/jenkinsci/jenkins/pull/230

          This makes Jenkins parse the junit compatible test result output produced by maven-android-plugin version 3.0.0-alpha-6 and later
          See
          http://code.google.com/p/maven-android-plugin/wiki/Changelog
          http://www.simpligility.com/2011/09/easier-android-test-reporting-with-maven-and-hudson/

          Richard Mortimer added a comment - See pull request https://github.com/jenkinsci/jenkins/pull/230 This makes Jenkins parse the junit compatible test result output produced by maven-android-plugin version 3.0.0-alpha-6 and later See http://code.google.com/p/maven-android-plugin/wiki/Changelog http://www.simpligility.com/2011/09/easier-android-test-reporting-with-maven-and-hudson/

          Code changed in jenkins
          User: Richard Mortimer
          Path:
          maven-plugin/src/main/java/hudson/maven/reporters/SurefireArchiver.java
          http://jenkins-ci.org/commit/jenkins/5668a06f37d3392af8232ceccdd83a9e7f4ec041
          Log:
          FIXES JENKINS-10913 Add support for maven-android-plugin integration test reports.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Richard Mortimer Path: maven-plugin/src/main/java/hudson/maven/reporters/SurefireArchiver.java http://jenkins-ci.org/commit/jenkins/5668a06f37d3392af8232ceccdd83a9e7f4ec041 Log: FIXES JENKINS-10913 Add support for maven-android-plugin integration test reports.

          Code changed in jenkins
          User: Olivier Lamy
          Path:
          maven-plugin/src/main/java/hudson/maven/reporters/SurefireArchiver.java
          http://jenkins-ci.org/commit/jenkins/62cbe7b3377819b5a7a60388f2307a6027c09201
          Log:
          Merge pull request #230 from oldelvet/jenkins-10913

          FIXES JENKINS-10913 Add support for maven-android-plugin integration test

          Compare: https://github.com/jenkinsci/jenkins/compare/9349370...62cbe7b

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Olivier Lamy Path: maven-plugin/src/main/java/hudson/maven/reporters/SurefireArchiver.java http://jenkins-ci.org/commit/jenkins/62cbe7b3377819b5a7a60388f2307a6027c09201 Log: Merge pull request #230 from oldelvet/jenkins-10913 FIXES JENKINS-10913 Add support for maven-android-plugin integration test Compare: https://github.com/jenkinsci/jenkins/compare/9349370...62cbe7b

          dogfood added a comment -

          dogfood added a comment - Integrated in jenkins_main_trunk #1112

          Wiebe Elsinga added a comment -

          Change made it impossible to use maven-android-plugin below 3.0.0

          message Jenkins:
          Cannot find setter nor field in com.jayway.maven.plugins.android.phase12integrationtest.InternalIntegrationTestMojo for 'skipTests'

          Wiebe Elsinga added a comment - Change made it impossible to use maven-android-plugin below 3.0.0 message Jenkins: Cannot find setter nor field in com.jayway.maven.plugins.android.phase12integrationtest.InternalIntegrationTestMojo for 'skipTests'
          Wiebe Elsinga made changes -
          Priority Original: Minor [ 4 ] New: Blocker [ 1 ]

          I've started to create a patch that corrects this issue. I'll make it only accept 3.0.0-alpha-6 and later plugins. Also make it accept the new name android-maven-plugin that is in the pipeline. Initial testing is ok. Will issue a pull request later today.

          I don't use 2.x series plugins any more so if you can test a pre-build that would be appreciated.

          Richard Mortimer added a comment - I've started to create a patch that corrects this issue. I'll make it only accept 3.0.0-alpha-6 and later plugins. Also make it accept the new name android-maven-plugin that is in the pipeline. Initial testing is ok. Will issue a pull request later today. I don't use 2.x series plugins any more so if you can test a pre-build that would be appreciated.

          Wiebe Elsinga added a comment -

          No problem

          Wiebe Elsinga added a comment - No problem

          Code changed in jenkins
          User: Richard Mortimer
          Path:
          changelog.html
          maven-plugin/src/main/java/hudson/maven/reporters/SurefireArchiver.java
          http://jenkins-ci.org/commit/jenkins/952f19fbfac8c5bef9ffbb345f33e4f1f8978b32
          Log:
          [FIXED JENKINS-10913] Add support for maven-android-plugin integration test

          Earlier versions of the plugin (2.x and prior to 3.0.0-alpha6) do not produce
          test reports and also do not accept the same configuration properties. Ignore
          those earlier versions.
          Also accept the new plugin name android-maven-plugin (changed to fit with
          Maven plugin naming guidelines).

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Richard Mortimer Path: changelog.html maven-plugin/src/main/java/hudson/maven/reporters/SurefireArchiver.java http://jenkins-ci.org/commit/jenkins/952f19fbfac8c5bef9ffbb345f33e4f1f8978b32 Log: [FIXED JENKINS-10913] Add support for maven-android-plugin integration test Earlier versions of the plugin (2.x and prior to 3.0.0-alpha6) do not produce test reports and also do not accept the same configuration properties. Ignore those earlier versions. Also accept the new plugin name android-maven-plugin (changed to fit with Maven plugin naming guidelines).
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

            oldelvet Richard Mortimer
            oldelvet Richard Mortimer
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: