Commit 646f825
test: guarantee test runs in test-readline-keys
Use common.mustCall() to guarantee that test functions (created by a
factory function) are run.
PR-URL: #11023
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent d8eed12 commit 646f825
1 file changed
Lines changed: 9 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1,10 +1,10 @@ | |||
| 1 | 1 | 'use strict'; | |
| 2 | - require('../common'); | ||
| 3 | - var PassThrough = require('stream').PassThrough; | ||
| 4 | - var assert = require('assert'); | ||
| 5 | - var inherits = require('util').inherits; | ||
| 6 | - var extend = require('util')._extend; | ||
| 7 | - var Interface = require('readline').Interface; | ||
| 2 | + const common = require('../common'); | ||
| 3 | + const PassThrough = require('stream').PassThrough; | ||
| 4 | + const assert = require('assert'); | ||
| 5 | + const inherits = require('util').inherits; | ||
| 6 | + const extend = require('util')._extend; | ||
| 7 | + const Interface = require('readline').Interface; | ||
| 8 | 8 | ||
| 9 | 9 | ||
| 10 | 10 | function FakeInput() { | |
@@ -55,7 +55,7 @@ function addTest(sequences, expectedKeys) { | |||
| 55 | 55 | ||
| 56 | 56 | const addKeyIntervalTest = (sequences, expectedKeys, interval = 550, | |
| 57 | 57 | assertDelay = 550) => { | |
| 58 | - return (next) => () => { | ||
| 58 | + const fn = common.mustCall((next) => () => { | ||
| 59 | 59 | ||
| 60 | 60 | if (!Array.isArray(sequences)) { | |
| 61 | 61 | sequences = [ sequences ]; | |
@@ -84,7 +84,8 @@ const addKeyIntervalTest = (sequences, expectedKeys, interval = 550, | |||
| 84 | 84 | } | |
| 85 | 85 | }; | |
| 86 | 86 | emitKeys(sequences); | |
| 87 | - }; | ||
| 87 | + }); | ||
| 88 | + return fn; | ||
| 88 | 89 | }; | |
| 89 | 90 | ||
| 90 | 91 | // regular alphanumerics | |
0 commit comments