On Wed, 2 Jul 2025 11:00:44 GMT, Manuel Hässig <mhaes...@openjdk.org> wrote:

> When a check in `CommandLineOptionTest` fails, the `AssertionError` message 
> contains the expected value, but not the observed value. To reduce the amount 
> of digging in the logs we have to do when analyzing a failure, this PR adds 
> the observed value to the error messages. So instead of 
> 
> 
> java.lang.AssertionError: Option 'CICompilerCount' is expected to have '12' 
> value
> 
> 
> a mismatch in the `CICompilerCount` will now print
> 
> 
> java.lang.AssertionError: Option 'CICompilerCount' is expected to have '12' 
> value, but is 'CICompilerCount = 6'.
> 
> 
> Testing:
>  - [x] Github Actions
>  - [x] tier1 through tier3 plus Oracle internal testing

This pull request has now been integrated.

Changeset: f153e415
Author:    Manuel Hässig <mhaes...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/f153e415d740f4ede272929171e9bb3e73ddbe1c
Stats:     7 lines in 1 file changed: 3 ins; 0 del; 4 mod

8361253: CommandLineOptionTest library should report observed values on failure

Reviewed-by: dholmes, shade

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

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

Reply via email to