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.

Sign up for GitHub

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

Jump to bottom

[testing] Abstract and apply memory usage testing across the repo #3722

Closed
1 task done
Westbrook opened this issue Oct 16, 2023 · 2 comments · Fixed by #4228 or #4264
Closed
1 task done

[testing] Abstract and apply memory usage testing across the repo #3722

Westbrook opened this issue Oct 16, 2023 · 2 comments · Fixed by #4228 or #4264
Assignees
@Rajdeepc @TarunAdobe
Labels
Performance Spectrum Engineering Testing

Comments

@Westbrook
Copy link
Collaborator

Code of conduct

  • I agree to follow this project's code of conduct.

Description of issue

Building off of #3721, abstract the usage of measureUserAgentSpecificMemory() to a test helper, and apply that helper across all of the packages in the project.

Address leaks, etc., as they are uncovered.
@Westbrook Westbrook mentioned this issue Oct 18, 2023
Merged
8 tasks
@Westbrook
Copy link
Collaborator Author

Questions to answer:

  • is it safe to run this "flag-customized" version of chromium for all tests? or should you create a group config using it that focuses only on the grid memory tests?

@Westbrook Westbrook pinned this issue Jan 11, 2024
@Westbrook Westbrook mentioned this issue Feb 20, 2024
Merged
8 tasks
@Rajdeepc Rajdeepc self-assigned this Mar 21, 2024
@Rajdeepc Rajdeepc linked a pull request Mar 28, 2024 that will close this issue
performance: added memory tests in packages #4228
Merged
13 tasks
@Westbrook Westbrook mentioned this issue Apr 2, 2024
Merged
13 tasks
@Westbrook
Copy link
Collaborator Author

Not complete yet. But good progress.

@Westbrook Westbrook reopened this Apr 2, 2024
@TarunAdobe TarunAdobe mentioned this issue Apr 12, 2024
Merged
13 tasks
@TarunAdobe TarunAdobe linked a pull request Apr 12, 2024 that will close this issue
chore: added memory tests for missing packages #4264
Merged
13 tasks
@Westbrook Westbrook unpinned this issue Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Rajdeepc Rajdeepc

@TarunAdobe TarunAdobe

Labels
Performance Spectrum Engineering Testing
Projects
None yet
Milestone
No milestone
3 participants
@Westbrook @Rajdeepc @TarunAdobe