Setup dignified.js

Author: dignifiedquire

.travis.yml

@@ -0,0 +1,19 @@
+language: node_js
+node_js:
+  - '4'
+  - '5'
+
+before_install:
+  - npm i -g npm
+  # Workaround for a permissions issue with Travis virtual machine images
+
+addons:
+  firefox: 'latest'
+
+before_script:
+  - export DISPLAY=:99.0
+  - sh -e /etc/init.d/xvfb start
+
+script:
+  - npm run lint
+  - npm test

package.json

@@ -4,8 +4,12 @@
   "description": "JavaScript Implementation of Block and BlockService",
   "main": "src/index.js",
   "scripts": {
-    "test": "mocha tests/index.js",
-    "lint": "standard"
+    "lint": "dignified-lint",
+    "build": "dignified-build",
+    "test": "dignified-test",
+    "test:node": "dignified-test node",
+    "test:browser": "dignified-test browser",
+    "release": "dignified-release"
   },
   "pre-commit": [
     "lint",
@@ -30,17 +34,17 @@
   "devDependencies": {
     "bs58": "^3.0.0",
     "chai": "^3.5.0",
+    "dignified.js": "github:dignifiedquire/dignified.js",
     "fs-blob-store": "^5.2.1",
     "ipfs-repo": "^0.3.0",
-    "mocha": "^2.4.5",
     "ncp": "^2.0.0",
     "pre-commit": "^1.1.2",
-    "rimraf": "^2.5.1",
-    "standard": "^5.4.1"
+    "rimraf": "^2.5.1"
   },
   "dependencies": {
     "async": "^1.5.2",
     "bl": "^1.0.1",
+    "detect-node": "^2.0.3",
     "multihashing": "^0.2.0"
   }
 }

src/block-service.js

@@ -1,15 +1,14 @@
+'use strict'
 const Block = require('./block')
 const bl = require('bl')
 const async = require('async')
 
-module.exports = BlockService
-
 // BlockService is a hybrid block datastore. It stores data in a local
 // datastore and may retrieve data from a remote Exchange.
 // It uses an internal `datastore.Datastore` instance to store values.
 function BlockService (ipfsRepo, exchange) {
   this.addBlock = (block, callback) => {
-    var ws = ipfsRepo.datastore.createWriteStream(block.key)
+    const ws = ipfsRepo.datastore.createWriteStream(block.key)
     ws.write(block.data)
     ws.on('finish', callback)
     ws.end()
@@ -44,7 +43,7 @@ function BlockService (ipfsRepo, exchange) {
       return callback(new Error('Invalid batch of multihashes'))
     }
 
-    var blocks = []
+    const blocks = []
 
     async.each(multihashes, (multihash, next) => {
       this.getBlock(multihash, (err, block) => {
@@ -77,3 +76,4 @@ function BlockService (ipfsRepo, exchange) {
   }
 }
 
+module.exports = BlockService

src/block.js

@@ -1,14 +1,16 @@
+'use strict'
 const util = require('./util')
 
 // Immutable block of data
-
-module.exports = Block
-
 function Block (data) {
   if (!data) { throw new Error('Block must be constructed with data') }
 
-  if (!(this instanceof Block)) { return new Block(data) }
+  if (!(this instanceof Block)) {
+    return new Block(data)
+  }
 
   this.data = new Buffer(data)
   this.key = util.hash(this.data)
 }
+
+module.exports = Block

src/index.js

@@ -1,3 +1,5 @@
+'use strict'
+
 exports.BlockService = require('./block-service.js')
 exports.Block = require('./block.js')
 exports.util = require('./util.js')

src/util.js

@@ -1,7 +1,10 @@
-var multihashing = require('multihashing')
+'use strict'
+
+const multihashing = require('multihashing')
+const isNode = require('detect-node')
 
 exports = module.exports
 
 // Hash is the global IPFS hash function. uses multihash SHA2_256, 256 bits
-exports.hash = (data) => { return multihashing(data, 'sha2-256') }
-exports.isBrowser = () => { return !!global.window }
+exports.hash = (data) => multihashing(data, 'sha2-256')
+exports.isBrowser = () => !isNode

tests/block-service-test.js renamed to test/block-service-test.js

@@ -1,5 +1,4 @@
-/* globals describe, it */
-
+/* eslint-env mocha */
 'use strict'
 
 const expect = require('chai').expect
@@ -9,7 +8,7 @@ const BlockService = require('../src').BlockService
 const IPFSRepo = require('ipfs-repo')
 
 describe('block-service', () => {
-  var bs
+  let bs
 
   it('create a block-service', (done) => {
     const repo = new IPFSRepo(process.env.IPFS_PATH)

tests/block-test.js renamed to test/block-test.js

@@ -1,5 +1,4 @@
-/* globals describe, it */
-
+/* eslint-env mocha */
 'use strict'
 
 const expect = require('chai').expect
@@ -14,7 +13,7 @@ describe('block', () => {
   })
 
   it('fail to create an empty block', (done) => {
-    var b
+    let b
     try {
       b = new Block()
     } catch (err) {
@@ -34,7 +33,7 @@ describe('block', () => {
     // it from the original implementation
     // It doesn't stricly verify the immutability of the Block object
     const block = new Block("Can't change this!")
-    var key = block.key
+    let key = block.key
     key = new Buffer('new key')
 
     expect(key.equals(block.key)).to.equal(false)

test/browser.js

@@ -0,0 +1,6 @@
+/* eslint-env mocha */
+'use strict'
+
+describe.skip('blocks', () => {
+  it('works in the browser')
+})