566 + // If there is only one user, we start authenticating with that one here.
567 + // If there are more users, the Greeter will handle that
568 + if (LightDM.Users.count == 1) {
569 + LightDM.Greeter.authenticate(LightDM.Users.data(0, LightDM.UserRoles.NameRole))
570 + }
This feels a bit mismatched. I don't like the different code paths, though I get why you did it. No change requested, just noting.
As for the testing changes... I love the new plugins. But why add the has-pin user to the full/ plugin? I'd assume the single-pin/ plugin was the choice for testing the PIN screen. Similarly, the passphrase screen isn't tested via the single-passphrase/ plugin.
Looks mostly fine to me.
566 + // If there is only one user, we start authenticating with that one here. Users.count == 1) { Greeter. authenticate( LightDM. Users.data( 0, LightDM. UserRoles. NameRole) )
567 + // If there are more users, the Greeter will handle that
568 + if (LightDM.
569 + LightDM.
570 + }
This feels a bit mismatched. I don't like the different code paths, though I get why you did it. No change requested, just noting.
As for the testing changes... I love the new plugins. But why add the has-pin user to the full/ plugin? I'd assume the single-pin/ plugin was the choice for testing the PIN screen. Similarly, the passphrase screen isn't tested via the single-passphrase/ plugin.