On Tue, 14 Feb 2023 11:21:36 GMT, Lance Andersen <lan...@openjdk.org> wrote:

>> The message is already validated using `expectedExceptionsMessageRegExp` in 
>> the `@Test` annotation.
>> 
>> Would you prefer if I use expectThrows instead, or perhaps inline the 
>> `BAD_ENTRY_NAME_OR_COMMENT` constant as a literal?
>
> Sorry if this was not clear,  we have gone away from using the annotation 
> element `exepectedExceptions` for new and updated tests and  have tried to 
> standardize on `assertThrows` and  `expectThrows` instead which is the basis 
> for my suggestion.
> 
> 
> Thank you for your other updates.  I will go through them later today

No worries, this makes sense. Updated to expectThrows with assertEquals.

-------------

PR: https://git.openjdk.org/jdk/pull/12290

Reply via email to