Skip to content
This repository has been archived by the owner on Feb 27, 2023. It is now read-only.

test: add unit test case for func GetAsInt #1170

Merged
merged 1 commit into from
Jan 10, 2020
Merged

Conversation

hs0210
Copy link
Contributor

@hs0210 hs0210 commented Jan 10, 2020

Signed-off-by: Hu Shuai hus.fnst@cn.fujitsu.com

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗?)

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

Signed-off-by: Hu Shuai <hus.fnst@cn.fujitsu.com>
@codecov-io
Copy link

Codecov Report

Merging #1170 into master will increase coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1170      +/-   ##
==========================================
+ Coverage   47.22%   47.31%   +0.09%     
==========================================
  Files         113      113              
  Lines        7181     7181              
==========================================
+ Hits         3391     3398       +7     
+ Misses       3522     3513       -9     
- Partials      268      270       +2
Impacted Files Coverage Δ
pkg/syncmap/syncmap.go 31.95% <0%> (+7.21%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7845c18...2fe7047. Read the comment docs.

Copy link
Member

@lowzj lowzj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lowzj lowzj merged commit 9769c2c into dragonflyoss:master Jan 10, 2020
@hs0210 hs0210 deleted the work branch January 10, 2020 07:21
sungjunyoung pushed a commit to sungjunyoung/Dragonfly that referenced this pull request May 8, 2022
Signed-off-by: Gaius <gaius.qi@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants