[Nikhil ad-hoc] [5] (5) Simple Integration test for buysell strategy

[nikhilsaraf]

Desired Behavior

Travis test run should fail of the build is unable to start up and trade

Impact

The desired behavior will allow me to confidently merge pull requests without having to sync the code and test locally

Feature Suggestion

We can achieve the desired behavior by having a simple integration test that generates the required accounts, assets, and trustlines, funding, and runs a simple trading strategy.

This should be run as a go test along with all the other existing tests.

Testing against the test network is ok for now (vs. spinning up a new private testnet)

This can run against the buysell strategy

[nikhilsaraf]

use hardcoded accounts for now, if testnet is reset anytime we can always circle back to autogenerated accounts.

[nikhilsaraf]

Can use testnet for now.

If it's easy and computationally inexpensive, can consider using docker quickstart in standalone mode for a "private" network. just start it with --standalone switch.

two options:

• just start the container at the beginning of the tests, run your bootstrap code and do it every time or
• start a container but cache it, so you don't have to run the bootstrap code each time.

second option may be actually slower because travis has some problem with caching large files. you need to check it out.

[nikhilsaraf]

@debnil assigning to myself since I'm working on part of the code that touches this