On Wed, 6 May 2026 09:40:07 -0700 Breno Leitao wrote: > > @@ -453,7 +471,10 @@ from lib.py import cmd > > basic_groups, > > qgroups, > > delegation, > > - queue_update], args=(cfg, NetshaperFamily())) > > + queue_update, > > + dup_leaves > > + ], > > Nit: dup_leaves], on a single line matches the surrounding style and > avoids the dangling bracket on its own line. Not worth a respin on its > own.
It was intentional to avoid having to touch the lines when adding more tests. Now that I think about it again I should probably keep the tests sorted and add a trailing ,
