-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Description
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 char
/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.
However, a null
CharSequence
/Character
value would continue to be displayed as null
, not "null"
or 'null'
.