fix: human readable option

package.json

@@ -57,12 +57,14 @@
   "dependencies": {
     "base32.js": "~0.1.0",
     "bignumber.js": "^9.0.0",
+    "bytes": "^3.1.0",
     "cids": "~0.7.0",
     "datastore-core": "~0.7.0",
     "datastore-fs": "~0.9.0",
     "datastore-level": "~0.12.0",
     "debug": "^4.1.0",
     "err-code": "^1.1.2",
+    "filesize": "^6.0.0",
     "interface-datastore": "~0.7.0",
     "ipfs-block": "~0.8.1",
     "ipfs-repo-migrations": "~0.1.0",

src/index.js

@@ -7,6 +7,8 @@ const debug = require('debug')
 const Big = require('bignumber.js')
 const errcode = require('err-code')
 const migrator = require('ipfs-repo-migrations')
+const filesize = require('filesize')
+const bytes = require('bytes')
 
 const constants = require('./constants')
 const backends = require('./backends')
@@ -258,19 +260,22 @@ class IpfsRepo {
       getSize(this.datastore),
       getSize(this.keys)
     ])
-    let size = blocks.size
+    const size = blocks.size
       .plus(datastore)
       .plus(keys)
 
-    if (options.human) {
-      size = size.div(1048576)
-    }
     return {
       repoPath: this.path,
-      storageMax: storageMax,
+      storageMax: options.human
+        ? filesize(storageMax, { exponent: 3 }) // exponent 3 specifies the symbol GB for base 2
+        : storageMax,
       version: version,
-      numObjects: blocks.count,
-      repoSize: size
+      numObjects: options.human
+        ? blocks.count.toNumber()
+        : blocks.count,
+      repoSize: options.human
+        ? filesize(size, { exponent: 2 }) // exponent 2 specifies the symbol MB for base 2
+        : size
     }
   }
 
@@ -308,7 +313,7 @@ class IpfsRepo {
   async _storageMaxStat () {
     try {
       const max = await this.config.get('Datastore.StorageMax')
-      return new Big(max)
+      return new Big(bytes(max))
     } catch (err) {
       return new Big(noLimit)
     }

test/repo-test.js

@@ -8,6 +8,7 @@ const path = require('path')
 const IPFSRepo = require('../')
 const Errors = require('../src/errors')
 const os = require('os')
+const bytes = require('bytes')
 
 module.exports = (repo) => {
   describe('IPFS Repo Tests', () => {
@@ -247,14 +248,17 @@ module.exports = (repo) => {
       })
 
       it('should return the max storage stat when set', async () => {
+        const maxStorage = '1GB'
+
         otherRepo = new IPFSRepo(path.join(os.tmpdir(), 'repo-' + Date.now()))
         await otherRepo.init({})
         await otherRepo.open()
-        await otherRepo.config.set('Datastore.StorageMax', 100)
+        await otherRepo.config.set('Datastore.StorageMax', maxStorage)
 
         const stat = await otherRepo.stat({})
 
-        expect(stat.storageMax.toNumber()).to.equal(100)
+        expect(stat).to.have.property('storageMax')
+        expect(stat.storageMax.toNumber()).to.equal(bytes(maxStorage))
       })
 
       it('should throw unexpected errors when closing', async () => {

test/stat-test.js

@@ -36,13 +36,12 @@ module.exports = (repo) => {
       const stats = await repo.stat({ human: true })
       expect(stats).to.exist()
       expect(stats).to.have.property('numObjects')
-      expect(stats).to.have.property('version')
+      expect(stats).to.have.property('version').and.be.above(0)
       expect(stats).to.have.property('repoPath')
-      expect(stats).to.have.property('repoSize')
-      expect(stats).to.have.property('storageMax')
+      expect(stats).to.have.property('repoSize').that.includes('MB')
+      expect(stats).to.have.property('storageMax').that.includes('GB')
 
       expect(stats.numObjects > '0').to.eql(true)
-      expect(stats.version > '0').to.eql(true)
       expect(stats.repoSize > '0').to.eql(true)
       expect(stats.storageMax > '0').to.eql(true)
     })