dalal-street-server icon indicating copy to clipboard operation
dalal-street-server copied to clipboard

Tests for matching engine and streams!

Open thakkarparth007 opened this issue 7 years ago • 8 comments

These two are risky systems. Need to test them properly. It should be easy to do that now because I've decoupled them. Matching engine still talks to the db directly for stoploss orders, and that needs to be fixed somehow. It should be done separately, or at least a better way should be there.

  • [x] Tests for Matching engine
  • [ ] Tests for Streams

thakkarparth007 avatar Jan 05 '18 22:01 thakkarparth007

@gauthamk97 and @adwaith007 this needs to be done by 26th of December.

akspi avatar Dec 25 '18 11:12 akspi

Well this has been partially completed. Closing this for now.

akspi avatar Feb 04 '19 15:02 akspi

No there's so many tests yet to write. It's almost criminal to close this issue. It's alright, let it remain open until we've written satisfactory tests

gauthamk97 avatar Feb 04 '19 16:02 gauthamk97

Updated the milestone to keep our internal demo section clean

gauthamk97 avatar Feb 04 '19 16:02 gauthamk97

@gauthamk97 @adwaith007 is this happening anytime soon?

akspi avatar Feb 09 '19 08:02 akspi

Tests for matching engine is done. Need to write for streams.

adwaith007 avatar Dec 20 '19 08:12 adwaith007

🎉 🎉 😄 😄

gauthamk97 avatar Dec 23 '19 02:12 gauthamk97

Major achievement!

coderick14 avatar Dec 23 '19 02:12 coderick14