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

App test runner remounts web components #22153

Open
alexkrolick opened this issue Jun 6, 2022 · 12 comments
Open

App test runner remounts web components #22153

alexkrolick opened this issue Jun 6, 2022 · 12 comments
Labels
E2E Issue related to end-to-end testing prevent-stale mark an issue so it is ignored by stale[bot] Reproducible Can be reproduced topic: snapshot type: bug

Comments

@alexkrolick
Copy link

alexkrolick commented Jun 6, 2022

Current behavior

When using the interactive test runner UI, if you hover over the chained cypress commands in the left panel, Cypress attempts to restore the DOM state at the given time.

With webcomponents/custom elements, restoring the DOM state causes a new instance of the component to be created each time, which retriggers side-effects such as initial network calls, leading to extremely high request volume. It also fails to restore the actual component state at the time if it uses shadow dom.

Possibly related: #5898

Desired behavior

Hovering previous states in the test runner should restore the state of custom elements' shadow DOM, instead of creating new top-level instances.

OR

There should be some way to use temporary image- or video-based snapshots instead of DOM captures in the replay UI.

Test code to reproduce


Cypress Version

10

Other

No response

@cypress-bot cypress-bot bot added the stage: investigating Someone from Cypress is looking into this label Jun 7, 2022
@AtofStryker AtofStryker added the Reproducible Can be reproduced label Jun 9, 2022
@AtofStryker
Copy link
Contributor

Hey @alexkrolick. Thank you for filing an issue. I believe I was able to reproduce some of the behavior you are seeing with this reproduction repository. Can you confirm this is the behavior you are seeing?

@cypress-bot cypress-bot bot added stage: routed to e2e-core and removed stage: investigating Someone from Cypress is looking into this labels Jun 9, 2022
@alexkrolick
Copy link
Author

Yep, the incrementing counter is a good repro side-effect.

@ThijSlim
Copy link

I am experiencing the same issues in e2e. Also experiencing this with the new Cypress 10 Component Testing.

@nagash77 nagash77 added E2E Issue related to end-to-end testing and removed E2E-core labels Nov 8, 2022
@borislav-minchev
Copy link

Hello, in our project we are using Cypress 12.8.1 and this issue still occurs. Have there been any updates on this?

@L-Ignatova
Copy link

We are experiencing these issues in e2e testing in Cypress 12. Is there a workaround we can implement for the time being or is a solution planned for a future release we can expect?

@AtofStryker
Copy link
Contributor

Hi @borislav-minchev & @L-Ignatova. We currently don't have this issue prioritized, but are always willing to accept contributions to fix the issue!

@cypress-app-bot
Copy link
Collaborator

This issue has not had any activity in 180 days. Cypress evolves quickly and the reported behavior should be tested on the latest version of Cypress to verify the behavior is still occurring. It will be closed in 14 days if no updates are provided.

@cypress-app-bot cypress-app-bot added the stale no activity on this issue for a long period label Oct 3, 2023
@alexkrolick
Copy link
Author

Issue still exists

@cypress-app-bot cypress-app-bot removed the stale no activity on this issue for a long period label Oct 4, 2023
@cypress-app-bot
Copy link
Collaborator

This issue has not had any activity in 180 days. Cypress evolves quickly and the reported behavior should be tested on the latest version of Cypress to verify the behavior is still occurring. It will be closed in 14 days if no updates are provided.

@cypress-app-bot cypress-app-bot added the stale no activity on this issue for a long period label May 27, 2024
@AtofStryker
Copy link
Contributor

still an issue

@cypress-app-bot cypress-app-bot removed the stale no activity on this issue for a long period label May 29, 2024
@alessandro308
Copy link

Is there some webhook to trigger the unregister of the webcomponent? So we can use snapshots...

@jennifer-shehane jennifer-shehane changed the title Test replay inspector remounts web components App test runner remounts web components Jul 26, 2024
@cypress-app-bot
Copy link
Collaborator

This issue has not had any activity in 180 days. Cypress evolves quickly and the reported behavior should be tested on the latest version of Cypress to verify the behavior is still occurring. It will be closed in 14 days if no updates are provided.

@cypress-app-bot cypress-app-bot added the stale no activity on this issue for a long period label Jan 23, 2025
@AtofStryker AtofStryker added prevent-stale mark an issue so it is ignored by stale[bot] and removed stale no activity on this issue for a long period labels Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E2E Issue related to end-to-end testing prevent-stale mark an issue so it is ignored by stale[bot] Reproducible Can be reproduced topic: snapshot type: bug
Projects
None yet
Development

No branches or pull requests