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

Fix Cypress fail aléatoire #3991

Merged
merged 1 commit into from
Sep 21, 2023
Merged

Fix Cypress fail aléatoire #3991

merged 1 commit into from
Sep 21, 2023

Conversation

baptou12
Copy link
Contributor

@baptou12 baptou12 commented Sep 20, 2023

Constat

Depuis quelque temps on a de plus en plus de fail de cypress en CI dû à une erreur :

read ECONNRESET
Error: read ECONNRESET
    at TCP.onStreamRead (node:internal/stream_base_commons:217:20)

Test run failed, code 1
More information might be available above
Cypress module has returned the following error message:
Could not find Cypress test run results
Error: Could not find Cypress test run results

Il semblerait que cette erreur soit systématique quand la version de chrome sur le runner Github est la version 117.
Or quand Github fait des mise à jours de ses runner il le fait progressivement (source)

Ça expliquerai l'augmentation de la fréquence de fail de la CI.

D'autres source dise que la version de cypress minimum (pas de source officielle...) pour chorme 117 est cypress 12.15.0 (source).

Resolution

Bump cypress, et vérifier que les tests passent bien avec une version de chrome >116

Note la version de chrome peut se voir dans une tâche end-to-end:

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Cypress:        10.11.0                                                                        │
  │ Browser:        Chrome 117 (headless)                                                          │
  │ Node Version:   v16.20.1 (/home/runner/runners/2.309.0/externals/node16/bin/node)              │
  │ Specs:          1 found (patrimoine.cy.js)                                                     │
  │ Searched:       cypress/e2e/patrimoine.cy.js                                                   │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘

Note: voici au moins une tâche de chaque scenario de test cypress en 117 qui a fonctionné avec la nouvelle version:

⚠️ Warning, même si c'est mieux (sur les nouveau runner) après avoir fait tourné pas mal de fois les tests il reste des fails aléatoire moins fréquents après cette mise à jours, je préconise de les traiter un par un en fonction de la fréquence.

@baptou12 baptou12 changed the title WIP: 2023-09-20 16:10:58 Fix Cypress fail aléatoire Sep 20, 2023
@baptou12 baptou12 force-pushed the fix_ci_chrome_version branch from b0a500f to 67e5497 Compare September 20, 2023 14:26
@baptou12 baptou12 marked this pull request as ready for review September 20, 2023 17:08
Copy link
Contributor

@Shamzic Shamzic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En espérant que cela réduise les fails de CI 👍
Qui plus est, en local je constate que les perf se sont clairement améliorées avec cette nouvelle version.

@baptou12 baptou12 merged commit 25185ca into master Sep 21, 2023
@baptou12 baptou12 deleted the fix_ci_chrome_version branch September 21, 2023 08:55
@guillett guillett added this to the BC 1406243900 milestone Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants