Skip to content

[ML] Add a script to run each unit test separately#2859

Merged
edsavage merged 26 commits intoelastic:mainfrom
edsavage:run_tests_individually
Jul 29, 2025
Merged

[ML] Add a script to run each unit test separately#2859
edsavage merged 26 commits intoelastic:mainfrom
edsavage:run_tests_individually

Conversation

@edsavage
Copy link
Copy Markdown
Contributor

@edsavage edsavage commented Jun 27, 2025

Add a script to provide a wrapper around the call to "cmake" that runs the test cases and provides some flexibility as to how the tests should be run in terms of how they are spread across processes. This is necessary when trying to isolate the impact memory usage of tests have upon one another.

The script is currently only called from within the Linux CI pipeline

Add a script to provide a wrapper around the call to "cmake" that runs the test cases and provides some flexibility as to how the tests should be run in terms of how they are spread across processes. This is necessary when trying to isolate the impact memory usage of tests have upon one another.
@prodsecmachine
Copy link
Copy Markdown

prodsecmachine commented Jun 27, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@edsavage edsavage removed the WIP label Jul 11, 2025
@edsavage edsavage marked this pull request as ready for review July 11, 2025 04:43
Copy link
Copy Markdown
Contributor

@valeriy42 valeriy42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Does this change the commands for how we can/should run tests? If so, can you please extend the documentation?

Update the CONTRIBUTING.md doc with details on the new feature enabling tests to be run individually.
@elastic-sonarqube
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@valeriy42 valeriy42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@edsavage edsavage merged commit 8cffe88 into elastic:main Jul 29, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants