Thanks for the PR. place_stop_order is now already resolved by #370 (I took the first PR first).
As for fixing the unit tests -- it does look like they removed the /position endpoint, which we should probably remove (both test and dead code). However, I don't think we should be replacing the test_get_position with a test on get_coinbase_accounts.
I noticed that you're testing that 'currency' exists in the get_coinbase_accounts response, which may be something to consider adding, but for now I'm going to close this.
Thanks for the PR.
place_stop_orderis now already resolved by #370 (I took the first PR first).As for fixing the unit tests -- it does look like they removed the
/positionendpoint, which we should probably remove (both test and dead code). However, I don't think we should be replacing thetest_get_positionwith a test onget_coinbase_accounts.I noticed that you're testing that 'currency' exists in the
get_coinbase_accountsresponse, which may be something to consider adding, but for now I'm going to close this.