Quote text-based arguments in display names for parameterized tests

[sbrannen]

Overview

While discussing #4714 with @sormuras, he inquired about the 'null' display name, which sparked an idea...

Namely, it would be quite nice to have an auto-quote feature for argument values in the display name for a @ParameterizedClass or @ParameterizedTest.

I therefore propose that we introduce a boolean quoteText() default true attribute (or similar) in @ParameterizedClass and @ParameterizedTest which signals to the ParameterizedInvocationNameFormatter that it should wrap CharSequence values (naturally including String) in double quotes ("") and wrap Character values in single quotes (''`).

Defaulting to true would make this an opt-out feature, but we could of course default to false to make it an opt-in feature.

Though, since we are introducing this in 6.0, perhaps it's better to make it an opt-out feature.

Note, however, that a null CharSequence/Character value would continue to be displayed as null, not "null" or 'null'.

Related Issues

• Replace ISO control characters in display names #4714
• CSV header names do not work with {argumentsWithNames} display name pattern #4783
• Consistently use name = value formatting for arguments with names in parameterized tests #4786

[sbrannen]

As discussed in #4714 (comment), the quoteText feature would result in automatic escaping of special characters within "" and '' quoted values.