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.

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

kernel: mailbox: Add some testcases for mailboxs #31012

Merged
merged 1 commit into from
Mar 2, 2021

Conversation

KangJianX
Copy link
Collaborator

Add some testcases to tesing different condition of mailbox
API. Check API robust in different input parameters is handled.
such as invoke API to get data with NULL input.

Signed-off-by: Jian Kang jianx.kang@intel.com

@KangJianX KangJianX added area: Kernel area: Tests Issues related to a particular existing or missing test labels Dec 27, 2020
@KangJianX KangJianX force-pushed the mailbox_coverage branch 2 times, most recently from ff1dfdd to f6154a5 Compare December 27, 2020 04:20
tests/kernel/mbox/mbox_api/src/test_mbox_api.c Outdated Show resolved Hide resolved
tests/kernel/mbox/mbox_api/src/test_mbox_api.c Outdated Show resolved Hide resolved
@KangJianX KangJianX force-pushed the mailbox_coverage branch 2 times, most recently from 8e19657 to cba3dd0 Compare December 28, 2020 05:28
@LeiW000
Copy link
Collaborator

LeiW000 commented Jan 11, 2021

@andyross , can it be merged now?

@enjiamai
Copy link
Collaborator

@KangJianX please rebase it and push again, thanks.

@KangJianX
Copy link
Collaborator Author

@enjiamai updated

@KangJianX KangJianX force-pushed the mailbox_coverage branch 2 times, most recently from 2c33ead to 8837fc2 Compare January 26, 2021 07:29
@KangJianX KangJianX requested a review from dcpleung as a code owner February 23, 2021 01:34
@LeiW000
Copy link
Collaborator

LeiW000 commented Feb 24, 2021

I think the PR can be merged now.

@enjiamai
Copy link
Collaborator

enjiamai commented Mar 1, 2021

@KangJianX Could you rebase it?

Add some testcases to tesing different condition of mailbox
API. Check API robust in different input parameters is handled.
such as invoke API to get data with NULL input.

Signed-off-by: Jian Kang <jianx.kang@intel.com>
@KangJianX
Copy link
Collaborator Author

@nashif @andyross Could this PR merge now?

@nashif nashif merged commit 088acda into zephyrproject-rtos:master Mar 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Kernel area: Tests Issues related to a particular existing or missing test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants