On Wed, 2022-06-29 at 11:13 +0200, Marc Haber wrote: > On Tue, Jun 28, 2022 at 01:34:23PM -0400, Matt Barry wrote: > > Good call. I've attached a minimal POC, which works in manual > > testing, > > but automated testing is tricky and there may be edge cases. (This > > test is blocking; NB (erroring out) might be easier to test, but > > which > > is the better UX?) > > A minimal test would be the test taking the lock, calling out to > adduser > and seeing whether it fails. The other side (testing whether adduser > properly takes the lock) is probably harder.
I was thinking the same thing; this too is easier to test if adduser's behavior is non-blocking (ie. fail instead of wait). Is that the desired response? mb
