RFR: 8352016: Improve java/lang/RuntimeTests/RuntimeExitLogTest.java [v3]
Takuya Kiriyama
tkiriyama at openjdk.org
Thu Jul 3 06:49:49 UTC 2025
On Fri, 4 Apr 2025 10:29:43 GMT, Takuya Kiriyama <tkiriyama at 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?
>
> Takuya Kiriyama has updated the pull request incrementally with one additional commit since the last revision:
>
> 8352016: Improve java/lang/RuntimeTests/RuntimeExitLogTest.java
Thanks for the review!
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24050#issuecomment-3031069608
More information about the core-libs-dev
mailing list