wboayue / rust-ibapi / 26429821420

92%

Build Type

push

github

Committed by web-flow

Commit Message

accounts: decode_server_time(_millis) proto-only at floor 213 (#635)

* accounts: decode_server_time(_millis) proto-only at floor 213

Gate is PROTOBUF_REST_MESSAGES_3 (213), the connection floor since #632.
Flip decode_server_time and decode_server_time_millis wrappers to
require_proto() over &ResponseMessage; extract decode_server_time_proto /
decode_server_time_millis_proto helpers (matches the file's existing
shape). Sync/async test_server_time + test_server_time_millis migrate
from current_time().encode_pipe() text frames to
proto_response(IncomingMessages::CurrentTime{,InMillis}, encode_proto()).
One-shot callsites wrap the decoder in a closure since
one_shot_with_retry processor signature is still Fn(&mut ResponseMessage).

Delete test_server_time_comprehensive (sync) + test_server_time_scenarios
(async) + ServerTimeTestCase + server_time_test_cases(): the invalid /
overflow timestamp scenarios tested text-decode error paths that cannot
exist under Option<i64> proto wire. test_server_time keeps the success +
no-response coverage; new test_decode_server_time(_millis)_proto_via_builder
and _rejects_text_framing tests in decoders/tests.rs cover the new
production surface.

Mark ResponseMessage::next_long #[allow(dead_code)] consistent with
next_optional_int: helper has no production callers after this PR but
stays test-exercised pending PR-D2/D3 cursor-primitive cleanup.

Per-domain count accounts 3/11/3 -> 1/11/1 (decode_managed_accounts
remains dual-format for PR-C3).

* accounts: drop vestigial Scenario 1/2 comments in test_server_time

Coverage Stats

18 of 19 new or added lines in 3 files covered. (94.74%)

5 existing lines in 3 files now uncovered.

19558 of 21184 relevant lines covered (92.32%)

27.22 hits per line