test: require handler to be run in sigwinch test

Use `common.mustCall()` to guarantee that the wrapped `_refreshSize()` functions are invoked. PR-URL: #11068 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Santiago Gimeno <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michal Zasso <[email protected]>
nodejs · Mar 9, 2017 · 50a868b · 50a868b
1 parent c1f45ec
commit 50a868b
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/test/pseudo-tty/test-stderr-stdout-handle-sigwinch.js b/test/pseudo-tty/test-stderr-stdout-handle-sigwinch.js
@@ -5,7 +5,7 @@ const originalRefreshSizeStderr = process.stderr._refreshSize;
 const originalRefreshSizeStdout = process.stdout._refreshSize;
 
 const wrap = (fn, ioStream, string) => {
-  return () => {
+  const wrapped = common.mustCall(() => {
     // The console.log() call prints a string that is in the .out file. In other
     // words, the console.log() is part of the test, not extraneous debugging.
     console.log(string);
@@ -16,7 +16,8 @@ const wrap = (fn, ioStream, string) => {
       if (!common.isSunOS || e.code !== 'EINVAL')
         throw e;
     }
-  };
+  });
+  return wrapped;
 };
 
 process.stderr._refreshSize = wrap(originalRefreshSizeStderr,