added some clarifying comments to trade.h and trade_executor.h
removed trade tests which are now covered in trade_executor_tests
aaaand forgot the CMakeList entry for trade_tests
fixed a bug causing an error in some tests due to the addition of the Trade's price member
removed old single-trade trader interface, and updated all tests. executor and full sim tests now share the TestTrader object for testing.