-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Automatically Check Additional Views for Memory Leaks on navigation #6734
Comments
|
See https://github.com/nasa/openmct/pull/7224/files which may address some of these issues. |
@michaelrogers I've added objects for testing these objects in the associated branch to this issue: |
@michaelrogers I've got the tests in place, and there are three tests failing currently:
so scatter plots, bar graphs, and complex display layouts. |
@michaelrogers fixed the memory leaks. I'm going to make a PR |
Follow on from #4406
#6718 introduces the ability to test views for memory leaks on navigation, as well as memory leak tests for a number of screens. To limit the scope of that PR we have split the screens between two issues. This issue contains the second and final batch of screens that need memory leak tests defined in the open repo.
Note that tests are defined for some but not all of these, but are disabled due to finding memory leaks. The memory leaks will need to be fixed before merging this PR.
The text was updated successfully, but these errors were encountered: