Skip to content

Commit 94583b6

Browse files
committedDec 19, 2018
remove build system (closes #652)
1 parent 0e94034 commit 94583b6

File tree

5 files changed

+45
-89
lines changed

5 files changed

+45
-89
lines changed
 

‎.babelrc

-13
This file was deleted.

‎.travis.yml

+1
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ node_js:
66
- "6"
77
- "8"
88
- "10"
9+
- "11"
910

1011
install:
1112
- npm install

‎karma.conf.js

+17-26
Original file line numberDiff line numberDiff line change
@@ -1,32 +1,16 @@
1-
// Karma configuration
2-
// Generated on Fri Dec 16 2016 13:09:51 GMT+0000 (UTC)
3-
41
module.exports = function (config) {
52
config.set({
6-
7-
// Base path that will be used to resolve all patterns (eg. files, exclude)
8-
basePath: '',
9-
103
// Frameworks to use
114
// available frameworks: https://npmjs.org/browse/keyword/karma-adapter
12-
frameworks: ['mocha', 'chai'],
5+
frameworks: ['browserify', 'mocha'],
136

147
// List of files / patterns to load in the browser
158
files: [
16-
'dist/debug.js',
17-
'dist/test.js'
9+
'src/browser.js',
10+
'src/common.js',
11+
'test.js'
1812
],
1913

20-
// List of files to exclude
21-
exclude: [
22-
'src/node.js'
23-
],
24-
25-
// Preprocess matching files before serving them to the browser
26-
// available preprocessors: https://npmjs.org/browse/keyword/karma-preprocessor
27-
preprocessors: {
28-
},
29-
3014
// Test results reporter to use
3115
// possible values: 'dots', 'progress'
3216
// available reporters: https://npmjs.org/browse/keyword/karma-reporter
@@ -40,10 +24,7 @@ module.exports = function (config) {
4024

4125
// Level of logging
4226
// possible values: config.LOG_DISABLE || config.LOG_ERROR || config.LOG_WARN || config.LOG_INFO || config.LOG_DEBUG
43-
logLevel: config.LOG_INFO,
44-
45-
// Enable / disable watching file and executing tests whenever any file changes
46-
autoWatch: true,
27+
logLevel: config.LOG_DEBUG,
4728

4829
// Start these browsers
4930
// available browser launchers: https://npmjs.org/browse/keyword/karma-launcher
@@ -55,12 +36,22 @@ module.exports = function (config) {
5536
}
5637
},
5738

39+
preprocessors: {
40+
// *Sigh* what a glob, folks!
41+
'{{!(node_modules),*.js},!(node_modules)/**/*.js}': ['browserify']
42+
},
43+
44+
browserify: {
45+
debug: true,
46+
transform: ['brfs']
47+
},
48+
5849
// Continuous Integration mode
5950
// if true, Karma captures browsers, runs the tests and exits
60-
singleRun: false,
51+
singleRun: true,
6152

6253
// Concurrency level
6354
// how many browser should be started simultaneous
64-
concurrency: Infinity
55+
concurrency: 1
6556
});
6657
};

‎package.json

+6-19
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
],
1414
"files": [
1515
"src",
16-
"dist/debug.js",
1716
"LICENSE",
1817
"README.md"
1918
],
@@ -28,37 +27,25 @@
2827
"lint": "xo",
2928
"test": "npm run test:node && npm run test:browser",
3029
"test:node": "istanbul cover _mocha -- test.js",
31-
"posttest:node": "cat ./coverage/lcov.info | coveralls",
32-
"pretest:browser": "npm run build",
3330
"test:browser": "karma start --single-run",
34-
"prebuild:debug": "mkdir -p dist && browserify --standalone debug -o dist/debug.es6.js .",
35-
"build:debug": "babel -o dist/debug.js dist/debug.es6.js > dist/debug.js",
36-
"build:test": "babel -d dist test.js",
37-
"build": "npm run build:debug && npm run build:test",
38-
"clean": "rimraf dist coverage"
31+
"posttest:node": "cat ./coverage/lcov.info | coveralls"
3932
},
4033
"dependencies": {
4134
"ms": "^2.1.1"
4235
},
4336
"devDependencies": {
44-
"@babel/cli": "^7.0.0",
45-
"@babel/core": "^7.0.0",
46-
"@babel/preset-env": "^7.0.0",
47-
"browserify": "16.2.3",
48-
"chai": "^4.2.0",
49-
"concurrently": "^4.1.0",
37+
"brfs": "^2.0.1",
38+
"browserify": "^16.2.3",
5039
"coveralls": "^3.0.2",
5140
"istanbul": "^0.4.5",
5241
"karma": "^3.1.4",
53-
"karma-chai": "^0.1.0",
54-
"karma-mocha": "^1.3.0",
42+
"karma-browserify": "^6.0.0",
5543
"karma-chrome-launcher": "^2.2.0",
44+
"karma-mocha": "^1.3.0",
5645
"mocha": "^5.2.0",
5746
"mocha-lcov-reporter": "^1.2.0",
58-
"rimraf": "^2.5.4",
5947
"xo": "^0.23.0"
6048
},
6149
"main": "./src/index.js",
62-
"browser": "./src/browser.js",
63-
"unpkg": "./dist/debug.js"
50+
"browser": "./src/browser.js"
6451
}

‎test.js

