ListOf AMP Arguments should support 'optional' flag
|Reported by:||faldridge||Owned by:|
Like all other AMP Argument subclasses, t.p.amp.ListOf should allow callers to pass 'optional=True' to its init() method, thereby allowing the argument to be omitted from the protocol.
Attached is a patch that provides the correct behavior with the necessary update to ListOf's docstring.
As the functionality is handled by the base class (t.p.amp.Argument), and not individual subclasses, and as the functionality already has unit tests, there are no unit tests included in this patch.
Change History (6)
comment:1 follow-up: 2 Changed 7 years ago by
|Owner:||changed from Glyph to faldridge|
comment:2 Changed 7 years ago by
|Owner:||changed from faldridge to Jean-Paul Calderone|