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

WIP: Attempt to fix cypress hanging tests #1982

Closed
wants to merge 30 commits into from
Closed

WIP: Attempt to fix cypress hanging tests #1982

wants to merge 30 commits into from

Conversation

gmrabian
Copy link
Contributor

@gmrabian gmrabian commented Sep 22, 2023

Description

tried result
arbitrary wait 1000 after each test file did not fix
pinning ubuntu version to 20.04 for cypress helped...maybe? got to a11y tests but those still froze
unskip tests did not fix: tests that are not skipped are hanging anyway 😢
micro package updates for cypress and concurrently still got a hanging test
modify config: remove settings that match default, set retry to 0, move setupNodeEvents to config everything caught fire
modify config: re-set video to false (default true until cy 13), readd default command timeout at 15s it keeps getting worse 😢
maybe add debug? too noisy 😵‍💫
give it one retry still failing
update a bunch of actions because they're breaking probably no impact on cypress
reskip flaky tests because now I don't see hanging they all passed 😱 UPDATE: could never replicate this 😭
make cypress run in docker on chrome 116? still hanging
try filtering out disable-dev-shm-usage flag nope
or maybe add it? nope
remove all tests that don't run in 2023 to limit overall load no
limit the matrix like managed care review does failed – we aren't running parallel jobs which requires cypress cloud
try experimental memory management hangs
try another experiment nope
next? old heads nope
old headless plus container locked to chrome version? JOY!

Related ticket(s)

MDCT-2977


How to test


Author checklist

  • I have performed a self-review of my code
  • I have added thorough tests, if necessary
  • I have updated relevant documentation, if necessary

@gmrabian gmrabian changed the title Attempt to fix cypress hanging tests WIP: Attempt to fix cypress hanging tests Sep 22, 2023
@gmrabian gmrabian mentioned this pull request Sep 28, 2023
3 tasks
@codeclimate
Copy link

codeclimate bot commented Sep 28, 2023

Code Climate has analyzed commit 41ffe13 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (90% is the threshold).

This pull request will bring the total coverage in the repository to 73.3% (0.0% change).

View more on Code Climate.

This was referenced Sep 28, 2023
@gmrabian
Copy link
Contributor Author

Work covered by
#1984
#1985
#1986

dystopia averted 😌

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.

1 participant