Skip to content
This repository has been archived by the owner on Oct 14, 2021. It is now read-only.

test: Add read with offset and size test case #63

Merged
merged 2 commits into from
Sep 29, 2021
Merged

test: Add read with offset and size test case #63

merged 2 commits into from
Sep 29, 2021

Conversation

JinnyYi
Copy link
Contributor

@JinnyYi JinnyYi commented Sep 29, 2021

fix #46

storager.go Outdated
@@ -77,6 +77,43 @@ func TestStorager(t *testing.T, store types.Storager) {
})
})

Convey("When Read a file with offset and size", func() {
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we also need tests for offset only and size only?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it.

@Xuanwo
Copy link
Contributor

Xuanwo commented Sep 29, 2021

Looks nice!

@Xuanwo Xuanwo merged commit 9456c45 into master Sep 29, 2021
@Xuanwo Xuanwo deleted the issue-46 branch September 29, 2021 03:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

All service should support read with offset and size
2 participants