Overview

Reviewing e2e test #3498219: Missing E2E test for Publish button hides bugs I found myself wishing the test was running at slow rate so I could see what was going on with each set. I want make sure it is testing the back-end portions I think it is

Proposed resolution

@callumharrod point out the there is cy.pause().
I would love if the test could be run in away that when key events happend like the test determine the preview is ready or a element was dragged on the page, I could have it wait until I pressed play to go forward.

My suggestion

  1. Add a debugPauses Cypress option that will pause tests at certain key points
  2. Add a cy.debugPause() function that will check this option and pause the test if needed
  3. Call cy.debugPause() in commands like previewReady

User interface changes

Issue fork experience_builder-3498825

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

tedbow created an issue. See original summary.

tedbow opened merge request !528

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Assigned: tedbow » Unassigned
Status: Active » Needs review
tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Issue tags: +e2e

hooroomoo made their first commit to this issue’s fork.

hooroomoo’s picture

hooroomoo
they/them
Location nyc🗽
commented

  • hooroomoo committed 3b8d477d on 0.x authored by tedbow 
    Issue #3498825 by tedbow: Make it easy to have e2e test pause...
hooroomoo’s picture

hooroomoo
they/them
Location nyc🗽
commented
Status: Needs review » Fixed
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Fixed » Needs work
Issue tags: +Needs documentation

I expect this to be documented in /ui/README.md. Can we get a tiny follow-up MR in this issue to rectify that? 😊🙏

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Assigned: Unassigned » tedbow

Assigning to myself for reminder but if anyone else wants to make follow-up MR please assign yourself. You can assign back to me for review

tedbow opened merge request !548

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Assigned: tedbow » Unassigned
Status: Needs work » Needs review
tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Title: Make it easy to have e2e test pause periodically during tests » Add a debug setting to pause e2e tests during key events, such as clicking on a component

Renaming the title case in body sees this in the queue, to give a better idea of what it does

hooroomoo’s picture

hooroomoo
they/them
Location nyc🗽
commented

Approved but I think it isn't letting me merge bc the MR is still marked as a draft.

hooroomoo’s picture

hooroomoo
they/them
Location nyc🗽
commented
Status: Needs review » Fixed
Issue tags: -Needs documentation

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.