search

moneyhub / moneyhub-api-client

Node.JS Client for the Moneyhub API
10 stars 7 forks source link

Accounts tests use config values #20

Closed gillyhl closed 2 years ago

gillyhl commented 2 years ago

The accounts smoke tests now use config values to get the test account Id and test pension ID to improve robustness.

jonbride commented 2 years ago

Maybe we should add in a different check for the account types being cash:current and pension, since references to those two types were removed?

gillyhl commented 2 years ago

Maybe we should add in a different check for the account types being cash:current and pension, since references to those two types were removed?

Good shout, I'll check for their existence!

gillyhl commented 2 years ago

@jonbride added the check for cash and pension accounts in the tests.