Integrated: 8291519: jdk/jfr/api/event/TestShouldCommit.java failed with Unexpected value of shouldCommit()

[Erik Gahlin]

On Fri, 7 Oct 2022 17:41:10 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

> Hi,
> 
> Could I have review of a test fix, which is a complete rewrite. 
> 
> Instead of verifying the clock, it checks the logic of the shouldCommit method. 
> 
> Testing: 100 * test/jdk/jdk/jfr/api/event/TestShouldCommit.java
> 
> Maybe there should be a test that checks that an event timestamp doesn't drift too much from the system clock, but I think there are better ways to check that than using the shouldCommit() method. 
> 
> Thanks
> Erik

This pull request has now been integrated.

Changeset: fba763f8
Author:    Erik Gahlin <egahlin at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/fba763f82528d2825831a26b4ae4e090c602208f
Stats:     178 lines in 1 file changed: 54 ins; 53 del; 71 mod

8291519: jdk/jfr/api/event/TestShouldCommit.java failed with Unexpected value of shouldCommit()

Reviewed-by: mgronlun

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

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