Skip to content

internal: (studio) quality of life improvements for the studio lifecycle #31663

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

Conversation

ryanthemanuel
Copy link
Collaborator

@ryanthemanuel ryanthemanuel commented May 8, 2025

Additional details

This PR adds a few quality of life improvements for the Studio initialization workflow. It provides additional debugging context when things fail.

Steps to test

How has the user experience changed?

PR Tasks

@ryanthemanuel ryanthemanuel self-assigned this May 8, 2025
Copy link

cypress bot commented May 8, 2025

cypress    Run #62263

Run Properties:  status check passed Passed #62263  •  git commit 6d0bf7b93b: internal: (studio) quality of life improvements for the studio lifecycle
Project cypress
Branch Review ryanm/internal/provide-more-info-on-studio-session-failures
Run status status check passed Passed #62263
Run duration 19m 25s
Commit git commit 6d0bf7b93b: internal: (studio) quality of life improvements for the studio lifecycle
Committer Ryan Manuel
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 8
Tests that did not run due to a developer annotating a test with .skip  Pending 1232
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 32157
View all changes introduced in this branch ↗︎
UI Coverage  46.24%
  Untested elements 188  
  Tested elements 166  
Accessibility  92.74%
  Failed rules  3 critical   9 serious   2 moderate   2 minor
  Failed elements 889  

@@ -47,6 +46,10 @@ const downloadStudioBundleToTempDirectory = async ({ studioUrl, projectId }: Opt
encrypt: 'signed',
})

if (!response.ok) {
throw new Error(`Failed to download studio bundle: ${response.statusText}`)
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the user experience with throwing this error?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We catch it up stream and report it. I'll take a look at @astone123's PR to make sure it plays nicely there, but prior to this change it would actually have failed in a different (more confusing) way.

@ryanthemanuel ryanthemanuel merged commit e3a7994 into develop May 9, 2025
84 of 89 checks passed
@ryanthemanuel ryanthemanuel deleted the ryanm/internal/provide-more-info-on-studio-session-failures branch May 9, 2025 20:44
cypress-app-bot pushed a commit that referenced this pull request May 11, 2025
cypress-app-bot pushed a commit that referenced this pull request May 11, 2025
cypress-app-bot pushed a commit that referenced this pull request May 11, 2025
cypress-app-bot pushed a commit that referenced this pull request May 11, 2025
cypress-app-bot pushed a commit that referenced this pull request May 11, 2025
cypress-app-bot pushed a commit that referenced this pull request May 11, 2025
cypress-app-bot pushed a commit that referenced this pull request May 12, 2025
cypress-app-bot pushed a commit that referenced this pull request May 12, 2025
cypress-app-bot pushed a commit that referenced this pull request May 12, 2025
cypress-app-bot pushed a commit that referenced this pull request May 12, 2025
cypress-app-bot pushed a commit that referenced this pull request May 12, 2025
cypress-app-bot pushed a commit that referenced this pull request May 12, 2025
jennifer-shehane pushed a commit that referenced this pull request May 12, 2025
* chore: updating v8 snapshot cache

* index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663)

* index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663)

---------

Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
jennifer-shehane pushed a commit that referenced this pull request May 12, 2025
* chore: updating v8 snapshot cache

* index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663)

* index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663)

---------

Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
jennifer-shehane pushed a commit that referenced this pull request May 12, 2025
* chore: updating v8 snapshot cache

* index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663)

* index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663)

---------

Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
@cypress-bot
Copy link
Contributor

cypress-bot bot commented May 20, 2025

Released in 14.4.0.

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

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

Successfully merging this pull request may close these issues.

2 participants