wboayue / rust-ibapi / 25290671414

92%

Build Type

push

github

Committed by web-flow

Commit Message

test(market-data): builders + body-verifying test migration (#500)

* test(market-data): builders + body-verifying test migration

Mirrors PRs #496-#499. Adds typed request builders for every
market-data encoder (historical: head_timestamp, historical_data,
historical_ticks, histogram_data + 4 cancels; realtime: realtime_bars,
tick_by_tick, market_depth + body-bearing cancel,
market_depth_exchanges, market_data + 4 cancels). Migrates
sync/async tests in both submodules to assert_request<B> for body
verification, and drops 17 inline encoder self-loop tests now
redundant with the migrated paths (kept test_encode_interval --
exercises the OffsetDateTime: ToField impl).

Response builders deferred -- market-data responses are text wire
only and existing inline literals already exercise the production
decoders end-to-end.

* test(market-data): drop unused builders surfaced by /simplify

Per feedback_no_speculative_test_infra: 8 cancel builders shipped with
no test consumer. Drop them. Production cancel paths fire from
subscription drop handlers and do not need typed builders until a test
needs to verify their bodies.

Also drop two dead Vec<TagValue> fields on HistoricalDataRequestBuilder
(chart_options) and RealtimeBarsRequestBuilder (options) -- always
default-empty, no setter; pass Default::default() in to_proto instead.

CancelMarketDepthRequestBuilder removed too despite having a body --
PR review flagged that the body is verified nowhere; revisit when a
test_cancel_market_depth lands.

-92 net LOC.

Coverage Stats

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

6 existing lines in 1 file now uncovered.

18332 of 20760 relevant lines covered (88.3%)

24.26 hits per line