Skip to content

Conversation

@astone123
Copy link
Contributor

@astone123 astone123 commented Oct 28, 2025

  • Closes N/A

Additional details

We updated the screen that the user sees in the studio panel depending on the button that they click to open the panel. This PR updates the e2e tests to match that behavior. It also fixes an issue where we were getting a detached DOM error when asserting on elements in the AUT iframe.

Detached DOM issue:
https://cloud.cypress.io/projects/ypt4pf/runs/66905/overview/ff0fae1f-3022-4865-9f74-a235d56437c0/replay?roarHideRunsWithDiffGroupsAndTags=1&ts=1761658484931.7&att=1

New test button issues:
https://cloud.cypress.io/projects/ypt4pf/runs/66914/overview/ca6734e3-2711-4abc-9cea-68e1d393dc88?roarHideRunsWithDiffGroupsAndTags=1

Steps to test

Run the tests in studio-new-tests.cy.ts

How has the user experience changed?

N/A

PR Tasks


Note

Refactors Studio test helper and updates e2e specs to align with new new-test flow and create-test-button selector, including conditional handling when launched from suite/spec header.

  • Studio e2e helper (packages/app/cypress/e2e/studio/helper.ts):
    • Refactor inputNewTestName to accept { name, creatingNewTestFromWelcomeScreen } and conditionally click new-test-button based on context.
    • Update assertions to use create-test-button instead of new-test-button.
  • Navigation tests (studio-navigation.cy.ts):
    • Use inputNewTestName({ creatingNewTestFromWelcomeScreen: false }) when creating tests from a suite.
  • New tests creation (studio-new-tests.cy.ts):
    • Use inputNewTestName({ creatingNewTestFromWelcomeScreen: false }) for suite/spec-header flows.
    • Remove redundant click of create-new-test-from-suite (now handled by helper) and ensure recording state before proceeding.

Written by Cursor Bugbot for commit e0a9b37. This will update automatically on new commits. Configure here.

@astone123 astone123 self-assigned this Oct 28, 2025
cursor[bot]

This comment was marked as outdated.

@cypress
Copy link

cypress bot commented Oct 28, 2025

cypress    Run #66941

Run Properties:  status check passed Passed #66941  •  git commit e0a9b3776b: Update packages/app/cypress/e2e/studio/studio-new-tests.cy.ts
Project cypress
Branch Review studio-welcome-test-update
Run status status check passed Passed #66941
Run duration 16m 48s
Commit git commit e0a9b3776b: Update packages/app/cypress/e2e/studio/studio-new-tests.cy.ts
Committer Adam Stone-Lord
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 3
Tests that did not run due to a developer annotating a test with .skip  Pending 132
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 5595
View all changes introduced in this branch ↗︎

@astone123 astone123 requested a review from mschile October 28, 2025 20:36
@astone123 astone123 merged commit 7079a22 into develop Oct 29, 2025
90 of 92 checks passed
@astone123 astone123 deleted the studio-welcome-test-update branch October 29, 2025 15:49
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.

4 participants