delta/node-new.git/test/parallel/test-process-emit.js, branch Remove-Python-3-tests

test: use Number.isNaN()

2017-11-26T16:47:33+00:00

PR-URL: https://github.com/nodejs/node/pull/17319
Reviewed-By: Vse Mozhet Byt 
Reviewed-By: Michaël Zasso 
Reviewed-By: Daijiro Wachi 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Jon Moss 
Reviewed-By: Khaidi Chu 
Reviewed-By: James M Snell

node: set process._eventsCount to 0 on startup

2016-02-15T15:33:17+00:00

process is an EventEmitter. There are operations that increment and
decrement the _eventsCount property of an EventEmitter.
process._eventsCount would previously get set to NaN. This change makes
process._eventsCount be calculated as expected.

PR-URL: https://github.com/nodejs/node/pull/5208
Reviewed-By: Brian White 
Reviewed-By: James M Snell 
Reviewed-By: Ben Noordhuis

test: update arrow function style

2016-01-28T16:31:31+00:00

This commit applies new arrow function linting rules across the
codebase. As it turns out, the only offenders were in the test
directory.

PR-URL: https://github.com/nodejs/node/pull/4813
Reviewed-By: Rod Vagg 
Reviewed-By: Stephen Belanger 
Reviewed-By: James M Snell 
Reviewed-By: Sakthipriyan Vairamani 
Reviewed-By: Saúl Ibarra Corretgé 
Reviewed-By: Michaël Zasso 
Reviewed-By: Matteo Collina 
Reviewed-By: Evan Lucas 
Reviewed-By: Jeremiah Senkpiel 
Reviewed-By: Roman Reiss

process: support symbol events

2016-01-21T20:17:38+00:00

Event emitters support symbols as event names. The process object
assumes that the event name is a string, and examines the first
three characters to check for signals. This causes an exception
if the event name is a symbol. This commit ensures that the
event name is a string before trying to slice() it.

PR-URL: https://github.com/nodejs/node/pull/4798
Reviewed-By: Sam Roberts 
Reviewed-By: Michaël Zasso 
Reviewed-By: Wyatt Preul