Range Error : Maximum call stack size exceeded

rameshrc commented 4 years ago

I have application with lit-html web component. When button is clicked on a test bed html, dialog (lit-html web component) appears with ok button. I am trying to hover over it. Since no solutions of hover work for me, i tried out "cypress-nhover": "1.0.0",

cy.get('dialog').get('.ok-button', { includeShadowDom: true }).nhover() It creates an error get .ok-button scrollIntoView {log: false} RangeError Maximum call stack size exceeded

Chrome 75 "dependencies": { "lit-element": "^2.3.1" }, "devDependencies": { "@open-wc/testing": "^2.0.0", "@open-wc/testing-karma": "^3.0.0", "@types/node": "13.11.1", "chrome-remote-interface": "0.28.2", "concurrently": "^5.1.0", "cypress": "^4.10.0", "cypress-nhover": "1.0.0", "deepmerge": "^3.2.0", "es-dev-server": "^1.54.1", "start-server-and-test": "^1.11.0", "tslib": "^2.0.0", "typescript": "^3.9.5" }

avallete commented 4 years ago

Hi,

cypress-nhover doesn't do any special thing to handle shadowDOM, that is probably the cause of the error.

If you can provide a MWE either into another repo or some third party public environment (like stackblitz.com) it may be a good start for me to investigate what is the root of the crash.

rameshrc commented 4 years ago

Hi,

cypress-nhover doesn't do any special thing to handle shadowDOM, that is probably the cause of the error.

If you can provide a MWE either into another repo or some third party public environment (like stackblitz.com) it may be a good start for me to investigate what is the root of the crash.

Please find test repo in here lithtml

rameshrc commented 4 years ago

Sorry forgot to add test files. Added them now

avallete commented 4 years ago

Thank's for your help @rameshrc. I know it sounds like a ping-pong game, but the issue doesn't seem to come from the plugin itself. (see my comment here https://github.com/cypress-io/cypress/issues/7986).

I'll be trying to find a way to fix the root of this issue into cypress. I may also try to look for a quick and dirty fix who would make the call to scrollIntoView optional to avoid this error.

avallete commented 4 years ago

After trying some tests, the issue isn't only due to scrollIntoView(even if it's the first part). Because when I remove the call from the command, then the command fails to retrieve the component into the shadowDom via CDP.

Right now I don't have any clue about how to deal with that, it will require more digging and a bunch of time. To understand how to make it work.

avallete commented 4 years ago

avallete / cypress-nhover

Range Error : Maximum call stack size exceeded #14