Feat/suite desktop core/invity mock for tests #16429
Conversation
There was a problem hiding this comment.
At this stage, I just copy pasted the responses from the life invity API.
Once we have first tests for buy, sell and exchange, we will optimize and slim down these files. Because then we will know what we need and don't.
There was a problem hiding this comment.
Hurray, Mocked API means we will always have same results. No need of masking that was anyway flaky.
| const quoteProviderLocator = '@coinmarket/offers/quote/provider'; | ||
| const quoteAmountLocator = '@coinmarket/offers/quote/crypto-amount'; |
There was a problem hiding this comment.
I don't like this approach, but I could not find better way.
I need to do a locator chaining and for some reason
this.page.getByTestId('@coinmarket/offers/quote').and(this.page.getByTestId('@coinmarket/offers/quote/provider')) didnt work for me and I had to use:
this.page.getByTestId('@coinmarket/offers/quote').getByTestId('@coinmarket/offers/quote/provider')
So i pulled '@coinmarket/offers/quote/provider' to a const definition so it woudn't be once in constructor and once in method validateBuyQuotes
Vere-Grey
force-pushed
the
feat/suite-desktop-core/invity-mock-for-tests
branch
2 times, most recently
from
66aef75 to
c223236
Compare
Vere-Grey
force-pushed
the
feat/suite-desktop-core/invity-mock-for-tests
branch
from
c223236 to
e7a18da
Compare
Description
Setups invity mocking for buy test. So far only web is working. I will work on desktop as next step
Related Issue
Resolve 15606, #16041