Skip to content

Commit 32b85de

Browse files
committed
Update build and test scripts
1 parent 09c7bf2 commit 32b85de

File tree

5 files changed

+45
-24
lines changed

5 files changed

+45
-24
lines changed

.babelrc

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
11
{
2-
"presets": ["es2015", "stage-0"]
2+
"presets": ["es2015", "stage-0"],
3+
"env": {
4+
"test": {
5+
"plugins": [
6+
"istanbul"
7+
]
8+
}
9+
}
310
}

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
node_modules
2+
/.nyc_output
23
/lib
34
/coverage

.travis.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,3 @@ node_js:
66
- '4'
77
after_success:
88
- npm run coveralls
9-
- npm run coverage-clean

package.json

Lines changed: 29 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -17,25 +17,41 @@
1717
"gcode"
1818
],
1919
"scripts": {
20-
"prepublish": "npm run build && npm test",
21-
"build": "gulp",
22-
"test": "gulp test",
23-
"coverage": "istanbul cover node_modules/mocha/bin/_mocha -- --require babel-core/register -R spec",
24-
"coverage-clean": "rm -rf ./coverage",
20+
"prepublish": "npm run eslint && npm run build && npm test",
21+
"eslint": "eslint src",
22+
"build": "babel --out-dir ./lib ./src",
23+
"test": "cross-env NODE_ENV=test nyc mocha",
2524
"coveralls": "cat ./coverage/lcov.info | ./node_modules/.bin/coveralls"
2625
},
2726
"main": "lib/index.js",
2827
"dependencies": {},
2928
"devDependencies": {
30-
"babel-core": "^6.26.0",
29+
"babel-cli": "^6.26.0",
30+
"babel-eslint": "^8.0.1",
31+
"babel-plugin-istanbul": "^4.1.5",
3132
"babel-preset-es2015": "^6.24.1",
3233
"babel-preset-stage-0": "^6.24.1",
33-
"chai": "^3.5.0",
34-
"coveralls": "^2.13.2",
35-
"gulp": "^3.9.1",
36-
"gulp-babel": "^7.0.0",
37-
"gulp-istanbul": "^1.1.2",
38-
"gulp-mocha": "^4.3.1",
39-
"mocha": "^3.5.3"
34+
"babel-register": "^6.26.0",
35+
"chai": "^4.1.2",
36+
"coveralls": "^3.0.0",
37+
"cross-env": "^5.0.5",
38+
"eslint": "^4.7.2",
39+
"eslint-config-trendmicro": "^1.0.0",
40+
"eslint-plugin-import": "^2.7.0",
41+
"eslint-plugin-jsx-a11y": "^5.1.1",
42+
"eslint-plugin-react": "^7.4.0",
43+
"mocha": "^3.5.3",
44+
"nyc": "^11.2.1"
45+
},
46+
"nyc": {
47+
"require": [
48+
"babel-register"
49+
],
50+
"reporter": [
51+
"lcov",
52+
"text"
53+
],
54+
"sourceMap": false,
55+
"instrument": false
4056
}
4157
}

test/index.js

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import chai from 'chai';
1+
import { expect } from 'chai';
22
import fs from 'fs';
33
import {
44
GCodeParser,
@@ -8,28 +8,26 @@ import {
88
parseStringSync,
99
parseFile,
1010
parseFileSync
11-
} from '../lib';
12-
13-
const expect = chai.expect;
14-
const should = chai.should();
11+
} from '../src';
1512

1613
describe('gcode-parser', () => {
1714
describe('Pass a null value as the first argument', () => {
1815
it('should call parseString\'s callback.', (done) => {
1916
parseString(null, (err, results) => {
20-
expect(err).to.be.okay;
17+
expect(err).to.be.null;
18+
expect(results.length).to.equal(0);
2119
done();
2220
});
2321
});
2422
it('should call parseFile\'s callback.', (done) => {
2523
parseFile(null, (err, results) => {
26-
expect(err).to.be.okay;
24+
expect(!!err).to.be.true;
2725
done();
2826
});
2927
});
3028
it('should call parseStream\'s callback.', (done) => {
3129
parseStream(null, (err, results) => {
32-
expect(err).to.be.okay;
30+
expect(!!err).to.be.true;
3331
done();
3432
});
3533
});
@@ -39,7 +37,7 @@ describe('gcode-parser', () => {
3937
it('should get empty results.', (done) => {
4038
const sampleText = '';
4139
parseString(sampleText, (err, results) => {
42-
expect(results.length).to.be.empty;
40+
expect(results.length).to.equal(0);
4341
done();
4442
});
4543
});

0 commit comments

Comments
 (0)