replaced get_barset with get_bars, test for positions fixed

[benrdavison]

-get_barset no longer exists, replaced by get_bars (requires import of TimeFrame)
-In tests, it appears that trying to get Position is failing due to buying then immediately selling test stock; to make a position request, I believe the stock needs to be held, so I changed the purchase amount to 2 shares (I know this is not elegant, maybe someone will have a better idea, just wanted to bring awareness to why tests were failing).

Thanks!
BD

[AxelGard]

Thank you!
Your changes looks good.

Yeah I need to rewrite the tests to actually be some proper mock tests, but they do catch things like this, so thanks!

[AxelGard]

Btw how are you liking cira? Anything you think is missing? @benrdavison

[AxelGard]

The test seems to fail due to the set version of alpaca is older and therefore does not have the TimeFrame. I can merge and re-run the test when I have upgraded the alpaca version. But then I need to check what else will break if I do so. Might take some time. 😅

[benrdavison]

Thanks! Yes, I started looking into trying to create a simpler wrapper for Alpaca, and stumbled upon this, really enjoying, thanks again.

[AxelGard]

Glad to hear that you like it!