On Mon, 11 Jul 2022 21:03:16 GMT, Weijun Wang <[email protected]> wrote:

>> test/jdk/javax/security/auth/login/modules/SafeLogout.java line 51:
>> 
>>> 49: 
>>> 50:     static void test(int pos) throws Exception {
>>> 51:         // Create random JAAS login config.
>> 
>> I'm probably missing something obvious, but can you explain why this test 
>> uses a random login config? I would add some comments explaining that more.
>
> I cannot find a way to test all combinations so I make it random. If it 
> really fails I make sure the login module name, flag, and whether there is a 
> login attempt are printed out so I can reproduce it.

Why isn't it sufficient to just call logout once per each login module?

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

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

Reply via email to