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

k_heap APIs have no tests #29654

Closed
andrewboie opened this issue Oct 29, 2020 · 0 comments · Fixed by #29814
Closed

k_heap APIs have no tests #29654

andrewboie opened this issue Oct 29, 2020 · 0 comments · Fixed by #29814
Labels
bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@andrewboie
Copy link
Contributor

Describe the bug
We have tests for underlying sys_heap, but no tests for k_heap. Need test case coverage for all these APIs with full code coverage.

To Reproduce

cd tests
grep -nIRC3 k_heap_alloc
grep -nIRC3 k_heap_free

Expected behavior
Test case coverage of these functions.

Impact
Who knows if it actually works?
@andrewboie andrewboie added bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug labels Oct 29, 2020
@megatherium-95 megatherium-95 mentioned this issue Nov 5, 2020
Merged
@andyross andyross removed their assignment Nov 11, 2020
megatherium-95 added a commit to megatherium-95/zephyr that referenced this issue Nov 12, 2020
@megatherium-95
tests: k_heap api: add tests for k_heap_api
ee14d57 
Add test cases to test k_heap_alloc() and k_heap_free() APIs

Fixes zephyrproject-rtos#29654

Signed-off-by: iva kik <[email protected]>
nashif pushed a commit that referenced this issue Nov 12, 2020
@megatherium-95 @nashif
tests: k_heap api: add tests for k_heap_api
10fdf95 
Add test cases to test k_heap_alloc() and k_heap_free() APIs

Fixes #29654

Signed-off-by: iva kik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tests: k_heap api: add tests for k_heap_api megatherium-95/zephyr
2 participants
@andyross @andrewboie