Skip to content

Add tests for tracking provider #1144

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

Closed

Conversation

ldorau
Copy link
Contributor

@ldorau ldorau commented Feb 27, 2025

Description

Add tests for tracking provider.

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly

@ldorau ldorau force-pushed the Add_tests_for_tracking_provider branch from a531874 to 4f91abe Compare February 27, 2025 14:32
#define TEST_LEVEL_SUCCESS 7
#define TEST_LEVEL_FAILURE 8

TEST_P(TrackingProviderTest, success_max__levels) {
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: success_max_levels

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Multilevel maps are needed to support the case
when one memory pool acts as a memory provider
for another memory pool (nested memory pooling).

Signed-off-by: Lukasz Dorau <[email protected]>
@ldorau ldorau force-pushed the Add_tests_for_tracking_provider branch from 4f91abe to 4830048 Compare February 28, 2025 12:20
@ldorau ldorau force-pushed the Add_tests_for_tracking_provider branch 3 times, most recently from 9bfe7b7 to 7051bb2 Compare February 28, 2025 20:54
Error out in case of double free() instead of assert
in coarse library.

Signed-off-by: Lukasz Dorau <[email protected]>
@ldorau ldorau force-pushed the Add_tests_for_tracking_provider branch from 7051bb2 to bf97c44 Compare March 1, 2025 10:41
@ldorau ldorau force-pushed the Add_tests_for_tracking_provider branch from bf97c44 to e75b939 Compare March 1, 2025 10:43
@ldorau
Copy link
Contributor Author

ldorau commented Mar 13, 2025

Incorporated into #1143

@ldorau ldorau closed this Mar 13, 2025
@ldorau ldorau deleted the Add_tests_for_tracking_provider branch March 13, 2025 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants