Skip to content

Commit

Permalink
test: refactor test-repl-definecommand
Browse files Browse the repository at this point in the history
The test was writing to both REPL input and output but only checking
output. Sending output to both streams seems like it was an error.

PR-URL: #17795
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
Trott authored and gibfahn committed Jan 24, 2018
1 parent 69944e3 commit 81d063c
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions test/parallel/test-repl-definecommand.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,14 +23,14 @@ r.defineCommand('say1', {
help: 'help for say1',
action: function(thing) {
output = '';
this.write(`hello ${thing}`);
this.output.write(`hello ${thing}\n`);
this.displayPrompt();
}
});

r.defineCommand('say2', function() {
output = '';
this.write('hello from say2');
this.output.write('hello from say2\n');
this.displayPrompt();
});

Expand All @@ -39,6 +39,12 @@ assert(/\n\.say1 help for say1\n/.test(output),
'help for say1 not present');
assert(/\n\.say2\n/.test(output), 'help for say2 not present');
inputStream.write('.say1 node developer\n');
assert(/> hello node developer/.test(output), 'say1 outputted incorrectly');
assert.ok(output.startsWith('hello node developer\n'),
`say1 output starts incorrectly: "${output}"`);
assert.ok(output.includes('> '),
`say1 output does not include prompt: "${output}"`);
inputStream.write('.say2 node developer\n');
assert(/> hello from say2/.test(output), 'say2 outputted incorrectly');
assert.ok(output.startsWith('hello from say2\n'),
`say2 output starts incorrectly: "${output}"`);
assert.ok(output.includes('> '),
`say2 output does not include prompt: "${output}"`);

0 comments on commit 81d063c

Please sign in to comment.