diff --git a/Makefile b/Makefile index b1756f017..4f83d2867 100644 --- a/Makefile +++ b/Makefile @@ -51,7 +51,7 @@ $(MBGL)/config/%.gypi: $(MBGL) $(MBGL)/configure .PHONY: test test: build - @`npm bin`/mocha -R spec + @`npm bin`/tape test/**/*.test.js .PHONY: clean clean: diff --git a/package.json b/package.json index ff2566bb3..0f53fe51e 100644 --- a/package.json +++ b/package.json @@ -3,12 +3,12 @@ "version": "0.0.1", "main": "lib/mbgl.node", "dependencies": { - "mkdirp": "^0.5.0", - "mocha": "^2.1.0", - "nan": "^1.4.1", - "request": "^2.51.0" + "nan": "^1.4.1" + }, + "devDependencies": { + "mkdirp": "^0.5.0", + "tape": "^3.5.0" }, - "devDependencies": {}, "scripts": { "install": "make", "test": "make test" diff --git a/test/file_source.test.js b/test/file_source.test.js index 1d7d86fe9..90a91de97 100644 --- a/test/file_source.test.js +++ b/test/file_source.test.js @@ -1,19 +1,21 @@ /* jshint node: true, unused: false */ -/* global describe, it */ 'use strict'; -var assert = require('assert'); + +var test = require('tape').test; var mbgl = require('..'); -describe('FileSource', function() { - - describe('constructor', function() { - it('must be called with new', function() { - assert.throws(function() { +test('FileSource', function(t) { + t.test('constructor', function(t) { + t.test('must be called with new', function(t) { + t.throws(function() { mbgl.FileSource(); }, /Use the new operator to create new FileSource objects/); + t.end(); }); + t.end(); }); + t.end(); }); diff --git a/test/map.test.js b/test/map.test.js index 1d542c7b3..c61263694 100644 --- a/test/map.test.js +++ b/test/map.test.js @@ -1,7 +1,7 @@ /* jshint node: true, unused: false */ -/* global describe, it, beforeEach, afterEach */ 'use strict'; -var assert = require('assert'); + +var test = require('tape').test; var mbgl = require('..'); var fs = require('fs'); @@ -12,153 +12,176 @@ mkdirp.sync('test/results'); var style = require('./fixtures/style.json'); -describe('Map', function() { +function setup(fileSource, callback) { + callback(new mbgl.Map(fileSource)); +} - describe('constructor', function() { - it('must be called with new', function() { - assert.throws(function() { +test('Map', function(t) { + t.test('constructor', function(t) { + t.test('must be called with new', function(t) { + t.throws(function() { mbgl.Map(); }, /Use the new operator to create new Map objects/); + + t.end(); }); - it('should require a FileSource object as first parameter', function() { - assert.throws(function() { + t.test('should require a FileSource object as first parameter', function(t) { + t.throws(function() { new mbgl.Map(); }, /Requires a FileSource as first argument/); - assert.throws(function() { + t.throws(function() { new mbgl.Map('fileSource'); }, /Requires a FileSource as first argument/); - assert.throws(function() { + t.throws(function() { new mbgl.Map({}); }, /Requires a FileSource as first argument/); + + t.end(); }); - it('should require the FileSource object to have request and cancel methods', function() { + t.test('should require the FileSource object to have request and cancel methods', function(t) { var fileSource = new mbgl.FileSource(); - assert.throws(function() { + t.throws(function() { new mbgl.Map(fileSource); }, /FileSource must have a request member function/); fileSource.request = 'test'; - assert.throws(function() { + t.throws(function() { new mbgl.Map(fileSource); }, /FileSource must have a request member function/); fileSource.request = function() {}; - assert.throws(function() { + t.throws(function() { new mbgl.Map(fileSource); }, /FileSource must have a cancel member function/); fileSource.cancel = 'test'; - assert.throws(function() { + t.throws(function() { new mbgl.Map(fileSource); }, /FileSource must have a cancel member function/); fileSource.cancel = function() {}; - assert.doesNotThrow(function() { + t.doesNotThrow(function() { new mbgl.Map(fileSource); }); + + t.end(); }); + t.end(); }); - describe('load styles', function() { - var map; - + t.test('load styles', function(t) { var fileSource = new mbgl.FileSource(); fileSource.request = function() {}; fileSource.cancel = function() {}; - beforeEach(function() { - map = new mbgl.Map(fileSource); + t.test('requires a string or object as the first parameter', function(t) { + setup(fileSource, function(map) { + t.throws(function() { + map.load(); + }, /Requires a map style as first argument/); + + t.throws(function() { + map.load('invalid'); + }, /Expect either an object or array at root/); + t.end(); + }); }); - afterEach(function() { - map = null; + t.test('accepts an empty stylesheet string', function(t) { + setup(fileSource, function(map) { + t.doesNotThrow(function() { + map.load('{}'); + }); + t.end(); + }); }); - it('requires a string or object as the first parameter', function() { - assert.throws(function() { - map.load(); - }, /Requires a map style as first argument/); - - assert.throws(function() { - map.load('invalid'); - }, /Expect either an object or array at root/); + t.test('accepts a JSON stylesheet', function(t) { + setup(fileSource, function(map) { + t.doesNotThrow(function() { + map.load(style); + }); + t.end(); + }); }); - it('accepts an empty stylesheet string', function() { - map.load('{}'); + t.test('accepts a stringified stylesheet', function(t) { + setup(fileSource, function(map) { + t.doesNotThrow(function() { + map.load(JSON.stringify(style)); + }); + t.end(); + }); }); - it('accepts a JSON stylesheet', function() { - map.load(style); - }); - - it('accepts a stringified stylesheet', function() { - map.load(JSON.stringify(style)); - }); + t.end(); }); - describe('render argument requirements', function() { - var map; - + t.test('render argument requirements', function(t) { var fileSource = new mbgl.FileSource(); fileSource.request = function(req) { fs.readFile(path.join('test', req.url), function(err, data) { req.respond(err, { data: data }); - assert.ifError(err); + t.error(err); }); }; fileSource.cancel = function() {}; - beforeEach(function() { - map = new mbgl.Map(fileSource); - }); + t.test('requires an object as the first parameter', function(t) { + setup(fileSource, function(map) { + t.throws(function() { + map.render(); + }, /First argument must be an options object/); - afterEach(function() { - map = null; - }); + t.throws(function() { + map.render('invalid'); + }, /First argument must be an options object/); - it('requires an object as the first parameter', function() { - assert.throws(function() { - map.render(); - }, /First argument must be an options object/); - - assert.throws(function() { - map.render('invalid'); - }, /First argument must be an options object/); - }); - - it('requires a callback as the second parameter', function() { - assert.throws(function() { - map.render({}); - }, /Second argument must be a callback function/); - - assert.throws(function() { - map.render({}, 'invalid'); - }, /Second argument must be a callback function/); - }); - - it('requires a style to be set', function(done) { - map.render({}, function(err) { - assert.ok(err); - assert.equal(err.message, 'Style is not set'); - done(); + t.end(); }); }); - it('returns an image', function(done) { - map.load(style); - map.render({}, function(err, data) { - assert.ifError(err); - fs.writeFileSync('test/results/image.png', data); - done(); + t.test('requires a callback as the second parameter', function(t) { + setup(fileSource, function(map) { + t.throws(function() { + map.render({}); + }, /Second argument must be a callback function/); + + t.throws(function() { + map.render({}, 'invalid'); + }, /Second argument must be a callback function/); + + t.end(); + }); + }); + + t.test('requires a style to be set', function(t) { + setup(fileSource, function(map) { + map.render({}, function(err) { + t.ok(err); + t.equal(err.message, 'Style is not set'); + t.end(); + }); + }); + }); + + t.test('returns an image', function(t) { + setup(fileSource, function(map) { + map.load(style); + map.render({}, function(err, data) { + t.error(err); + fs.writeFileSync('test/results/image.png', data); + t.end(); + }); }); }); }); + t.end(); });