Add promise tests and write test for files api

dignifiedquire · dignifiedquire · commit 9bfde1b88f88 · 2016-02-26T15:47:56.000+01:00
diff --git a/src/request-api.js b/src/request-api.js
@@ -36,7 +36,7 @@ function onRes (buffer, cb) {
         }
         if (payload) {
           error.code = payload.Code
-          error.message = payload.Message
+          error.message = payload.Message || payload.toString()
         }
         cb(error)
       })
diff --git a/test/api/files.spec.js b/test/api/files.spec.js
@@ -34,6 +34,30 @@ describe('.files', () => {
     })
   })
 
+  it('files.write', (done) => {
+    apiClients['a'].files
+      .write('/test-folder/test-file-2.txt', new Buffer('hello world'), {create: true}, (err) => {
+        expect(err).to.not.exist
+
+        apiClients['a'].files.read('/test-folder/test-file-2.txt', (err, stream) => {
+          expect(err).to.not.exist
+
+          let buf = ''
+          stream
+            .on('error', (err) => {
+              expect(err).to.not.exist
+            })
+            .on('data', (data) => {
+              buf += data
+            })
+            .on('end', () => {
+              expect(buf).to.be.equal('hello world')
+              done()
+            })
+        })
+      })
+  })
+
   it('files.stat', (done) => {
     apiClients['a'].files.stat('/test-folder/test-file', (err, res) => {
       expect(err).to.not.exist
@@ -89,4 +113,91 @@ describe('.files', () => {
       done()
     })
   })
+
+  describe('promise', () => {
+    it('files.mkdir', () => {
+      return apiClients['a'].files.mkdir('/test-folder')
+    })
+
+    it('files.cp', () => {
+      return apiClients['a'].files
+        .cp(['/ipfs/Qma4hjFTnCasJ8PVp3mZbZK5g2vGDT4LByLJ7m8ciyRFZP', '/test-folder/test-file'])
+    })
+
+    it('files.ls', () => {
+      return apiClients['a'].files.ls('/test-folder')
+        .then((res) => {
+          expect(res.Entries.length).to.equal(1)
+        })
+    })
+
+    it('files.write', (done) => {
+      return apiClients['a'].files
+        .write('/test-folder/test-file-2.txt', new Buffer('hello world'), {create: true})
+        .then(() => {
+          return apiClients['a'].files.read('/test-folder/test-file-2.txt')
+        })
+        .then((stream) => {
+          let buf = ''
+          stream
+            .on('error', (err) => {
+              expect(err).to.not.exist
+            })
+            .on('data', (data) => {
+              buf += data
+            })
+            .on('end', () => {
+              expect(buf).to.be.equal('hello world')
+              done()
+            })
+        })
+    })
+
+    it('files.stat', () => {
+      return apiClients['a'].files.stat('/test-folder/test-file')
+        .then((res) => {
+          expect(res).to.deep.equal({
+            Hash: 'Qma4hjFTnCasJ8PVp3mZbZK5g2vGDT4LByLJ7m8ciyRFZP',
+            Size: 12,
+            CumulativeSize: 20,
+            Blocks: 0,
+            Type: 'file'
+          })
+        })
+    })
+
+    it('files.stat file that does not exist', () => {
+      return apiClients['a'].files.stat('/test-folder/does-not-exist')
+        .catch((err) => {
+          expect(err).to.exist
+          expect(err.code).to.be.eql(0)
+        })
+    })
+
+    it('files.read', (done) => {
+      if (!isNode) {
+        return done()
+      }
+
+      apiClients['a'].files.read('/test-folder/test-file')
+        .then((stream) => {
+          let buf = ''
+          stream
+            .on('error', (err) => {
+              expect(err).to.not.exist
+            })
+            .on('data', (data) => {
+              buf += data
+            })
+            .on('end', () => {
+              expect(new Buffer(buf)).to.deep.equal(testfile)
+              done()
+            })
+        })
+    })
+
+    it('files.rm', () => {
+      return apiClients['a'].files.rm('/test-folder', { 'recursive': true })
+    })
+  })
 })

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ function onRes (buffer, cb) {`
`36`	`36`	`}`
`37`	`37`	`if (payload) {`
`38`	`38`	`error.code = payload.Code`
`39`		`- error.message = payload.Message`
	`39`	`+ error.message = payload.Message \|\| payload.toString()`
`40`	`40`	`}`
`41`	`41`	`cb(error)`
`42`	`42`	`})`