Skip to content

fix: wire trade order test endpoint#113

Open
suihan253561554 wants to merge 1 commit into
singlesly:mainfrom
suihan253561554:codex/trade-order-test-endpoint-81
Open

fix: wire trade order test endpoint#113
suihan253561554 wants to merge 1 commit into
singlesly:mainfrom
suihan253561554:codex/trade-order-test-endpoint-81

Conversation

@suihan253561554
Copy link
Copy Markdown

Summary

  • Wire TradeService.createTradeOrderTest to BingxTradeOrderTestEndpoint instead of the live order endpoint.
  • Export the test endpoint for public use.
  • Add service-level unit coverage proving live orders and test orders dispatch different endpoint classes.
  • Mark Trade order test as implemented in the README.
  • Remove an unused import so npm run lint passes.

Closes #81

Verification

  • npm run build
  • npm test -- --runInBand src/bingx-client/services/trade.service.spec.ts src/bingx-client/bingx-api.client.spec.ts
  • npm run lint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wire trade order test endpoint into TradeService

1 participant