Skip to content
This repository was archived by the owner on Feb 12, 2024. It is now read-only.

feat: dag get cli #688

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions src/cli/commands/dag.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
'use strict'

module.exports = {
command: 'dag',

description: 'Interact with ipld dag objects.',

builder (yargs) {
return yargs
.commandDir('dag')
},

handler (argv) {
}
}
43 changes: 43 additions & 0 deletions src/cli/commands/dag/get.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
'use strict'

const utils = require('../../utils')
const CID = require('cids')
const debug = require('debug')
const log = debug('cli:block')
log.error = debug('cli:block:error')

module.exports = {
command: 'get <ref>',

describe: 'Get a dag node from ipfs.',

builder: {},

handler (argv) {
utils.getIPFS((err, ipfs) => {
if (err) {
throw err
}

const refParts = argv.ref.split('/')
const cidString = refParts[0]
const pathString = refParts.slice(1).join('/')
const cid = new CID(cidString)

let lookupFn
if (pathString) {
lookupFn = () => ipfs.dag.resolve(cid, pathString)
} else {
lookupFn = () => ipfs.dag.get(cid)
}

lookupFn()
.then((obj) => {
console.log(obj)
})
.catch((err) => {
console.error(err)
})
})
}
}
18 changes: 18 additions & 0 deletions test/cli/test-dag.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
/* eslint-env mocha */
'use strict'

const expect = require('chai').expect
const repoPath = require('./index').repoPath
const describeOnlineAndOffline = require('../utils/on-and-off')
const ipfs = require('../utils/ipfs-exec')(repoPath)

describe('dag', () => {
describeOnlineAndOffline(repoPath, () => {
it('get', () => {
Copy link
Member

Choose a reason for hiding this comment

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

Should probably have a test-case as well when we try to resolve a hash that does not exists.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah I wanted to write more extensive tests but I couldn't even get them to run bc of some issue with a networking module

return ipfs('dag get z43AaGF23fmvRnDP56Ub9WcJCfzSfqtmzNCCvmz5eudT8dtdCDS/parentHash').then((out) => {
let expectHash = new Buffer('c8c0a17305adea9bbb4b98a52d44f0c1478f5c48fc4b64739ee805242501b256', 'hex')
expect(out).to.be.eql(expectHash)
})
})
})
})