twenty icon indicating copy to clipboard operation
twenty copied to clipboard

Setup E2E testing

Open lucasbordeau opened this issue 1 year ago • 5 comments

Twenty starts to have a lot of features and different use cases to test.

When doing refactors we usually try to test the app manually to make sure everything is working as expected.

But as it can now take a long time, we would like to have at least manual e2e tests to launch locally while developing or reviewing PR that involve refactors.

We already have integration tests with storybook and a lot of unit tests with jest, we also have visual regression testing with chromatic.

So e2e should focus on the most basics things to detect bugs :

  • Ensuring login works
  • Ensuring there's no bug or errors when we visit each page
  • Ensuring different workflows are executing without bugs or errors

Where e2e would compound efficiently is to offer us a minimal test coverage on a lot of small workflows because tests are easy to write.

We would see later how to integrate this in CI as it is not the main priority right now.

TODO :

  • [ ] Create a new package twenty-e2e-testing and install playwright following its documentation
  • [ ] Setup environment variables for the base url, etc.
  • [ ] Create a basic test that visits /objects/companies and that verifies there's a table shown

lucasbordeau avatar Jul 22 '24 09:07 lucasbordeau

Hey @lucasbordeau I would like to take this issue, please assign it to me.

BarryByte avatar Jul 23 '24 15:07 BarryByte

Any chance I can work on this issue? @Bonapara @lucasbordeau

pacyL2K19 avatar Jul 24 '24 10:07 pacyL2K19

@BarryByte I'll assign you since you're first.

lucasbordeau avatar Jul 25 '24 09:07 lucasbordeau

hey @BarryByte just checking, are you still working on it and made some progress? let us know if its available to take, I'm interested to work on this, also just want to mention, making draft PRs are great way to avoid this workflow.

cc @lucasbordeau

Nabhag8848 avatar Aug 05 '24 06:08 Nabhag8848

sorry, currently i am unable to do take this forward. @lucasbordeau Please assign @Nabhag8848 for this issue. Thank you.

BarryByte avatar Aug 05 '24 08:08 BarryByte