On Fri, 18 Oct 2024 00:38:06 GMT, Bradford Wetmore <wetm...@openjdk.org> wrote:
>> Artur Barashev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Adjust line length > > test/jdk/javax/net/ssl/TLSv13/SSLEngineNoServerHelloClientShutdown.java line > 1: > >> 1: /* > > New tests which exercise existing implementation functionality belong in the > implementation test directories (i.e. `sun/security/ssl`), not the API > directories (i.e. `javax/net/ssl`). If the code changes were mainly in the > `javax.net.ssl` code, then they would live here. > > Suggest you move these two new tests to: > > - `open/test/jdk/sun/security/ssl/InputRecord` Thanks, I'll move the tests! ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21043#discussion_r1806488269
