[SUREFIRE-2101] Phrased test names with missing @DisplayName result in a "null" test name

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.0.0-M7
• Fix Version/s: 3.0.0-M8
• Component/s: Maven Surefire Plugin
• Environment: All platforms

Language

Java

• Assignee: Slawomir Jaranowski
• Reporter: Ashley Scopes
• Created: 19/Jun/22 12:49
• Updated: 27/Dec/22 22:06
• Resolved: 27/Dec/22 22:06

Description

If I enable phrased test naming in config, and then write a test such as the following:

// Note no @DisplayName("...")
class SomeTest {
  void fooHasThreeCharacters() {
    assertThat(foo).hasSize(3);
  }
}

...then the console reports will contain a null name rather than something meaningful.

I am proposing that when this phrased name is unavailable, Surefire should fall back to the original test name that would have been provided if phrased naming was disabled. The reasoning behind this is that showing an unformatted class name is more useful to the developer reading Maven logs than just the phrase null.

Issue Links

• links to: GitHub Pull Request #549
• links to: GitHub Pull Request #587