test: check this value for nextTick()

Depending on how many arguments are provided, `nextTick()` may run its callback with `this` set to `null` or not. Add assertions for both cases. PR-URL: #14645 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Refael Ackermann <[email protected]>
nodejs · Aug 10, 2017 · 0d3ef5b · 0d3ef5b
1 parent 2249234
commit 0d3ef5b
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/test/parallel/test-next-tick.js b/test/parallel/test-next-tick.js
@@ -21,6 +21,7 @@
 
 'use strict';
 const common = require('../common');
+
 const assert = require('assert');
 
 process.nextTick(common.mustCall(function() {
@@ -40,8 +41,23 @@ const obj = {};
 process.nextTick(function(a, b) {
   assert.strictEqual(a, 42);
   assert.strictEqual(b, obj);
+  assert.strictEqual(this, undefined);
 }, 42, obj);
 
+process.nextTick((a, b) => {
+  assert.strictEqual(a, 42);
+  assert.strictEqual(b, obj);
+  assert.deepStrictEqual(this, {});
+}, 42, obj);
+
+process.nextTick(function() {
+  assert.strictEqual(this, null);
+}, 1, 2, 3, 4);
+
+process.nextTick(() => {
+  assert.deepStrictEqual(this, {});
+}, 1, 2, 3, 4);
+
 process.on('exit', function() {
   process.nextTick(common.mustNotCall());
 });