mirror of
https://github.com/trezor/trezor-firmware.git
synced 2024-12-18 04:18:10 +00:00
fix(python/debuglink): fix formatting for expected messages
This was broken by the previous commit removing f-strings. For this reason, `format()` was renamed to `to_string` to make it clearer that it's not the `str.format` method.
This commit is contained in:
parent
c38efaa9a5
commit
2a2da21879
@ -307,7 +307,7 @@ class MessageFilter:
|
|||||||
|
|
||||||
return True
|
return True
|
||||||
|
|
||||||
def format(self, maxwidth=80):
|
def to_string(self, maxwidth=80):
|
||||||
fields = []
|
fields = []
|
||||||
for field in self.message_type.FIELDS.values():
|
for field in self.message_type.FIELDS.values():
|
||||||
if field.name not in self.fields:
|
if field.name not in self.fields:
|
||||||
@ -316,7 +316,7 @@ class MessageFilter:
|
|||||||
if isinstance(value, IntEnum):
|
if isinstance(value, IntEnum):
|
||||||
field_str = value.name
|
field_str = value.name
|
||||||
elif isinstance(value, MessageFilter):
|
elif isinstance(value, MessageFilter):
|
||||||
field_str = value.format(maxwidth - 4)
|
field_str = value.to_string(maxwidth - 4)
|
||||||
elif isinstance(value, protobuf.MessageType):
|
elif isinstance(value, protobuf.MessageType):
|
||||||
field_str = protobuf.format_message(value)
|
field_str = protobuf.format_message(value)
|
||||||
else:
|
else:
|
||||||
@ -563,7 +563,7 @@ class TrezorClientDebugLink(TrezorClient):
|
|||||||
for i in range(start_at, stop_at):
|
for i in range(start_at, stop_at):
|
||||||
exp = expected[i]
|
exp = expected[i]
|
||||||
prefix = " " if i != current else ">>> "
|
prefix = " " if i != current else ">>> "
|
||||||
output.append(textwrap.indent(exp, prefix))
|
output.append(textwrap.indent(exp.to_string(), prefix))
|
||||||
if stop_at < len(expected):
|
if stop_at < len(expected):
|
||||||
omitted = len(expected) - stop_at
|
omitted = len(expected) - stop_at
|
||||||
output.append(f" (...{omitted} following responses omitted)")
|
output.append(f" (...{omitted} following responses omitted)")
|
||||||
|
Loading…
Reference in New Issue
Block a user