go-ipld-format icon indicating copy to clipboard operation
go-ipld-format copied to clipboard
verified publisher icon ipfs

Import DAG reader tests involving `NodePromise`s to actually test `NavigableIPLDNode`

Open schomatis opened this issue 6 years ago • 1 comments

The node prefetching mechanism has been abstracted away from the DAG reader in go-unixfs into the NavigableIPLDNode here, so the PR https://github.com/ipfs/go-unixfs/pull/12 which employs the Walker (that relies on NavigableIPLDNode) had to remove the related tests that acted directly on the internal NodePromise structure,

https://github.com/ipfs/go-unixfs/pull/12/files#diff-5a645da404e1845b770b90d1c81e71c6

Those tests should be recreated here (possibly without involving the DAG reader now, just testing the FetchChild and related methods of NavigableIPLDNode to actually create the NodePromises following the spirit of the original test).

schomatis avatar Jan 18 '19 20:01 schomatis

Hi, @schomatis . I am interested in this work and reading the TestSeekAndReadLargeandTestReadAndCancel` in dagreader_test.go right now. Will come back here if there is some questions.

bjzhang avatar Jan 21 '19 12:01 bjzhang