feat(files.add): update tests to new files add API

Author: daviddias

package.json

@@ -2,7 +2,7 @@
   "name": "interface-ipfs-core",
   "version": "0.7.2",
   "description": "A test suite and interface you can use to implement a IPFS core interface.",
-  "main": "lib/index.js",
+  "main": "src/index.js",
   "jsnext:main": "src/index.js",
   "scripts": {
     "test": "exit(0)",
@@ -47,4 +47,4 @@
     "greenkeeperio-bot <support@greenkeeper.io>",
     "nginnever <ginneversource@gmail.com>"
   ]
-}
+}

src/files.js

@@ -53,21 +53,27 @@ module.exports = (common) => {
       describe('.add', () => {
         it('stream', (done) => {
           const buffered = new Buffer('some data')
+          const expectedMultihash = 'QmVv4Wz46JaZJeH5PMV4LGbRiiMKEmszPYY3g6fjGnVXBS'
+
           const rs = new Readable()
           rs.push(buffered)
           rs.push(null)
 
           const arr = []
-          const filePair = {path: 'data.txt', content: rs}
+          const filePair = {
+            path: 'data.txt',
+            content: rs
+          }
+
           arr.push(filePair)
 
           ipfs.files.add(arr, (err, res) => {
             expect(err).to.not.exist
             expect(res).to.be.length(1)
-            expect(res[0].path).to.equal('data.txt')
-            expect(res[0].node.size()).to.equal(17)
-            const mh = 'QmVv4Wz46JaZJeH5PMV4LGbRiiMKEmszPYY3g6fjGnVXBS'
-            expect(bs58.encode(res[0].node.multihash()).toString()).to.equal(mh)
+            const file = res[0]
+            expect(file.path).to.equal('data.txt')
+            expect(file.size).to.equal(17)
+            expect(file.hash).to.equal(expectedMultihash)
             done()
           })
         })
@@ -77,41 +83,40 @@ module.exports = (common) => {
             path: 'testfile.txt',
             content: smallFile
           }
+          const expectedMultihash = 'Qma4hjFTnCasJ8PVp3mZbZK5g2vGDT4LByLJ7m8ciyRFZP'
 
           ipfs.files.add([file], (err, res) => {
             expect(err).to.not.exist
 
-            const added = res[0] != null ? res[0] : res
-            const mh = bs58.encode(added.node.multihash()).toString()
-            expect(mh).to.equal('Qma4hjFTnCasJ8PVp3mZbZK5g2vGDT4LByLJ7m8ciyRFZP')
-            expect(added.path).to.equal('testfile.txt')
-            expect(added.node.links).to.have.length(0)
+            const file = res[0]
+            expect(file.hash).to.equal(expectedMultihash)
+            expect(file.path).to.equal('testfile.txt')
             done()
           })
         })
 
         it('buffer', (done) => {
+          const expectedMultihash = 'Qma4hjFTnCasJ8PVp3mZbZK5g2vGDT4LByLJ7m8ciyRFZP'
+
           ipfs.files.add(smallFile, (err, res) => {
             expect(err).to.not.exist
-
             expect(res).to.have.length(1)
-            const mh = bs58.encode(res[0].node.multihash()).toString()
-            expect(mh).to.equal('Qma4hjFTnCasJ8PVp3mZbZK5g2vGDT4LByLJ7m8ciyRFZP')
-            expect(res[0].path).to.equal(mh)
-            expect(res[0].node.links).to.have.length(0)
+            const file = res[0]
+            expect(file.hash).to.equal(expectedMultihash)
+            expect(file.path).to.equal(file.hash)
             done()
           })
         })
 
         it('BIG buffer', (done) => {
+          const expectedMultihash = 'Qme79tX2bViL26vNjPsF3DP1R9rMKMvnPYJiKTTKPrXJjq'
+
           ipfs.files.add(bigFile, (err, res) => {
             expect(err).to.not.exist
-
             expect(res).to.have.length(1)
-            expect(res[0].node.links).to.have.length(58)
-            const mh = bs58.encode(res[0].node.multihash()).toString()
-            expect(res[0].path).to.equal(mh)
-            expect(mh).to.equal('Qme79tX2bViL26vNjPsF3DP1R9rMKMvnPYJiKTTKPrXJjq')
+            const file = res[0]
+            expect(file.hash).to.equal(expectedMultihash)
+            expect(file.path).to.equal(file.hash)
             done()
           })
         })
@@ -121,9 +126,13 @@ module.exports = (common) => {
             path: `test-folder/${name}`,
             content: directoryContent[name]
           })
+
           const emptyDir = (name) => ({
             path: `test-folder/${name}`
           })
+
+          const expectedRootMultihash = 'QmVvjDy7yF7hdnqE8Hrf4MHo5ABDtb5AbX6hWbD3Y42bXP'
+
           const dirs = [
             content('pp.txt'),
             content('holmes.txt'),
@@ -137,13 +146,10 @@ module.exports = (common) => {
 
           ipfs.files.add(dirs, (err, res) => {
             expect(err).to.not.exist
+            const root = res[res.length - 1]
 
-            const added = res[res.length - 1]
-            const mh = bs58.encode(added.node.multihash()).toString()
-            expect(added.node.links).to.have.length(6)
-            expect(added.path).to.equal('test-folder')
-            expect(mh).to.equal('QmVvjDy7yF7hdnqE8Hrf4MHo5ABDtb5AbX6hWbD3Y42bXP')
-
+            expect(root.path).to.equal('test-folder')
+            expect(root.hash).to.equal(expectedRootMultihash)
             done()
           })
         })
@@ -154,9 +160,13 @@ module.exports = (common) => {
               path: `test-folder/${name}`,
               content: directoryContent[name]
             })
+
             const emptyDir = (name) => ({
               path: `test-folder/${name}`
             })
+
+            const expectedRootMultihash = 'QmVvjDy7yF7hdnqE8Hrf4MHo5ABDtb5AbX6hWbD3Y42bXP'
+
             const files = [
               content('pp.txt'),
               content('holmes.txt'),
@@ -171,11 +181,9 @@ module.exports = (common) => {
             ipfs.files.createAddStream((err, stream) => {
               expect(err).to.not.exist
 
-              stream.on('data', (tuple) => {
-                if (tuple.path === 'test-folder') {
-                  const mh = bs58.encode(tuple.node.multihash()).toString()
-                  expect(mh).to.equal('QmVvjDy7yF7hdnqE8Hrf4MHo5ABDtb5AbX6hWbD3Y42bXP')
-                  expect(tuple.node.links).to.have.length(6)
+              stream.on('data', (file) => {
+                if (file.path === 'test-folder') {
+                  expect(file.hash).to.equal(expectedRootMultihash)
                 }
               })
 
@@ -232,14 +240,14 @@ module.exports = (common) => {
 
     describe('promise API', () => {
       describe('.add', () => {
+        const expectedMultihash = 'Qma4hjFTnCasJ8PVp3mZbZK5g2vGDT4LByLJ7m8ciyRFZP'
+
         it('buffer', () => {
           return ipfs.files.add(smallFile)
             .then((res) => {
-              const added = res[0] != null ? res[0] : res
-              const mh = bs58.encode(added.node.multihash()).toString()
-              expect(mh).to.equal('Qma4hjFTnCasJ8PVp3mZbZK5g2vGDT4LByLJ7m8ciyRFZP')
-              expect(added.path).to.equal(mh)
-              expect(added.node.links).to.have.length(0)
+              const file = res[0]
+              expect(file.hash).to.equal(expectedMultihash)
+              expect(file.path).to.equal(file.hash)
             })
         })
       })