Skip to content

chore: handle errors #31854

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

Merged
merged 5 commits into from
Jun 10, 2025
Merged

chore: handle errors #31854

merged 5 commits into from
Jun 10, 2025

Conversation

estrada9166
Copy link
Member

  • Closes

Additional details

Handle different type of errors:

  • Check internet connection
  • Check storage
  • Check runns only e2e
  • Check supported browsers

Steps to test

How has the user experience changed?

PR Tasks

@estrada9166 estrada9166 requested a review from ryanthemanuel June 9, 2025 22:03
Copy link

cypress bot commented Jun 9, 2025

cypress    Run #62971

Run Properties:  status check passed Passed #62971  •  git commit 8b9d1317ac: Fix ts
Project cypress
Branch Review alejandro/feat/handle-errors
Run status status check passed Passed #62971
Run duration 15m 01s
Commit git commit 8b9d1317ac: Fix ts
Committer estrada9166
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 2
Tests that did not run due to a developer annotating a test with .skip  Pending 704
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 740
View all changes introduced in this branch ↗︎
UI Coverage  0%
  Untested elements 4  
  Tested elements 0  
Accessibility  97.09%
  Failed rules  0 critical   1 serious   0 moderate   0 minor
  Failed elements 4  

docsUrl: 'https://on.cypress.io/prompt-download-error',
}
},
promptSupportedBrowser: stripIndent`\
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's add a test for this. We should be able to flex the test logic based on the browser we're running. If we're in firefox/webkit, verify the error is the browser error. https://github.com/cypress-io/cypress/blob/feat/cy-prompt/packages/driver/cypress/e2e/commands/prompt/prompt.cy.ts#L3-L8

@estrada9166 estrada9166 merged commit 30e48d1 into feat/cy-prompt Jun 10, 2025
87 of 89 checks passed
@estrada9166 estrada9166 deleted the alejandro/feat/handle-errors branch June 10, 2025 18:47
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.

2 participants