-
Notifications
You must be signed in to change notification settings - Fork 299
feat: add support for js-ipfs dag api and also some tests #957
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM!
expect(result).to.not.exist() | ||
expect(err.message).to.equal('ipfs-http-client is missing DAG resolver for "raw" multicodec') | ||
expect(err.message).to.equal('Missing IPLD format "git-raw"') | ||
done() | ||
}) | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can these be moved (and deduped) into interface-ipfs-core
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. This comes up a lot - what's the criteria on deciding where to put them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I missed this - I mostly view it as a place where we put core tests that are shared between js-ipfs
and js-ipfs-http-client
.
I'm waiting for these tests to pop up again in |
Made by @achingbrain for ipfs-inactive/js-ipfs-http-client#957 License: MIT Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
License: MIT Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
1fa9ced
to
1a72d9f
Compare
@achingbrain I've rebased and added the tests to |
Made by @achingbrain for ipfs-inactive/js-ipfs-http-client#957 License: MIT Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
License: MIT Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
Also support for loading raw blocks.