search

IBMStockTrader / looper

Performance/stress test for IBM Stock Trader
Apache License 2.0
0 stars 12 forks source link

Add Looper run validation using entries from MondoDB #5

Closed Raunak-S closed 2 years ago

Raunak-S commented 3 years ago

PR for incomplete changes, but wanted to submit them so they aren't lost in the future.

Uses gson and calls to TradeHistory to receive stock purchase entries and compare them to the 'correct' values which are calculated based on iterations argument.

Alternative (most likely a better idea) would be to create a proxy endpoint in Broker that calls the same methods in TradeHistory, so that Looper doesn't need to connect directly to TradeHistory.

jwalcorn commented 2 years ago

I've decided not to merge this. I agree with the alternate proposal, of proxying stuff through Broker, instead of this direct interaction with Trade History (and through its public Route?). I appreciate the effort, but if we decide this needs to change, will do it that way instead.