Add minimal performance test to detect telemetry eviction handover - Githubissues

akhenry / openmct-yamcs

Open MCT YAMCS plugin

13 stars 9 forks source link

Add minimal performance test to detect telemetry eviction handover #425

Open scottbell opened 7 months ago

scottbell commented 7 months ago

Closes #7312

Describe your changes:

Add telemetry eviction tests by way of FinalizationRegistry

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Is this change backwards compatible? For example, developers won't need to change how they are calling the API or how they've extended core plugins such as Tables or Plots.

Author Checklist

[x] Changes address original issue?
[x] Tests included and/or updated with changes?
[x] Command line build passes?
[x] Has this been smoke tested?
[ ] Testing instructions included in associated issue OR is this a dependency/testcase change?

Reviewer Checklist

[ ] Changes appear to address issue?
[ ] Reviewer has tested changes by following the provided instructions?
[ ] Changes appear not to be breaking changes?
[ ] Appropriate automated tests included?
[ ] Code style and in-line documentation are appropriate?
[ ] Has associated issue been labelled unverified? (only applicable if this PR closes the issue)
[ ] Has associated issue been labelled bug? (only applicable if this PR is for a bug fix)

scottbell commented 7 months ago

Note this is not a PR meant to go into review, but a handoff of the telemetry eviction testing ticket to @ozyx. There are some notes here on the current status of this branch. There are two other branches associated with this ticket using the aborted Memlab approach; these can be deleted at your discretion.