-
Notifications
You must be signed in to change notification settings - Fork 3
tests/lapi: add coroutine test #135
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
Conversation
0142f42
to
f858c44
Compare
240aaba
to
c280aab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, Sergey!
Thanks for the patch!
Please consider my comments below.
Follows up ligurio/lua-c-api-tests#135 Related to tarantool/tarantool#11470
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, Sergey!
Looks like the branch isn't updated with changes.
44ffaa2
to
c06bcce
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fixes!
LGTM!
The patch bumps a luzer's version, full list of commits is in [1]. Notable changes are: - Added support LuaCov. - Added support Address and UndefinedBehaviour sanitizers. - Method `oneof()` in FuzzedDataProvider returns an item's index as a second value. - Fixed a memory leak in a Lua-based implementation of `TestOneInput()`. - An initial buffer size in FuzzedDataProvider. 1. ligurio/luzer@82d41c5...3f4eb03 Needed for #135
c06bcce
to
40663fe
Compare
The patch bumps a luzer's version, full list of commits is in [1]. Notable changes are: - Added support LuaCov. - Added support Address and UndefinedBehaviour sanitizers. - Method `oneof()` in FuzzedDataProvider returns an item's index as a second value. - Fixed a memory leak in a Lua-based implementation of `TestOneInput()`. - An initial buffer size in FuzzedDataProvider. 1. ligurio/luzer@82d41c5...3f4eb03 Needed for #135
The patch adds a fuzzing test for the coroutine library.
40663fe
to
fe276ef
Compare
Follows up ligurio/lua-c-api-tests#135 Related to tarantool/tarantool#11470
Follows up ligurio/lua-c-api-tests#135 Related to tarantool/tarantool#11470
The patch bumps a luzer's version, full list of commits is in [1]. Notable changes are: - Added support LuaCov. - Added support Address and UndefinedBehaviour sanitizers. - Method `oneof()` in FuzzedDataProvider returns an item's index as a second value. - Fixed a memory leak in a Lua-based implementation of `TestOneInput()`. - An initial buffer size in FuzzedDataProvider. 1. ligurio/luzer@82d41c5...3f4eb03 Needed for #135
Follows up ligurio/lua-c-api-tests#135 Related to tarantool/tarantool#11470
Follows up ligurio/lua-c-api-tests/pull/135 Related to tarantool/tarantool/issues/11401 Related to tarantool/tarantool/pull/11301
Depends on ligurio/luzer#57