Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

e2e tests are failing randomly on dependencies installation part #571

Closed
rostislav-simonik opened this issue May 24, 2023 · 1 comment · Fixed by #578
Closed

e2e tests are failing randomly on dependencies installation part #571

rostislav-simonik opened this issue May 24, 2023 · 1 comment · Fixed by #578
Labels
type/bug Something is not working the way it should

Comments

@rostislav-simonik
Copy link
Collaborator

rostislav-simonik commented May 24, 2023

Each e2e test is installing dependencies on its own. Which from time to time results in failure. The Windows runner with node 18 is affected the most. Wasn't able to investigate the root problem.

Currently, we are installing dependencies with npm packager. I have tried with yarn it gets a better ratio but the problem is still there. pnpm gets even better results, my guess is it's somehow related to network connectivity failures.

E2E tests also take too much to execute, the major part of its installation of dependencies.

Unless we identify the root issue I'm recommending introducing the following changes

  • short-term: replace with yarn packager -> shorter installation time, decreases rates of failure.
    • it's low complexity, it requires just replacing npm run install with yarn install
  • medium-term: replace with pnpm packager -> it reduces installation even more than yarn, and also decreases rates of failure.
    • it requires a little bit of refactoring of tests, to fix output for prisma and nexus-prisma and to fix post-install script for prisma@4.0.0 version.
@rostislav-simonik
Copy link
Collaborator Author

By adding more logs in #578 I have narrowed the issue to the installation of dependencies, added logic to kill installation if it crosses the expected period, and added retry logic (max up to 3 retries)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug Something is not working the way it should
Projects
None yet
1 participant