Flaky test: Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/root/cypress/packages/data-context/test/unit/sources/GitDataSource.spec.ts)

[rachelruderman]

Link to dashboard or CircleCI failure

https://app.circleci.com/pipelines/github/cypress-io/cypress/41761/workflows/4f6901d8-6339-4309-b284-e03055666f1c/jobs/1732010

Link to failing test in GitHub

https://github.com/cypress-io/cypress/blob/develop/packages/data-context/test/unit/sources/GitDataSource.spec.ts#L46

Analysis

The missing done doesn't fix the test failure -- it's a symptom of the test failure. That said, the missing done is an easy fix and will help us see the actual error that's happening. To do so, let's just add a try/catch, where the try ends with done() and the catch ends with done(new Error(error))

Cypress Version

10.4.0

Other

Search for this issue number in the codebase to find the test(s) skipped until this issue is fixed

[rachelruderman]

Added .skip to this test in #23167. Please remove .skip when test is stable

[flotwig]

Looked at this as part of #25671 - retries can be added, but it fails persistently if it flakes even once.

[lmiller1990]

Still flaky ... we will fix it, or write a new test that does the same thing that isn't flaky.

Hey team! Please add your planning poker estimate with Zenhub @astone123 @marktnoonan @mike-plummer @warrensplayer @ZachJW34

[warrensplayer]

Hey team! Please add your planning poker estimate with Zenhub @astone123 @lmiller1990 @marktnoonan @mike-plummer @ZachJW34

[lmiller1990]

I'd like to limit this to half a day max - I'd also like to know if we've already got coverage for this somewhere else, and if we do, just delete this test.

[cypress-bot]

Released in 12.9.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v12.9.0, please open a new issue.