Browse Source

code style tweaks

babel-plugin-for-integration-tests
Sindre Sorhus 9 years ago
parent
commit
6ae46ce004
  1. 13
      cli.js
  2. 2
      lib/babel.js
  3. 14
      lib/fork.js
  4. 1
      lib/logger.js
  5. 12
      lib/runner.js
  6. 2
      lib/test.js
  7. 1
      test/fixture/async-await.js
  8. 1
      test/fixture/generators.js
  9. 4
      test/fork.js
  10. 4
      test/test.js

13
cli.js

@ -8,9 +8,9 @@ var globby = require('globby');
var meow = require('meow'); var meow = require('meow');
var updateNotifier = require('update-notifier'); var updateNotifier = require('update-notifier');
var chalk = require('chalk'); var chalk = require('chalk');
var Promise = require('bluebird');
var fork = require('./lib/fork'); var fork = require('./lib/fork');
var log = require('./lib/logger'); var log = require('./lib/logger');
var Promise = require('bluebird');
// Bluebird specific // Bluebird specific
Promise.longStackTraces(); Promise.longStackTraces();
@ -37,7 +37,10 @@ var cli = meow({
] ]
}, { }, {
string: ['_'], string: ['_'],
boolean: ['fail-fast', 'serial'] boolean: [
'fail-fast',
'serial'
]
}); });
var fileCount = 0; var fileCount = 0;
@ -151,7 +154,7 @@ function init(files) {
return handlePaths(files) return handlePaths(files)
.map(function (file) { .map(function (file) {
return path.join(process.cwd(), file); return path.resolve('.', file);
}) })
.then(function (files) { .then(function (files) {
if (files.length === 0) { if (files.length === 0) {
@ -189,9 +192,7 @@ function handlePaths(files) {
return file; return file;
}) })
.then(function (files) { .then(flatten)
return flatten(files);
})
.filter(function (file) { .filter(function (file) {
return path.extname(file) === '.js' && path.basename(file)[0] !== '_'; return path.extname(file) === '.js' && path.basename(file)[0] !== '_';
}); });

2
lib/babel.js

@ -1,8 +1,6 @@
'use strict'; 'use strict';
var resolveFrom = require('resolve-from'); var resolveFrom = require('resolve-from');
var createEspowerPlugin = require('babel-plugin-espower/create'); var createEspowerPlugin = require('babel-plugin-espower/create');
var hasGenerators = parseInt(process.version.slice(1), 10) > 0; var hasGenerators = parseInt(process.version.slice(1), 10) > 0;
var path = process.argv[2]; var path = process.argv[2];

14
lib/fork.js

@ -1,23 +1,19 @@
'use strict'; 'use strict';
var childProcess = require('child_process'); var childProcess = require('child_process');
var Promise = require('bluebird'); var Promise = require('bluebird');
var dirname = require('path').dirname; var path = require('path');
var join = require('path').join;
module.exports = fork;
function fork(args) { module.exports = function (args) {
if (!Array.isArray(args)) { if (!Array.isArray(args)) {
args = [args]; args = [args];
} }
var babel = join(__dirname, 'babel.js'); var babel = path.join(__dirname, 'babel.js');
var file = args[0]; var file = args[0];
var options = { var options = {
silent: true, silent: true,
cwd: dirname(file) cwd: path.dirname(file)
}; };
var ps = childProcess.fork(babel, args, options); var ps = childProcess.fork(babel, args, options);
@ -58,4 +54,4 @@ function fork(args) {
}; };
return promise; return promise;
} };

1
lib/logger.js

@ -1,5 +1,4 @@
'use strict'; 'use strict';
var prettyMs = require('pretty-ms'); var prettyMs = require('pretty-ms');
var figures = require('figures'); var figures = require('figures');
var Squeak = require('squeak'); var Squeak = require('squeak');

12
lib/runner.js

@ -76,13 +76,11 @@ Runner.prototype.addAfterEachHook = function (title, cb) {
}; };
Runner.prototype._runTestWithHooks = function (test) { Runner.prototype._runTestWithHooks = function (test) {
var self = this; var beforeHooks = this.tests.beforeEach.map(function (hook) {
var beforeHooks = self.tests.beforeEach.map(function (hook) {
return new Test(hook.title, hook.fn); return new Test(hook.title, hook.fn);
}); });
var afterHooks = self.tests.afterEach.map(function (hook) { var afterHooks = this.tests.afterEach.map(function (hook) {
return new Test(hook.title, hook.fn); return new Test(hook.title, hook.fn);
}); });
@ -96,14 +94,12 @@ Runner.prototype._runTestWithHooks = function (test) {
}; };
Runner.prototype._runTest = function (test) { Runner.prototype._runTest = function (test) {
var self = this;
// add test result regardless of state // add test result regardless of state
// but on error, don't execute next tests // but on error, don't execute next tests
return test.run() return test.run()
.finally(function () { .finally(function () {
self._addTestResult(test); this._addTestResult(test);
}); }.bind(this));
}; };
Runner.prototype.concurrent = function (tests) { Runner.prototype.concurrent = function (tests) {

2
lib/test.js

@ -3,8 +3,8 @@ var isGenerator = require('is-generator').fn;
var Promise = require('bluebird'); var Promise = require('bluebird');
var setImmediate = require('set-immediate-shim'); var setImmediate = require('set-immediate-shim');
var fnName = require('fn-name'); var fnName = require('fn-name');
var assert = require('./assert');
var co = require('co'); var co = require('co');
var assert = require('./assert');
var TIMEOUT_MAX_VALUE = 2147483647; var TIMEOUT_MAX_VALUE = 2147483647;

1
test/fixture/async-await.js

@ -1,5 +1,4 @@
'use strict'; 'use strict';
const test = require('../../'); const test = require('../../');
test('async function', async function (t) { test('async function', async function (t) {

1
test/fixture/generators.js

@ -1,5 +1,4 @@
'use strict'; 'use strict';
const test = require('../../'); const test = require('../../');
test('generator function', function * (t) { test('generator function', function * (t) {

4
test/fork.js

@ -1,10 +1,10 @@
'use strict'; 'use strict';
var test = require('tape'); var test = require('tape');
var join = require('path').join; var path = require('path');
var fork = require('../lib/fork.js'); var fork = require('../lib/fork.js');
function fixture(name) { function fixture(name) {
return join(__dirname, 'fixture', name); return path.join(__dirname, 'fixture', name);
} }
test('emits test event', function (t) { test('emits test event', function (t) {

4
test/test.js

@ -3,7 +3,7 @@ var childProcess = require('child_process');
var Promise = require('bluebird'); var Promise = require('bluebird');
var figures = require('figures'); var figures = require('figures');
var test = require('tape'); var test = require('tape');
var join = require('path').join; var path = require('path');
var Runner = require('../lib/runner'); var Runner = require('../lib/runner');
var ava = require('../lib/test'); var ava = require('../lib/test');
@ -866,7 +866,7 @@ test('change process.cwd() to a test\'s directory', function (t) {
execCli('fixture/process-cwd.js', function (err, stdout) { execCli('fixture/process-cwd.js', function (err, stdout) {
t.ifError(err); t.ifError(err);
t.is(stdout.trim(), join(__dirname, 'fixture')); t.is(stdout.trim(), path.join(__dirname, 'fixture'));
t.end(); t.end();
}); });
}); });

Loading…
Cancel
Save