-
Notifications
You must be signed in to change notification settings - Fork 558
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
Request help: Supplementary of Unit Tests #257
Comments
i want to be in charge of |
thanks |
I want to do |
thanks |
I want to do |
@KevinChens thanks |
I want to do |
@L2ncE thanks |
I'd like to try |
thanks |
Can I try internal/bytesconv? |
thanks |
Can I try again and take charge of the |
thanks |
can i try |
thanks |
@ViolaPioggia PR is open~ |
Can I still try |
thanks |
Can I try |
Thanks |
pkg/common/utils can I hava a try? |
Thanks |
Can I try ? pkg/protocol/header.go |
I want to try pkg/protocol/header.go |
Sorry for not seeing the comments on the issue until now. |
thanks |
i want to try pkg/common/utils, thanks |
can i try pkg/route? |
please assign me pkg/common/utils |
I'd like to give adding unit tests for pkg/common/utils a try. Could I be assigned to this? |
@orthur2 just raise a pr and refer to this issue |
Issue Description
Write Unit Tests for some modules of Hertz, we mainly focus on Statement Coverage. The packages are shown below, you can choose and reply by comment.
Before you pick the task, please comply with the following requirements:
print
.Packages
pkg/app
: Current 70.87%, Target 80%, by @chaoranz758pkg/route
: Current 78.49%, Target 85%, bypkg/protocol/http1
: Current 33.94%, Target 80%, by @a631807682pkg/protocol/uri.go
: Current 70.35%, Target 80%, Raised to 86.4%, by @L2ncEpkg/protocol/args.go
: Current 56.57%, Target 80%, Raised to 81.3%, by @justlorainpkg/protocol/header.go
: Current 74.25%, Target 80%, by @XiaoYi-bytepkg/protocol/request.go
: Current 51.31%, Target 80%, Raised to 85.3%, by @BaiZe1998pkg/protocol/response.go
: Current 60.14%(Overall 56%), Target 80%, Raised to 81.8% by @gityh2021pkg/protocol/multipart.go
: Current 20.25%, Target 80%, Raised to 79.75%, by @gityh2021pkg/network
: Current 68.14%, Target 80%, Raised to 95.2%, by @a631807682pkg/common/utils
: Current 72.34%, Target 80%, bypkg/common/config
: Current 32.91%, Target 80%, Raised to 100%, by @rogerogerspkg/common/adaptor
: Current 76.60%, Target 85%, Raised to 91.5%, by @L2ncEinternal/bytesconv
: Current 1.68%, Target 80%, Raised to 82.35%, by @Alexdenerpkg/common/test
: Current 3.2%, Target 80%, Raised to 98.1% by @a631807682Standard coverage stats: https://app.codecov.io/gh/cloudwego/hertz
The text was updated successfully, but these errors were encountered: