Skip to content
This repository has been archived by the owner on Dec 20, 2024. It is now read-only.

test: add unit test case for core.go #1260

Merged
merged 1 commit into from
Mar 24, 2020
Merged

Conversation

fenggw-fnst
Copy link
Contributor

Signed-off-by: Guangwen Feng [email protected]

Ⅰ. Describe what this PR did

Add unit test case for func calculateTimeout in core.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

go test github.com/dragonflyoss/Dragonfly/dfget/core -run Test -v

Ⅴ. Special notes for reviews

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 79fd076 into dragonflyoss:master Mar 24, 2020
@fenggw-fnst fenggw-fnst deleted the work branch March 25, 2020 10:00
sungjunyoung pushed a commit to sungjunyoung/Dragonfly that referenced this pull request May 8, 2022
…gonflyoss#1260)

* fix: client synchronizer report error lock
* chore: remove superfluous retry to sync and get pieces from other peers
* chore: optimize back source and sync piece logic
* chore: print apk output in e2e
* fix: grpc send with io.EOF error

Signed-off-by: Jim Ma <[email protected]>
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.

3 participants