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

Test Replay: [Upload] Display more informative errors #29022

Closed
cacieprins opened this issue Feb 27, 2024 · 1 comment · Fixed by #29312
Closed

Test Replay: [Upload] Display more informative errors #29022

cacieprins opened this issue Feb 27, 2024 · 1 comment · Fixed by #29312

Comments

@cacieprins
Copy link
Contributor

cacieprins commented Feb 27, 2024

In certain cases, the app can display cryptic error messages that are not very helpful - to our users, or to us.

Failed Upload error messages should display after the "Uploaded Cloud Artifacts" section.

Failed Recording error messages should display before the "Uploading Cloud Artifacts" section.

Failed Uploads, Cause: HTTP Status Code 403, 404, 413

Source issues:

Acceptance: surface more informative error:

Unable to upload this spec's Test Replay recording due to an error during the upload process:

$URL responded with HTTP $CODE, $DESCRIPTION

Failed Uploads, Cause: Network Error
Socket Hangup, Proxy Error, ECONNRESET are likely caused by a network or network configuration error.
Acceptance: surface a more informative error; solution: fix network configuration

Acceptance: surface a more informative error:

Unable to upload this spec's Test Replay recording due to a network error during the upload process: $ERR

Please verify your network configuration for accessing $URL.

Source issues:

Failed Uploads, Cause: runtime error
Sometimes we encounter an "cannot read .pipes of undefined" error during upload. This is being thrown from ReadableStream with an unknown cause.
Acceptance: Display an error:

Unable to upload this spec's Test Replay recording. An error occurred while trying to read $FILENAME:

$ERR

Failed Initialization
Sometimes, we fail to initialize Protocol in a way that does not throw a terminal error. This results in either an undefined archivePath on ProtocolManager.ts in app (Error: no test data recorded). Sometimes, an inability to open the sqlite file in beforeSpec results on a terminal error (SqliteError: Unable to open database file). Messaging on both can be improved:

Unable to initialize Test Replay recording for this spec (TBD - determine what is failing, and throw an appropriate error).

Unable to initialize Test Replay recording for this spec. (TBD: should we display the path to the sqlite file we were trying to open?)

Failed Recording
An unknown error, ftruncate error seems to be thrown from indexed-tarfile when we append, sometimes.
Acceptance: Display a more informative error; potential solution: increase available disk space

Unable to add one or more Test Replay assets to the recording. Please verify disk space availability.

SqliteError: Database or Disk is Full can be encountered whenever we write to the database, or close it.
Acceptance: Display a more informative error; solution: increase available disk space

Unable to add one or more Test Replay events to the recording. Please verify disk space availability.

@cacieprins cacieprins self-assigned this Feb 27, 2024
@jennifer-shehane jennifer-shehane added type: user experience Improvements needed for UX Test Replay type: error message and removed type: user experience Improvements needed for UX labels Feb 27, 2024
@jennifer-shehane jennifer-shehane changed the title Test Replay: Display more informative errors Test Replay: [Upload] Display more informative errors Feb 29, 2024
@cypress-bot
Copy link
Contributor

cypress-bot bot commented May 8, 2024

Released in 13.9.0.

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

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators May 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants