You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by ma...@apache.org on 2020/08/02 18:08:08 UTC

[logging-pipelines] branch master updated: Use warnings-ng for JUnit issues recording

This is an automated email from the ASF dual-hosted git repository.

mattsicker pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/logging-pipelines.git


The following commit(s) were added to refs/heads/master by this push:
     new 5d8dec4  Use warnings-ng for JUnit issues recording
5d8dec4 is described below

commit 5d8dec42442abf87eaf5abd6a2d7dadbe84bf2e8
Author: Matt Sicker <bo...@gmail.com>
AuthorDate: Sun Aug 2 13:07:28 2020 -0500

    Use warnings-ng for JUnit issues recording
    
    Also splits Linux and Windows test runs into separate test lists for
    easier debugging.
---
 log4j/Jenkinsfile | 34 ++++++++++++++--------------------
 1 file changed, 14 insertions(+), 20 deletions(-)

diff --git a/log4j/Jenkinsfile b/log4j/Jenkinsfile
index 65cd554..549f39c 100644
--- a/log4j/Jenkinsfile
+++ b/log4j/Jenkinsfile
@@ -59,20 +59,17 @@ pipeline {
                 stage('Ubuntu') {
                     steps {
                         mvn '-Dmaven.test.failure.ignore=true verify'
-                    }
-                    post {
-                        always {
-                            // record linux run of tests
-                            junit '**/*-reports/*.xml'
-                            // additional warnings generated during build
-                            // TODO: would be nice to be able to include checkstyle, cpd, pmd, and spotbugs,
-                            //       but current site build takes too long
-                            recordIssues enabledForFailure: true,
-                                    sourceCodeEncoding: 'UTF-8',
-                                    referenceJobName: 'log4j/master',
-                                    tools: [mavenConsole(), errorProne(), java(),
-                                            taskScanner(highTags: 'FIXME', normalTags: 'TODO', includePattern: '**/*.java', excludePattern: '*/target/**')]
-                        }
+                        // TODO: would be nice to be able to include checkstyle, cpd, pmd, and spotbugs,
+                        //       but current site build takes too long
+                        recordIssues sourceCodeEncoding: 'UTF-8',
+                                referenceJobName: 'log4j/master',
+                                tools: [
+                                    junitParser(id: 'LinuxTests', pattern: '**/target/*-reports/TEST-*.xml', reportEncoding: 'UTF-8'),
+                                    mavenConsole(),
+                                    errorProne(),
+                                    java(),
+                                    taskScanner(highTags: 'FIXME', normalTags: 'TODO', includePattern: '**/*.java', excludePattern: '*/target/**')
+                                ]
                     }
                 }
                 stage('Windows') {
@@ -85,12 +82,9 @@ pipeline {
                         // also note that we don't need to use the jenkins system property here as it's ubuntu-specific
                         bat 'if exist %userprofile%\\.embedmongo\\ rd /s /q %userprofile%\\.embedmongo'
                         mvn '-Dmaven.test.failure.ignore=true verify'
-                    }
-                    post {
-                        always {
-                            // record windows run of tests
-                            junit '**/*-reports/*.xml'
-                        }
+                        recordIssues sourceCodeEncoding: 'UTF-8',
+                                referenceJobName: 'log4j/master',
+                                tools: [junitParser(id: 'WindowsTests', pattern: '**/target/*-reports/TEST-*.xml', reportEncoding: 'UTF-8')]
                     }
                 }
             }