wboayue / rust-ibapi / 25285032604

92%

Build Type

push

github

Committed by web-flow

Commit Message

test(accounts): builders + body-verifying test migration (#496)

PR 2 of the test-builders plan (todos/test-builders.md). Adds typed
response and request builders for the accounts domain (everything
beyond positions, which landed in PR 1) and migrates the sync and
async account tests onto them, upgrading msg-id-only assertions to
full request-body verification via assert_request<B>.

Builders added (testdata/builders/accounts.rs)
- Response: managed_accounts, account_summary, account_summary_end,
  account_value, account_download_end, account_update_multi,
  account_update_multi_end, family_codes, current_time, pnl, pnl_single
  -- each implements ResponseEncoder for text + to_proto/encode_proto
  for protobuf
- Request: request_managed_accounts, request_account_summary,
  cancel_account_summary, request_account_updates,
  cancel_account_updates, request_account_updates_multi,
  cancel_account_updates_multi, request_pnl, cancel_pnl,
  request_pnl_single, cancel_pnl_single, request_family_codes,
  request_current_time -- each impls RequestEncoder

Tests migrated
- src/accounts/sync/tests.rs: 21 tests, all assert_request_msg_id calls
  upgraded to assert_request<B> with body verification (request_id,
  account, model_code, etc.)
- src/accounts/async/tests.rs: 16 tests, same upgrade
- 40 new builder unit tests covering text encode, proto round-trips,
  and encode_request msg-id headers

Cleanup
- Drop unused string constants from src/testdata/responses.rs (kept
  MANAGED_ACCOUNT, ACCOUNT_UPDATE_MULTI_CURRENCY, ACCOUNT_VALUE,
  MARKET_RULE -- still used by recorder.rs and decoder tests)

Coverage Stats

504 of 598 new or added lines in 3 files covered. (84.28%)

1 existing line in 1 file now uncovered.

17656 of 20011 relevant lines covered (88.23%)

24.41 hits per line