wboayue / rust-ibapi / 26325939103

92%

push

github

accounts: proto-only decoders at floor 210 (#623)

* accounts: proto-only decoders at floor 210

Delete text branches in 9 accounts decoders whose originating outgoing-request
gate is ≤ 210 (floor): position, position_multi, account_summary, account_value,
account_portfolio_value, account_multi_value (gate 207); pnl, pnl_single (gate
210). Add decode_account_update_time_proto and rewire decode_account_update_time
(gate 207) which was previously text-only without decode_proto_or_text — broken
under proto framing per rule 15.

Drop server_version plumbing through decode_account_update_message and
dispatch_unsolicited_message. Migrate decoder-side, stream-decoder, sync, async,
and connection tests from text fixtures to proto_response + builder.encode_proto.

Stays dual-format (gate > 210, deletion blocked by floor):
- decode_managed_accounts (StartApi handshake, gate 213)
- decode_family_codes (gate 212)
- decode_server_time / decode_server_time_millis (gate 213)

Tracker (plans/legacy-text-protocol-cleanup.md) updated with the partial
migration and the 4 stays-dual decoders.

* accounts: builder for PortfolioValue/AccountUpdateTime, drop ACCOUNT_VALUE

/simplify: replace inline proto::PortfolioValue and proto::AccountUpdateTime
construction in stream_decoders tests with builders in
src/testdata/builders/accounts.rs, mirroring the per-message-type builder
convention already used for AccountValue/AccountSummary/PnL/etc.

Drop unused testdata::responses::ACCOUNT_VALUE constant; remove the
unnecessary #[allow(dead_code)] from MANAGED_ACCOUNT (still used by
transport::recorder.
EOF
)

98 of 123 new or added lines in 4 files covered. (79.67%)

19140 of 20763 relevant lines covered (92.18%)

28.18 hits per line