Skip to content

Commit

Permalink
test: clean up repl-reset-event file
Browse files Browse the repository at this point in the history
* Change vars to let/const
* Add mustCall
* equal -> strictEqual
* remove timeout

PR-URL: #9931
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
courtnek authored and targos committed Dec 26, 2016
1 parent b8511ab commit 420b3b8
Showing 1 changed file with 22 additions and 25 deletions.
47 changes: 22 additions & 25 deletions test/parallel/test-repl-reset-event.js
Original file line number Diff line number Diff line change
@@ -1,52 +1,49 @@
'use strict';
var common = require('../common');
const common = require('../common');
common.globalCheck = false;

var assert = require('assert');
var repl = require('repl');
const assert = require('assert');
const repl = require('repl');

// Create a dummy stream that does nothing
const dummy = new common.ArrayStream();

function testReset(cb) {
var r = repl.start({
const r = repl.start({
input: dummy,
output: dummy,
useGlobal: false
});
r.context.foo = 42;
r.on('reset', function(context) {
r.on('reset', common.mustCall(function(context) {
assert(!!context, 'REPL did not emit a context with reset event');
assert.equal(context, r.context, 'REPL emitted incorrect context');
assert.equal(context.foo, undefined, 'REPL emitted the previous context' +
', and is not using global as context');
assert.strictEqual(context, r.context, 'REPL emitted incorrect context');
assert.strictEqual(
context.foo,
undefined,
'REPL emitted the previous context, and is not using global as context'
);
context.foo = 42;
cb();
});
}));
r.resetContext();
}

function testResetGlobal(cb) {
var r = repl.start({
function testResetGlobal() {
const r = repl.start({
input: dummy,
output: dummy,
useGlobal: true
});
r.context.foo = 42;
r.on('reset', function(context) {
assert.equal(context.foo, 42,
'"foo" property is missing from REPL using global as context');
cb();
});
r.on('reset', common.mustCall(function(context) {
assert.strictEqual(
context.foo,
42,
'"foo" property is missing from REPL using global as context'
);
}));
r.resetContext();
}

var timeout = setTimeout(function() {
common.fail('Timeout, REPL did not emit reset events');
}, 5000);

testReset(function() {
testResetGlobal(function() {
clearTimeout(timeout);
});
});
testReset(common.mustCall(testResetGlobal));

0 comments on commit 420b3b8

Please sign in to comment.