ScalABM / auctions

A functional API for auction simulations
Apache License 2.0
13 stars 1 forks source link

Add tests! #46

Closed davidrpugh closed 7 years ago

davidrpugh commented 7 years ago

This PR is for adding unit, functional, and performance tests to the code base.

davidrpugh commented 7 years ago

@bherd-rb I have found a bug in the remove method of the FourHeapOrderBook class. Am working on a fix now.

davidrpugh commented 7 years ago

@bherd-rb I have fixed the bug and pushed these changes. Let's merge this PR and I will open another one to add more tests...

davidrpugh commented 7 years ago

@bherd-rb More tests...more bugs! But progress is being made...I am now working on sorting out and issue with discriminatory price double auctions. Hopefully will have sorted soon...

davidrpugh commented 7 years ago

For now I am going to focus on unit and functional tests...and save performance testing for version 0.2.0 or later...