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

Reply via email to