wboayue / rust-ibapi / 25269131536

92%

push

github

fix(protocol): swap arg order in check_version error (#494)

Error::ServerVersion is documented as (required, actual, feature) and
its Display reads "server version {0} required, got {1}: {2}". The
construction in check_version passed (server_version, feature.min_version)
— wrong order. The user-visible message via error_handler::error_message
read "Server version 50 required for tick-by-tick, but connected to
version 137" — backwards.

Swap the args. Update the test to match the convention and add a
Display-format assertion that pins the rendered string.

5 of 5 new or added lines in 1 file covered. (100.0%)

17126 of 19327 relevant lines covered (88.61%)

24.92 hits per line