test_framework: improve the syntax of serialize() methods
ClosedPublic
Actions

Authored by PiRK on Thu, Mar 20, 08:27.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCefe279d43307: test_framework: improve the syntax of serialize() methods

Summary

These changes do not affect behavior.

Don't unnecessarily initialize an empty bytes object before appending to it.
Don't use an intermediate variable before returning in trivial cases.
Add a few typehints when they are not obvious to the reader (e.g. when *string* actually means *bytes* because of python2 legacy)

Test Plan

ninja check-functional-extended

Diff Detail

Repository

rABC Bitcoin ABC

Branch

messages_improvements

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 32783
Build 65054: Build Diff	build-without-wallet · build-clang · build-debug · build-diff · build-clang-tidy
Build 65053: arc lint + arc unit

Event Timeline

PiRK created this revision.Thu, Mar 20, 08:27

Herald added a reviewer: Restricted Project. · View Herald TranscriptThu, Mar 20, 08:27

PiRK requested review of this revision.Thu, Mar 20, 08:27

PiRK planned changes to this revision.Thu, Mar 20, 08:30

back-off unrelated changes

This whole diff is a test on how Grok v3 can helpful to do semi-automated changes on a large amount of code.
It worked quite well, I just had to revert a few changes msg_avaproofs.serialize and msg_avaproofsreq.serialize, which imo were not trivial enough to justify removing the intermadiate var and all the comments to make it a one-liner.

Some unrelated blank line changes (annoying) and indentation changes (fixed by the linters).

PiRK added inline comments.Thu, Mar 20, 08:40

test/functional/test_framework/messages.py

241

this is manually changed. Grok's solution was worse imo

def serialize(self, *, with_time=True) -> bytes:
    """Serialize in addrv1 format (pre-BIP155)"""
    assert self.net == self.NET_IPV4
    if with_time:
        # VERSION messages serialize CAddress objects without time
        return (
            struct.pack("<I", self.time)
            + struct.pack("<Q", self.nServices)
            + b"\x00" * 10
            + b"\xff" * 2
            + socket.inet_aton(self.ip)
            + struct.pack(">H", self.port)
        )
    return (
        struct.pack("<Q", self.nServices)
        + b"\x00" * 10
        + b"\xff" * 2
        + socket.inet_aton(self.ip)
        + struct.pack(">H", self.port)
    )

Harbormaster completed remote builds in B32782: Diff 53197.Thu, Mar 20, 08:42

Harbormaster completed remote builds in B32783: Diff 53198.Thu, Mar 20, 08:45

Fabien accepted this revision.Thu, Mar 20, 09:42

This revision is now accepted and ready to land.Thu, Mar 20, 09:42

Closed by commit rABCefe279d43307: test_framework: improve the syntax of serialize() methods (authored by PiRK). · Explain WhyThu, Mar 20, 11:32

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABCefe279d43307: test_framework: improve the syntax of serialize() methods.