[jdk17] Integrated: 8268602: a couple runtime/os tests don't check exit code

Igor Ignatyev iignatyev at openjdk.java.net
Fri Jun 11 12:45:18 UTC 2021


On Fri, 11 Jun 2021 09:51:24 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

> Hi all,
> 
> could you please review this trivial and test-only patch that adds checks of exit code to two `runtime/os` tests?
> from JBS:
>> `runtime/os/TestUseCpuAllocPath.java` and `AvailableProcessors.java` spawn new JVMs but don't check their exit code which might lead to both type-I and type-II errors
> 
> testing: ``runtime/os` on `{linux,windows,macosx}-x64`
> 
> Thanks,
> -- Igor

This pull request has now been integrated.

Changeset: cce8da2c
Author:    Igor Ignatyev <iignatyev at openjdk.org>
URL:       https://git.openjdk.java.net/jdk17/commit/cce8da2c60812873b63de4e65bc992bc6d825fe1
Stats:     4 lines in 2 files changed: 2 ins; 0 del; 2 mod

8268602: a couple runtime/os tests don't check exit code

Reviewed-by: dholmes

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

PR: https://git.openjdk.java.net/jdk17/pull/25


More information about the hotspot-runtime-dev mailing list