On Fri, 21 Mar 2025 09:05:27 GMT, KIRIYAMA Takuya <d...@openjdk.org> wrote:
>> The current test program for the logging feature added in JDK-8301627 does >> not fully check some important cases. >> >> Issue Details: >> The test does not properly check cases where logging might not happen due to >> different logging levels. (e.g. ALL, TRACE, WARNING, etc.) >> The check for the logged stack trace is not enough, as it does not confirm >> enough details in the output. >> >> Fix Details: >> Added more test cases to check behavior under different logging levels. >> Improved the stack trace check by verifying more details in the logged >> output. >> These changes make the test more complete and ensure that the logging >> feature works as expected. >> Also, any existing test cases prior to this pull request are retained. >> >> The test was verified in the following OS environments, and it passed >> successfully in both environments. >> - Windows Server 2022 Standard 21H2 >> - Red Hat Enterprise Linux release 9.2 (Plow) >> >> Could you please review this fix? > > KIRIYAMA Takuya has updated the pull request incrementally with one > additional commit since the last revision: > > 8352016: Improve java/lang/RuntimeTests/RuntimeExitLogTest.java test/jdk/java/lang/RuntimeTests/RuntimeExitLogTest.java line 2: > 1: /* > 2: * Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved. Hello @tkiriyama, this should be `2023, 2025, `. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24050#discussion_r2014552843