Skip to content

Protocol-Test: Remove unnecessary MAX_INV Limit to MAX_MESSAGE test / Or replace. #137

@jjgg22zh

Description

@jjgg22zh

The following Test found in
https://github.com/bcoin-org/bcash/blob/master/test/protocol-test.js#L151

The GetDataPacket size values to 1800003
Whilst the current updated MAX_MESSAGE value is 2097152

There is a possibility of changing the typeof invitems, or simply removing the test I'd be willing to hear out what is the better option. So that I can replace or remove the check in the test file for ci purposes.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions