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

Add unit tests for WASM related code; 为wasm模块补充单测 #105

Closed
Tracked by #776
seeflood opened this issue Jul 2, 2021 · 10 comments
Closed
Tracked by #776

Add unit tests for WASM related code; 为wasm模块补充单测 #105

seeflood opened this issue Jul 2, 2021 · 10 comments
Assignees
Labels
area/wasm good first issue Good for newcomers help wanted Extra attention is needed kind/easy kind/enhancement New feature or request wasm WebAssembly technology

Comments

@seeflood
Copy link
Member

seeflood commented Jul 2, 2021

What would you like to be added:
Add unit tests for WASM related code:
understand the implementation of the wasm module and add unit tests for it. The unit test coverage of the module should reach 60%.
If you encounter any problems in the process, you can ask @zhenjunMa for help.
The code path is pkg/wasm/

By doing this improvement,you will learn that how Layotto supports multilingual programming based on WASM

chinese:
看明白wasm模块的实现原理并补充单测,使模块的单测覆盖率达到60%
过程中遇到问题可以问 @zhenjunMa
代码路径为 pkg/wasm/

做完这些优化后,你将学习到Layotto如何通过wasm支持多语言编程

Why is this needed:
Currently the test coverage of wasm is low:
image

@seeflood seeflood added kind/enhancement New feature or request help wanted Extra attention is needed labels Jul 2, 2021
@seeflood
Copy link
Member Author

seeflood commented Jul 2, 2021

@oneyue610
Copy link

OK

@seeflood
Copy link
Member Author

seeflood commented Jul 2, 2021

OK

Ok,already assigned.

@seeflood
Copy link
Member Author

@oneyue610 Hi, are you still working on it?

@seeflood
Copy link
Member Author

seeflood commented Aug 5, 2021

@oneyue610 Hi,are you still working on it?
Since you didn't response,I will reassign this task to someone else :<
If you still want to do this,just tell me and i will assign to u again

@Xunzhuo
Copy link
Member

Xunzhuo commented Aug 17, 2021

@assign

@seeflood seeflood added the wasm WebAssembly technology label Jun 4, 2022
@seeflood
Copy link
Member Author

seeflood commented Jun 15, 2022

@Xunzhuo Hi, shall we reassign it to others?
Since we are going to do a new round of activity, I think we can treat it as an activity task

@seeflood seeflood changed the title Add unit tests for WASM related code Add unit tests for WASM related code; 为wasm模块补充单测 Jun 15, 2022
@Xunzhuo
Copy link
Member

Xunzhuo commented Jun 16, 2022

Yes of course, @seeflood, maybe I should focus more on integration with istio

@ChloroplastYu
Copy link
Contributor

ChloroplastYu commented Oct 13, 2023

Hi, can you assign this issue to me? @zhenjunMa @seeflood

@Xunzhuo
Copy link
Member

Xunzhuo commented Oct 31, 2023

Closed by #1013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/wasm good first issue Good for newcomers help wanted Extra attention is needed kind/easy kind/enhancement New feature or request wasm WebAssembly technology
Projects
None yet
Development

No branches or pull requests

5 participants