[SUREFIRE-1293] Simplify org.apache.maven.plugin.surefire.report.TestSetRunListener by using the null object pattern

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.20
• Component/s: Maven Surefire Plugin
• Labels: github
• External issue URL: https://github.com/apache/maven-surefire/pull/127
• Assignee: Tibor Digana
• Reporter: Benedikt Ritter
• Created: 09/Oct/16 15:11
• Updated: 13/Oct/16 20:32
• Resolved: 12/Oct/16 23:01

Description

The class org.apache.maven.plugin.surefire.report.TestSetRunListener has a lot of checks like this:

if( field != null )
{
    // do something with field
}

This can be simplified by providing fallback implementations for the fields being used by TestSetRunListener.

Issue Links

• links to: GitHub Pull Request #127