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

tests: net: socket: websocket move to new ztest API #48224

Conversation

hunterhu3000
Copy link
Collaborator

@hunterhu3000 hunterhu3000 commented Jul 24, 2022

Move net/socket/websocket tests to use new ztest API

Signed-off-by: Hu Zhenyu zhenyu.hu@intel.com

rlubos
rlubos previously approved these changes Jul 25, 2022
enjiamai
enjiamai previously approved these changes Jul 25, 2022
tests/net/socket/websocket/src/main.c Outdated Show resolved Hide resolved
@hunterhu3000 hunterhu3000 dismissed stale reviews from enjiamai and rlubos via 70e0004 July 30, 2022 13:00
@hunterhu3000 hunterhu3000 changed the title test: Update the new ZTEST APIs tests/net/socket/websocket tests: net: socket: websocket move to new ztest API Jul 30, 2022
@hunterhu3000
Copy link
Collaborator Author

CONFIG_ZTEST_SHUFFLE=y can be turned on

rlubos
rlubos previously approved these changes Aug 1, 2022
enjiamai
enjiamai previously approved these changes Aug 14, 2022
Move net/socket/websocket tests to use new ztest API
TEST=twister -T tests/net/socket/websocket

Signed-off-by: Hu Zhenyu <zhenyu.hu@intel.com>
@hunterhu3000
Copy link
Collaborator Author

@enjiamai the numbers in the function name is not added by me, it was there before.
@nashif Thank to review and +1, so that the patch can be merged to mainline.

@hunterhu3000
Copy link
Collaborator Author

@nashif can we merge this PR?

@github-actions
Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Dec 20, 2022
@github-actions github-actions bot closed this Jan 3, 2023
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.

None yet

7 participants