-
Notifications
You must be signed in to change notification settings - Fork 950
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
doc: refine test doc #856
doc: refine test doc #856
Conversation
Thanks for your contribution. 🍻 @Letty5411 |
Codecov Report
@@ Coverage Diff @@
## master #856 +/- ##
=======================================
Coverage 14.89% 14.89%
=======================================
Files 114 114
Lines 6826 6826
=======================================
Hits 1017 1017
Misses 5723 5723
Partials 86 86 Continue to review full report at Codecov.
|
docs/test/test.md
Outdated
@@ -15,110 +15,136 @@ Test in pouch could be divided into following parts: | |||
* [unit testing](https://en.wikipedia.org/wiki/Unit_testing#Description) | |||
* [integration testing](https://en.wikipedia.org/wiki/Integration_testing) | |||
|
|||
More info could be gotten from the wiki page. | |||
*Unit testing* uses [go testing](https://golang.org/pkg/testing/) package, named with `_test.go` postfix and always locates in the same directory with the code tested. [client/client_test.go](https://github.com/alibaba/pouch/blob/master/client/client_test.go) is a good example of unit test. |
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.
s/postfix/suffix
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.
fix it.
Signed-off-by: letty <[email protected]>
LGTM |
Signed-off-by: letty [email protected]
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews