Code review comment for lp:~allenap/maas/database-locks-revisited

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Thanks for fixing this. One of those little things that kept us awake at night.

Two small things: in the test names, "crashes" sounds overly unintentional. Why not "raises" or "fails"? Or even just "checks that"?

review: Approve

« Back to merge proposal