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

feat(fsnode): add type helper #24

Merged
merged 2 commits into from
Oct 22, 2018
Merged

Conversation

overbool
Copy link
Contributor

@overbool overbool commented Oct 19, 2018

Addresses: #22

(edit: remove the "fixes" tag to avoid closing the issue, we still need to incorporate these functions in the MFS clients.)

mfs_test.go Outdated
}
}

func TestIsFile(t *testing.T) {
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: could we merge this two test functions? or include a helper test function that would abstract all the boiler plate code of creating an MFS to experiment with? (I feel like this should exist already somewhere in this file.)

Copy link
Contributor Author

@overbool overbool Oct 20, 2018

Choose a reason for hiding this comment

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

@schomatis I had merged two test function. However, I think it is not a completed job, because the mfs_test.go objective and test style is not clear. If you don't mind, maybe we could merge this pr firstly and then fix this issue (#9)?

@schomatis schomatis merged commit be09650 into ipfs:master Oct 22, 2018
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.

2 participants