On Fri, 13 Jan 2023 06:44:05 GMT, Tejesh R <[email protected]> wrote: >> test/jdk/javax/swing/JFileChooser/CustomApproveButtonTest.java line 83: >> >>> 81: } >>> 82: if (customApproveButton.getText() == null) { >>> 83: fail("Custom Approve Button Text is null in >>> FileChooser!"); >> >> Suggestion: >> >> fail("Approve button text is null in FileChooser!"); > > Updated.
I wonder why do you like title-style capitalisation in exception messages. “Approve Button Text” is appropriate if it's a title of something. Since it's a message, “Approve button text” should be used instead. ------------- PR: https://git.openjdk.org/jdk/pull/11901
