On Tue, 16 Aug 2022 13:49:34 GMT, Jayashree Huttanagoudar <d...@openjdk.org> 
wrote:

>> Could you please review the changes?
>> This patch is to address : 
>> https://bugs.openjdk.org/browse/JDK-8215916?jql=labels%20%3D%20starter-bug
>
> Jayashree Huttanagoudar has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Address review comment for cross-platform in the test case

The original "cross-platform" case is exactly what this bug is about. The 
overall login succeeds but the reason of the optional login module failure is 
not logged in the debug output. You need to update the source code of 
`LoginContext.java` to print out the reason and then confirm it in the test by 
examining the content of `new String(byes)`.

Until now, you still only have a test change in your commits. This is not a 
test-only bug.

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

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

Reply via email to