Warn when surefire naming strategy is used

[mpkorstanje]

⚡️ What's your motivation?

With Surefire 3.5.4 the heuristic for naming tests changed[1]. As a result the surefire strategy no longer works as expected. But the long strategy now does provide reasonable results.

On Surefire <= 3.5.2 with the surefire strategy the test results would be rendered as:

<testcase name="a few cukes" classname="Belly" time="0.062">

On Surefire >= 3.5.4 with the long strategy the test results will be rendered as:

<testcase name="Belly - a few cukes" classname="RunCucumberTest" time="0.06">

1. [SUREFIRE-2298] fix xml output with junit 5 nested classes (fix integration with Cucumber and Archunit) apache/maven-surefire#828

Closes: #2984

🏷️ What kind of change is this?

• ⚡ New feature (non-breaking change which adds new behaviour)

♻️ Anything particular you want feedback on?

There is no way to detect which version of Surefire is being used. So recommending an upgrade of Surefire seems to be the only sensible thing to do.

📋 Checklist:

• Wait for Surefire 3.5.4 to be released.
• Upgrade to Surefire 3.5.4
• I agree to respect and uphold the Cucumber Community Code of Conduct
• I've changed the behaviour of the code
  • I have added/updated tests to cover my changes.
• My change requires a change to the documentation.
  • I have updated the documentation accordingly.
• Users should know about my change
  • I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.