On Fri, 19 Aug 2022 21:47:00 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> Jayashree Huttanagoudar has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Address review comments
>
> test/jdk/javax/security/auth/login/LoginContext/OptionalJaas/UnixNTPlatform.java
> line 85:
>
>> 83: throw new RuntimeException();
>> 84: } else {
>> 85: System.out.printf("-- call stack is -- %n%s%n", s);
>
> I'd rather print out the call stack if the test fails. Who would want to read
> the test output if it succeeds?
Aim of this patch is to print the call stack trace right ? And if the test case
passes that means we have to print the call stack trace.
And if the test case fails are suggesting to print the exception we are
throwing (i.e RuntimeException) ?
-------------
PR: https://git.openjdk.org/jdk/pull/9159
