From 2a2699f985cb535f529d6a2e32c5f041ad6215c3 Mon Sep 17 00:00:00 2001 From: Francisco Baio Dias Date: Thu, 28 Jan 2016 11:43:01 +0000 Subject: [PATCH 1/2] Use buffer-loader for tests --- package.json | 3 +-- tests/test-core/browser.js | 2 +- tests/test-core/test-block.js | 6 ++++-- 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/package.json b/package.json index 67fca4e721..a9cf32c707 100644 --- a/package.json +++ b/package.json @@ -35,8 +35,8 @@ "homepage": "https://github.com/ipfs/js-ipfs#readme", "devDependencies": { "async": "^1.5.2", - "binary-loader": "0.0.1", "brfs": "^1.4.3", + "buffer-loader": "0.0.1", "chai": "^3.4.1", "fs-blob-store": "^5.2.1", "idb-plus-blob-store": "^1.0.0", @@ -55,7 +55,6 @@ "ncp": "^2.0.0", "nexpect": "^0.5.0", "pre-commit": "^1.1.2", - "raw-loader": "^0.5.1", "rimraf": "^2.4.4", "standard": "^5.4.1", "transform-loader": "^0.2.3", diff --git a/tests/test-core/browser.js b/tests/test-core/browser.js index 3d03ee511a..aff5aa7833 100644 --- a/tests/test-core/browser.js +++ b/tests/test-core/browser.js @@ -4,7 +4,7 @@ const async = require('async') const store = require('idb-plus-blob-store') const _ = require('lodash') -const repoContext = require.context('raw!./../repo-example', true) +const repoContext = require.context('buffer!./../repo-example', true) const idb = window.indexedDB || window.mozIndexedDB || diff --git a/tests/test-core/test-block.js b/tests/test-core/test-block.js index b4f3b54de0..6ea044fc1d 100644 --- a/tests/test-core/test-block.js +++ b/tests/test-core/test-block.js @@ -10,9 +10,12 @@ const Block = require('ipfs-merkle-dag').Block const isNode = !global.window +// FIXME remove this (it is only here because otherwise webpack is not loading Buffer and it is needed to the tests bellow) +const noop = new Buffer([]) // eslint-disable-line + const fileA = isNode ? fs.readFileSync(process.cwd() + '/tests/repo-example/blocks/12207028/122070286b9afa6620a66f715c7020d68af3d10e1a497971629c07606bfdb812303d.data') - : new Buffer(require('binary!./../repo-example/blocks/12207028/122070286b9afa6620a66f715c7020d68af3d10e1a497971629c07606bfdb812303d.data'), 'binary') + : require('buffer!./../repo-example/blocks/12207028/122070286b9afa6620a66f715c7020d68af3d10e1a497971629c07606bfdb812303d.data') // console.log('=>', fileA) // console.log('=>', fileA.length) @@ -27,7 +30,6 @@ describe('block', () => { ipfs.block.get(mh, (err, block) => { expect(err).to.not.exist const eq = fileA.equals(block.data) - console.log(block) expect(eq).to.equal(true) done() }) From b97133731a3202660aed66fdd8e779cd19e119d2 Mon Sep 17 00:00:00 2001 From: Francisco Baio Dias Date: Thu, 28 Jan 2016 14:16:54 +0000 Subject: [PATCH 2/2] Use regular functions on tests that use Buffer --- tests/test-core/test-block.js | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/tests/test-core/test-block.js b/tests/test-core/test-block.js index 6ea044fc1d..a2c19b979b 100644 --- a/tests/test-core/test-block.js +++ b/tests/test-core/test-block.js @@ -10,20 +10,15 @@ const Block = require('ipfs-merkle-dag').Block const isNode = !global.window -// FIXME remove this (it is only here because otherwise webpack is not loading Buffer and it is needed to the tests bellow) -const noop = new Buffer([]) // eslint-disable-line - const fileA = isNode ? fs.readFileSync(process.cwd() + '/tests/repo-example/blocks/12207028/122070286b9afa6620a66f715c7020d68af3d10e1a497971629c07606bfdb812303d.data') : require('buffer!./../repo-example/blocks/12207028/122070286b9afa6620a66f715c7020d68af3d10e1a497971629c07606bfdb812303d.data') -// console.log('=>', fileA) -// console.log('=>', fileA.length) - -describe('block', () => { +// TODO use arrow funtions again when https://github.com/webpack/webpack/issues/1944 is fixed +describe('block', function () { var ipfs - it('get', done => { + it('get', function (done) { ipfs = new IPFS() const b58mh = 'QmVtU7ths96fMgZ8YSZAbKghyieq7AjxNdcqyVzxTt3qVe' const mh = new Buffer(base58.decode(b58mh)) @@ -64,7 +59,7 @@ describe('block', () => { }) }) - it('stat', done => { + it('stat', function (done) { const mh = new Buffer(base58 .decode('QmVtU7ths96fMgZ8YSZAbKghyieq7AjxNdcqyVzxTt3qVe')) ipfs.block.stat(mh, (err, stats) => {