search

jenkinsci / parallel-test-executor-plugin

Parallel test executor plugin
https://plugins.jenkins.io/parallel-test-executor/
50 stars 55 forks source link

Do not consider a build with unloadable `TestResult` #247

Closed jglick closed 1 year ago

jglick commented 1 year ago

When junitReport.xml is malformed, https://github.com/jenkinsci/junit-plugin/blob/b4cf28bc7724c49bc93560c7bf95acdca4bbde1b/src/main/java/hudson/tasks/junit/TestResultAction.java#L227-L240 can cause ParallelTestExecutor.getTestResult to nonetheless consider that build a valid baseline, but with zero results which of course causes split calculation to not work sanely. Ignore such builds, and continue searching for better ones, or at worst fall back to source scanning.

jglick commented 1 year ago

248 works as a hotfix. Ought to add a test case.

jglick commented 1 year ago

(motivation: https://github.com/jenkinsci/jenkins/pull/7875 / https://github.com/jenkinsci/junit-plugin/pull/520)