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 static_locator.go #1374

Merged
merged 1 commit into from
Jun 9, 2020

Conversation

fenggw-fnst
Copy link
Contributor

Signed-off-by: Guangwen Feng fenggw-fnst@cn.fujitsu.com

Ⅰ. Describe what this PR did

Add unit test case for static_locator.go

Ⅱ. Does this pull request fix one issue?

NONE

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@codecov-commenter
Copy link

codecov-commenter commented Jun 2, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1374      +/-   ##
==========================================
+ Coverage   51.88%   51.98%   +0.10%     
==========================================
  Files         135      135              
  Lines        8803     8803              
==========================================
+ Hits         4567     4576       +9     
+ Misses       3859     3850       -9     
  Partials      377      377              
Impacted Files Coverage Δ
dfget/locator/static_locator.go 80.82% <0.00%> (+12.32%) ⬆️

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 5981693...b23a54f. Read the comment docs.

Copy link
Contributor

@zhouhaibing089 zhouhaibing089 left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
@fenggw-fnst
Copy link
Contributor Author

@zhouhaibing089 Thanks for the review, done, PTAL.

@zhouhaibing089
Copy link
Contributor

LGTM

@zhouhaibing089 zhouhaibing089 merged commit 862a6a7 into dragonflyoss:master Jun 9, 2020
@fenggw-fnst fenggw-fnst deleted the work branch June 10, 2020 05:03
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