Ticket #2667 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

twisted.protocols.amp.Command should have methods for argument serialization and parsing

Reported by: radix Owned by: radix
Priority: normal Milestone:
Component: core Keywords:
Cc: Branch:
Author: Launchpad Bug:

Description

This will make it easier to write unit tests.

Change History

Changed 3 years ago by radix

  • status changed from new to assigned

See #2657 for where the work for this fix is happening.

Changed 3 years ago by radix

  • status changed from assigned to closed
  • resolution set to fixed

(In [20391]) Merge amp-factoring-2657+2667.

Author: radix Reviewer: therve Fixes #2657 Fixes #2667 Fixes #2656 Fixes #2664

It is now possible to use public methods on Command objects to serialize and parse arguments and responses based on that Commandn's arguments and response schemas. It was already possible to serialize responses via the Command in this way; analogs for the other three cases were added. (#2657 & #2667).

Some docstrings in nearby code were alse fixed in this branch. (#2656 & #2664).

Note: See TracTickets for help on using tickets.