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

Handle "ResizeObserver loop limit exceeded" exception #121

Open
Bronstrom opened this issue Jul 27, 2022 · 3 comments
Open

Handle "ResizeObserver loop limit exceeded" exception #121

Bronstrom opened this issue Jul 27, 2022 · 3 comments
Labels
question Further information is requested

Comments

@Bronstrom
Copy link
Contributor

After moving elements of our React application, some of our Cypress tests report the uncaught exception: "ResizeObserver loop limit exceeded." Most of the OSCAL_Edit_Tests.cy.js will display this error. Please note, when running our workflow tests will fail to this exception, although tests will not fail locally in Cypress but still display the exception.

Acceptance Criteria

"ResizeObserver loop limit exceeded" is no longer avoided and the uncaught exception does not show when running tests.

Implementation Notes

This error may be related to the communication between OSCALLoader and App in our React application.

@Bronstrom Bronstrom changed the title Handle ResizeObserver loop limit exceeded error Handle "ResizeObserver loop limit exceeded" exception Jul 27, 2022
@laurelmay
Copy link
Contributor

@Bronstrom is there any reason to believe that this error still occurs? Can we reproduce?

@laurelmay laurelmay added the question Further information is requested label Mar 15, 2023
@Bronstrom
Copy link
Contributor Author

Bronstrom commented Mar 23, 2023

@kylelaker I'm still seeing these occur when running develop branches on both the Viewer and REST service locally:

image

I believe this exception is still likely due to some interaction between OSCALLoader and App.

@tuckerzp
Copy link
Contributor

tuckerzp commented Mar 23, 2023

@Bronstrom I also had that error. From what I read, they are safe to ignore... I could be wrong on that. I think it happens becuase cypress changes the sizing windows so rapidly.

I added a exception catch in commands.js
https://github.com/EasyDynamics/oscal-editor-deployment/pull/232/files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants