corbado / javascript

MIT License
7 stars 2 forks source link

Introduce test automation with Playwright #67

Closed incorbador closed 5 months ago

incorbador commented 6 months ago

Why

As we are completing our initial version of react-sdk and react we want to add automated testing for the react package. We want to do that now because it tells us, if our code so far is testable. Additionally we then know if we break some of the existing capabilities of react-sdk and react when we change things in the frontend or in the backend.

TODO

Acceptance criteria

Implementation idea

What do we need to mock:

incorbador commented 5 months ago

Tradeoff:

=> For now we will use Playwright because of its better passkey support

Aby-JS commented 5 months ago

Implementation plan for this ticket after research:

incorbador commented 5 months ago

@Aby-JS Fixed test OTP codes have been deployed. We made a small change though. For our three test projects (pro-503401103218055321, pro-3652881945085154854 and pro-1743528526530787479) all email addresses that follow the pattern integration-test-\s@corbado.com will get the same OTP code 150919. (This makes it easier for other integration tests).

Aby-JS commented 5 months ago

@incorbador the PR for this has been merged. Can you please add a rule for main branch that we need to run this job before PR is merged. You can do this in settings for the javascript repository.