+21-31
Original file line numberDiff line numberDiff line change
@@ -1,42 +1,32 @@
11
/* eslint-env mocha */
2-
'use strict';
32

4-
let chai;
5-
6-
let expect;
7-
8-
let debug;
9-
10-
if (typeof module !== 'undefined') {
11-
chai = require('chai');
12-
expect = chai.expect;
13-
debug = require('./src');
14-
}
3+
const assert = require('assert');
4+
const debug = require('./src');
155

166
describe('debug', () => {
177
it('passes a basic sanity check', () => {
188
const log = debug('test');
199
log.enabled = true;
2010
log.log = () => {};
2111

22-
expect(() => log('hello world')).to.not.throw();
12+
assert.doesNotThrow(() => log('hello world'));
2313
});
2414

2515
it('allows namespaces to be a non-string value', () => {
2616
const log = debug('test');
2717
log.enabled = true;
2818
log.log = () => {};
2919

30-
expect(() => debug.enable(true)).to.not.throw();
20+
assert.doesNotThrow(() => debug.enable(true));
3121
});
3222

3323
it('honors global debug namespace enable calls', () => {
34-
expect(debug('test:12345').enabled).to.equal(false);
35-
expect(debug('test:67890').enabled).to.equal(false);
24+
assert.deepStrictEqual(debug('test:12345').enabled, false);
25+
assert.deepStrictEqual(debug('test:67890').enabled, false);
3626

3727
debug.enable('test:12345');
38-
expect(debug('test:12345').enabled).to.equal(true);
39-
expect(debug('test:67890').enabled).to.equal(false);
28+
assert.deepStrictEqual(debug('test:12345').enabled, true);
29+
assert.deepStrictEqual(debug('test:67890').enabled, false);
4030
});
4131

4232
it('uses custom log function', () => {
@@ -50,7 +40,7 @@ describe('debug', () => {
5040
log('using custom log function again');
5141
log('%O', 12345);
5242

53-
expect(messages.length).to.equal(3);
43+
assert.deepStrictEqual(messages.length, 3);
5444
});
5545

5646
describe('extend namespace', () => {
@@ -60,7 +50,7 @@ describe('debug', () => {
6050
log.log = () => {};
6151

6252
const logBar = log.extend('bar');
63-
expect(logBar.namespace).to.be.equal('foo:bar');
53+
assert.deepStrictEqual(logBar.namespace, 'foo:bar');
6454
});
6555

6656
it('should extend namespace with custom delimiter', () => {
@@ -69,7 +59,7 @@ describe('debug', () => {
6959
log.log = () => {};
7060

7161
const logBar = log.extend('bar', '--');
72-
expect(logBar.namespace).to.be.equal('foo--bar');
62+
assert.deepStrictEqual(logBar.namespace, 'foo--bar');
7363
});
7464

7565
it('should extend namespace with empty delimiter', () => {
@@ -78,46 +68,46 @@ describe('debug', () => {
7868
log.log = () => {};
7969

8070
const logBar = log.extend('bar', '');
81-
expect(logBar.namespace).to.be.equal('foobar');
71+
assert.deepStrictEqual(logBar.namespace, 'foobar');
8272
});
8373
});
8474

8575
describe('rebuild namespaces string (disable)', () => {
8676
it('handle names, skips, and wildcards', () => {
8777
debug.enable('test,abc*,-abc');
8878
const namespaces = debug.disable();
89-
expect(namespaces).to.equal('test,abc*,-abc');
79+
assert.deepStrictEqual(namespaces, 'test,abc*,-abc');
9080
});
9181

9282
it('handles empty', () => {
9383
debug.enable('');
9484
const namespaces = debug.disable();
95-
expect(namespaces).to.equal('');
96-
expect(debug.names).to.deep.equal([]);
97-
expect(debug.skips).to.deep.equal([]);
85+
assert.deepStrictEqual(namespaces, '');
86+
assert.deepStrictEqual(debug.names, []);
87+
assert.deepStrictEqual(debug.skips, []);
9888
});
9989

10090
it('handles all', () => {
10191
debug.enable('*');
10292
const namespaces = debug.disable();
103-
expect(namespaces).to.equal('*');
93+
assert.deepStrictEqual(namespaces, '*');
10494
});
10595

10696
it('handles skip all', () => {
10797
debug.enable('-*');
10898
const namespaces = debug.disable();
109-
expect(namespaces).to.equal('-*');
99+
assert.deepStrictEqual(namespaces, '-*');
110100
});
111101

112102
it('names+skips same with new string', () => {
113103
debug.enable('test,abc*,-abc');
114104
const oldNames = [...debug.names];
115105
const oldSkips = [...debug.skips];
116106
const namespaces = debug.disable();
117-
expect(namespaces).to.equal('test,abc*,-abc');
107+
assert.deepStrictEqual(namespaces, 'test,abc*,-abc');
118108
debug.enable(namespaces);
119-
expect(oldNames.map(String)).to.deep.equal(debug.names.map(String));
120-
expect(oldSkips.map(String)).to.deep.equal(debug.skips.map(String));
109+
assert.deepStrictEqual(oldNames.map(String), debug.names.map(String));
110+
assert.deepStrictEqual(oldSkips.map(String), debug.skips.map(String));
121111
});
122112
});
123113
});

1 commit comments

Comments
 (1)

PolGirons commented on Jan 10, 2022

@PolGirons
This conversation has been locked and limited to collaborators.