- Oct 15, 2018
-
-
Ruy Adorno authored
Fixes usage of strictEqual arguments order in test/parallel/test-process-wrap.js Part of Node+JS Interactive 2018 Code & Learn PR-URL: https://github.com/nodejs/node/pull/23576 Reviewed-By:
Anna Henningsen <anna@addaleax.net> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By:
Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By:
Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
-
Yitong authored
PR-URL: https://github.com/nodejs/node/pull/23578 Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
-
Salman Shakeel authored
Fixed strictEqual() parameters order according to the documentation for the assertion. PR-URL: https://github.com/nodejs/node/pull/23579 Reviewed-By:
Anatoli Papirovski <apapirovski@mac.com> Reviewed-By:
-
Justin Lee authored
PR-URL: https://github.com/nodejs/node/pull/23583 Reviewed-By:
-
seantcoyote authored
There's no issue for this PR, it's a task at the Node+JS Code-n-Learn. PR-URL: https://github.com/nodejs/node/pull/23584 Reviewed-By:
-
blakehall authored
The argument order in the strictEqual check was in the wrong order. The first argument is now the actual value and the second argument is the expected value. PR-URL: https://github.com/nodejs/node/pull/23587 Reviewed-By:
-
Illescas, Ricardo authored
PR-URL: https://github.com/nodejs/node/pull/23589 Reviewed-By:
-
Jason Nutter authored
Usage of assert.strictEqual in test-net-server-listen-remove-callback.js incorrectly passes the expected value as the first argument and actual value as the second argument. PR-URL: https://github.com/nodejs/node/pull/23590 Reviewed-By:
-
Weijia Wang authored
PR-URL: https://github.com/nodejs/node/pull/22370 Refs: https://github.com/nodejs/node/issues/22160 Reviewed-By:
Denys Otrishko <shishugi@gmail.com> Reviewed-By:
Matteo Collina <matteo.collina@gmail.com> Reviewed-By:
-
Maki Toda authored
PR-URL: https://github.com/nodejs/node/pull/23548 Reviewed-By:
-
Clinton Pahl authored
The argument order in the strictEqual check against stderr is in the wrong order. The first argument is now the actual value and the second argument is the expected value. PR-URL: https://github.com/nodejs/node/pull/23552 Reviewed-By:
Gabriel Schulhof <gabriel.schulhof@intel.com> Reviewed-By:
-
Murtaza H authored
In test/parallel/test-fs-access.js PR-URL: https://github.com/nodejs/node/pull/23553 Reviewed-By:
-
Marcos Frony authored
The following files were not being ignored: deps/npm/node_modules/node-gyp/gyp/tools/.DS_Store deps/npm/node_modules/node-gyp/gyp/tools/Xcode/.DS_Store PR-URL: https://github.com/nodejs/node/pull/23554 Reviewed-By:
Hitesh Kanwathirtha <digitalinfinity@gmail.com> Reviewed-By:
Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By:
Refael Ackermann <refack@gmail.com>
-
Jackelin Herrera authored
PR-URL: https://github.com/nodejs/node/pull/23555 Reviewed-By:
-
peter authored
Added a minimalist benchmark test for the async-hooks. PR-URL: https://github.com/nodejs/node/pull/23556 Reviewed-By:
-
Victor Poriazov authored
Fixes the order of assert.strictEqual arguments. PR-URL: https://github.com/nodejs/node/pull/23558 Reviewed-By:
-
naris93 authored
PR-URL: https://github.com/nodejs/node/pull/23559 Reviewed-By:
Sam Ruby <rubys@intertwingly.net> Reviewed-By:
-
Joe Grosspietsch authored
PR-URL: https://github.com/nodejs/node/pull/23561 Reviewed-By:
-
Chris Bautista authored
PR-URL: https://github.com/nodejs/node/pull/23537 Reviewed-By:
-
Justin denBroeder authored
The assert.strictEqual function should have the actual value first and the expected value second to enable easier debugging of AssertionErrors. PR-URL: https://github.com/nodejs/node/pull/23538 Reviewed-By:
-
Ivan Lukasevych authored
PR-URL: https://github.com/nodejs/node/pull/23539 Reviewed-By:
-
Allan Zheng authored
PR-URL: https://github.com/nodejs/node/pull/23540 Reviewed-By:
-
David Xue authored
PR-URL: https://github.com/nodejs/node/pull/23541 Reviewed-By:
-
Erin Bush authored
PR-URL: https://github.com/nodejs/node/pull/23542 Reviewed-By:
-
Cody Hazelwood authored
Drop `Free` and `std::unique_ptr` in favor of Node's `MallocedBuffer` for `char[]` buffer memory mangement. PR-URL: https://github.com/nodejs/node/pull/23543 Reviewed-By:
Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
-
KelvinLawHF1 authored
PR-URL: https://github.com/nodejs/node/pull/23544 Reviewed-By:
-
Carl Richmond authored
PR-URL: https://github.com/nodejs/node/pull/23545 Reviewed-By:
-
Sean Healy authored
Refactor callback functions to modern arrow functions. Also, added `common.mustCall` to `online` callbacks. PR-URL: https://github.com/nodejs/node/pull/23546 Reviewed-By:
Сковорода Никита Андреевич <chalkerx@gmail.com> Reviewed-By:
-
Eli Itah authored
PR-URL: https://github.com/nodejs/node/pull/23547 Reviewed-By:
-
Oktavianus Ludiro authored
The previous code did not pass correct argument order to assert.strictEqual(). Before: First argument provided is expected value Second argument provided is actual value After: First argument provided is actual value Second argument provided is expected value PR-URL: https://github.com/nodejs/node/pull/23527 Reviewed-By:
-
Diana Lee authored
PR-URL: https://github.com/nodejs/node/pull/23528 Reviewed-By:
-
ssamuels0916 authored
PR-URL: https://github.com/nodejs/node/pull/23529 Reviewed-By:
-
jaxyz authored
PR-URL: https://github.com/nodejs/node/pull/23530 Reviewed-By:
-
nofwayy authored
PR-URL: https://github.com/nodejs/node/pull/23531 Reviewed-By:
-
Andrew J D McCann authored
PR-URL: https://github.com/nodejs/node/pull/23532 Reviewed-By:
-
Marcos Frony authored
Revert arguments for assert and ensure that the first argument is always the actual value being tested. PR-URL: https://github.com/nodejs/node/pull/23534 Reviewed-By:
-
MarianneDr authored
PR-URL: https://github.com/nodejs/node/pull/23535 Reviewed-By:
-
garrik.leonardo@gmail.com authored
One assertion on test-tls-connect-address-family.js was out fo date, here we are fixing it to match the signature of strictEqual method. PR-URL: https://github.com/nodejs/node/pull/23536 Reviewed-By:
-
Michael Chen authored
PR-URL: https://github.com/nodejs/node/pull/23510 Reviewed-By:
-
edgarzapeka authored
PR-URL: https://github.com/nodejs/node/pull/23511 Reviewed-By:
-
