summaryrefslogtreecommitdiff
path: root/test/known_issues
Commit message (Collapse)AuthorAgeFilesLines
* test: remove stale entry from known_issues.statusRichard Lau2023-04-081-3/+0
| | | | | | | | | | | The `test-crypto-authenticated-stream` test was moved out of `test/known_issues` and now lives in `test/parallel` PR-URL: https://github.com/nodejs/node/pull/47454 Refs: https://github.com/nodejs/node/pull/33981 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
* test: move test-shadow-realm-gc.js to known_issuesJoyee Cheung2023-04-042-0/+16
| | | | | | | | | | | | | There is actually a leak. The test doesn't exercise the right path to create a substantial enough object graph (e.g. accessing something that results in the loading of a binding). This does something more complicated in the test and moves it to known_issues until we find a fix. PR-URL: https://github.com/nodejs/node/pull/47355 Refs: https://github.com/nodejs/node/issues/47353 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
* Revert "src: let http2 streams end after session close"Rich Trott2023-02-211-0/+54
| | | | | | | | | | | | This reverts commit dee882e94f4caa4e1cd608013f90f6a14629403f. Moved the test that demonstrated what this commit was fixing to the `known_issues` folder. Fixes: https://github.com/nodejs/node/issues/46234 PR-URL: https://github.com/nodejs/node/pull/46721 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Debadree Chatterjee <debadree333@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com>
* test: add trailing commas in `test/known_issues`Antoine du Hamel2023-02-0110-14/+14
| | | | | PR-URL: https://github.com/nodejs/node/pull/46408 Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
* readline: fix detection of carriage returnAntoine du Hamel2023-01-241-42/+0
| | | | | | | | | Fixes: https://github.com/nodejs/node/issues/45992 PR-URL: https://github.com/nodejs/node/pull/46306 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
* test: add failing test for readline with carriage returnAlec Mev2023-01-221-0/+42
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/46075 Refs: https://github.com/nodejs/node/issues/45992 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
* test: remove unnecessary noop function args to `mustCall()`Antoine du Hamel2022-10-201-2/+1
| | | | | | | | | | RefsL https://github.com/nodejs/node/pull/45027 PR-URL: https://github.com/nodejs/node/pull/45047 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Zeyu "Alex" Yang <himself65@outlook.com>
* lib: enable global WebCrypto by defaultAntoine du Hamel2022-09-171-0/+21
| | | | | | | | | | | Enables `--experimental-global-webcrypto` by default, and ensures that the classic `node:crypto` core module is still available in `--eval` or `--print` contexts. PR-URL: https://github.com/nodejs/node/pull/42083 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* debugger: prevent simultaneous heap snapshotsRich Trott2021-08-051-48/+0
| | | | | | | | | | | | Fixes: https://github.com/nodejs/node/issues/39555 PR-URL: https://github.com/nodejs/node/pull/39638 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* test: add known issues test for debugger heap snapshot raceRich Trott2021-08-011-0/+48
| | | | | | | | Refs: https://github.com/nodejs/node/issues/39555 PR-URL: https://github.com/nodejs/node/pull/39557 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* debugger: indicate server is endingRich Trott2021-07-121-57/+0
| | | | | | | | | | | Currently, we say "listening" when we are ending the server. Change it to "ending". Fixes: https://github.com/nodejs/node/issues/39272 PR-URL: https://github.com/nodejs/node/pull/39334 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Yash Ladha <yash@yashladha.in>
* test: add test for debugger restart message issueRich Trott2021-07-071-0/+57
| | | | | | | | | | | Running "restart" in the debugger confusingly prints an out-of-date "Debugger listening on..." message before printing a second updated one. Refs: https://github.com/nodejs/node/issues/39272 PR-URL: https://github.com/nodejs/node/pull/39273 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* path: fix posix.relative() on WindowsRich Trott2021-04-022-24/+0
| | | | | | | Fixes: https://github.com/nodejs/node/issues/13683 PR-URL: https://github.com/nodejs/node/pull/37747 Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
* test: add known_issues test for #13683Rich Trott2021-03-172-0/+26
| | | | | | | | | | Add a known_issues test for a known Windows issue. Refs: https://github.com/nodejs/node/issues/13683 PR-URL: https://github.com/nodejs/node/pull/37744 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com>
* lib,src: update cluster to use ParentMichael Dawson2021-01-052-3/+3
| | | | | | | | | | | | | | | Doc deprecate isMaster and setupMaster in favor of isPrimary and setupPrimary. Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: https://github.com/nodejs/node/pull/36478 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Beth Griggs <bgriggs@redhat.com>
* fs: rimraf should not recurse on failurebcoe2020-10-132-33/+1
| | | | | | PR-URL: https://github.com/nodejs/node/pull/35566 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
* vm: add tests for function declarations using [[DefineOwnProperty]]ExE Boss2020-07-161-0/+21
| | | | | | | | Refs: https://github.com/nodejs/node/issues/31808 PR-URL: https://github.com/nodejs/node/pull/34032 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Gus Caplan <me@gus.host>
* test: add issue ref for known_issues testRich Trott2020-07-111-0/+1
| | | | | | PR-URL: https://github.com/nodejs/node/pull/34267 Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: James M Snell <jasnell@gmail.com>
* test: add known issue for fs.open() keeping event loop openRich Trott2020-07-082-0/+32
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/34228 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Robert Nagy <ronagy@icloud.com>
* vm: add run-after-evaluate microtask modeAnna Henningsen2020-06-263-43/+1
| | | | | | | | | | | | This allows timeouts to apply to e.g. `Promise`s and `async function`s from code running inside of `vm.Context`s, by giving the Context its own microtasks queue. Fixes: https://github.com/nodejs/node/issues/3020 PR-URL: https://github.com/nodejs/node/pull/34023 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Denys Otrishko <shishugi@gmail.com>
* fs: remove custom Buffer pool for streamsRobert Nagy2020-06-231-142/+0
| | | | | | | | | | | | | | The performance benefit of using a custom pool are negligable. Furthermore, it causes problems with Workers and transferrable. Rather than further adding complexity for compat with Workers, just remove the pooling logic. Refs: https://github.com/nodejs/node/issues/33880#issuecomment-644430693 Fixes: https://github.com/nodejs/node/issues/31733 PR-URL: https://github.com/nodejs/node/pull/33981 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* test: skip crypto test on arm buildbotsBen Noordhuis2020-04-041-0/+3
| | | | | | | | | | | | | | | The Raspberry Pis are too slow to run this computationally expensive test in a timely manner, it times out after 240 seconds. PR-URL: https://github.com/nodejs/node/pull/32636 Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Matheus Marchini <mat@mmarchini.me> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com>
* test: add known issues test for #31733Ben Noordhuis2020-04-021-0/+142
| | | | | | | | | | | | Authenticated decryption works for file streams up to 32768 bytes but not beyond. Other streams and direct decryption are not affected. Refs: https://github.com/nodejs/node/issues/31733 PR-URL: https://github.com/nodejs/node/pull/31734 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
* test: always skip vm-timeout-escape-queuemicrotaskDenys Otrishko2020-03-101-0/+5
| | | | | | | | | | | | | | This issue is unfixable with the current implementations of V8 and Node.js. Changes to make this fixable are unlikely to happen in the foreseeable future. The test itself is flaky and skipped. It serves as a demonstration of the issue only. PR-URL: https://github.com/nodejs/node/pull/31980 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Matheus Marchini <mat@mmarchini.me> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
* test: increase timeout in vm-timeout-escape-queuemicrotaskDenys Otrishko2020-02-261-2/+2
| | | | | | | | | | | | | | It looks like under high load the loop isn't even started and therefore successfully finishes without 'escaping'. After increasing the timeout during parallel run of the test failure rate decreased from 15/1000 to 0/1000. PR-URL: https://github.com/nodejs/node/pull/31966 Refs: https://github.com/nodejs/node/issues/25529 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
* test: add known issue test for sync writable callbackJames M Snell2020-02-171-0/+44
| | | | | | | | | | | | | | | If the write callbacks are invoked synchronously with an error, onwriteError would cause the error event to be emitted synchronously, making it impossible to attach an error handler after the call that triggered it. PR-URL: https://github.com/nodejs/node/pull/31756 Refs: https://github.com/nodejs/quic/commit/b0d469c69c49c9186c1a581a7cebce4c5d398947 Refs: https://github.com/nodejs/quic/pull/341 Reviewed-By: Robert Nagy <ronagy@icloud.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>
* test: add test-fs-writeFileSync-invalid-windowsRich Trott2019-07-111-0/+21
| | | | | | | | | | | | | Add a known_issues test for the Windows returning ENOTFOUND where EINVAL is more appropriate. This happens with various functions in the `fs` module when an invalid path is used. Refs: https://github.com/nodejs/node/issues/8987 PR-URL: https://github.com/nodejs/node/pull/28569 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
* test: fix flaky test-vm-timeout-escape-nexttickRich Trott2019-07-022-15/+17
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/28461 Fixes: https://github.com/nodejs/node/issues/24120 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* test: add github refs to flaky testsSam Roberts2019-06-101-0/+2
| | | | | | | | | PR-URL: https://github.com/nodejs/node/pull/28123 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* test: unskip copyfile permission testcjihrig2019-04-221-6/+0
| | | | | | | | | | | | This is no longer a flakey test, and should run everywhere. PR-URL: https://github.com/nodejs/node/pull/27241 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* test: move known issue test to parallelcjihrig2019-04-221-52/+0
| | | | | | | | | | | | | | As of libuv 1.28.0, this bug is fixed, and the test can be moved to parallel. This commit also updates an error code check to work on Windows. PR-URL: https://github.com/nodejs/node/pull/27241 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* test: fix error code typocjihrig2019-04-011-3/+5
| | | | | | | | | | | PR-URL: https://github.com/nodejs/node/pull/27024 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* test: add known_issues test for fs.copyFile()Rich Trott2019-03-292-0/+56
| | | | | | | | | | | | | | | | On macOS, fs.copyFile() may not respect file permissions. There is a PR for libuv that should fix this, but until it lands and we can either float a patch or upgrade libuv, have a known_issues test. Ref: https://github.com/nodejs/node/issues/26936 Ref: https://github.com/libuv/libuv/pull/2233 PR-URL: https://github.com/nodejs/node/pull/26939 Refs: https://github.com/nodejs/node/issues/26936 Refs: https://github.com/libuv/libuv/pull/2233 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
* test: remove test-path-parse-6229.js from known issuesRuben Bridgewater2019-03-291-34/+0
| | | | | | | | | | | This test precedented the official documentation that states that this is an expected behavior. PR-URL: https://github.com/nodejs/node/pull/26913 Refs: https://github.com/nodejs/node/issues/6229 Refs: https://github.com/nodejs/node/pull/12181 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* benchmark,doc,lib: capitalize more commentsRuben Bridgewater2019-03-273-3/+3
| | | | | | PR-URL: https://github.com/nodejs/node/pull/26849 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
* benchmark,doc,lib,test: capitalize commentsRuben Bridgewater2019-03-102-4/+4
| | | | | | PR-URL: https://github.com/nodejs/node/pull/26483 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* benchmark,doc,lib,test: capitalize commentsRuben Bridgewater2019-02-282-3/+3
| | | | | | | | | | This updates a lot of comments. PR-URL: https://github.com/nodejs/node/pull/26223 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
* process: expose process.features.inspectorJoyee Cheung2019-02-021-1/+1
| | | | | | | | | | | | | | Instead of using process.config.variables.v8_enable_inspector to detect whether inspector is enabled in the build. PR-URL: https://github.com/nodejs/node/pull/25819 Refs: https://github.com/nodejs/node/issues/25343 Reviewed-By: Eugene Ostroukhov <eostroukhov@google.com> Reviewed-By: Gus Caplan <me@gus.host> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>
* test: remove known_issues/test-cluster-bind-privileged-portRich Trott2019-01-241-25/+0
| | | | | | | | | | | | The test was added to check for a bug in macOS Mojave, but it turns out the issue is a macOS feature, not a bug. Refs: https://github.com/nodejs/node/issues/21679#issuecomment-456589386 Fixes: https://github.com/nodejs/node/issues/21679 PR-URL: https://github.com/nodejs/node/pull/25649 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gus Caplan <me@gus.host>
* test: improve known_issues/test-vm-timeout-escape-queuemicrotaskRich Trott2019-01-151-4/+7
| | | | | | | | | | | | Improve known_issues/test-vm-timeout-escape-queuemicrotask to mitigate CI failures on ubuntu1604-arm64. Failures are due to a race condition. Use `common.platformTimeout()` to help, adjust timeout to make sure `queueMicrotasks()` has a chance to run, and improve error message. PR-URL: https://github.com/nodejs/node/pull/25503 Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Ben Coe <bencoe@gmail.com>
* tools: capitalize sentencesRuben Bridgewater2018-12-101-1/+1
| | | | | | | | | | | | This adds the `capitalized-comments` eslint rule to verify that actual sentences use capital letters as starting letters. It ignores special words and all lines below 62 characters. PR-URL: https://github.com/nodejs/node/pull/24808 Reviewed-By: Sam Ruby <rubys@intertwingly.net> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
* test: mark test-vm-timeout-escape-nexttick flakyGireesh Punathil2018-11-291-0/+4
| | | | | | | | | | | This is a known failure so mark it such, so that CI is green / amber while the issue is being progressed. PR-URL: https://github.com/nodejs/node/pull/24712 Refs: https://github.com/nodejs/node/pull/24620 Refs: https://github.com/nodejs/node/issues/24120 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
* test: replcae anonymous closure with arrow functionSarath Govind K K2018-11-221-2/+2
| | | | | | | | | PR-URL: https://github.com/nodejs/node/pull/24476 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
* test: fix flaky test-vm-timeout-escape-queuemicrotaskRich Trott2018-11-122-3/+2
| | | | | | | | | | Use a larger timeout on slower platforms so that the timeout doesn't fire before the error condition occurs. PR-URL: https://github.com/nodejs/node/pull/24296 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
* test: fix flaky test-vm-timeout-escape-nexttickRich Trott2018-11-091-1/+1
| | | | | | | | | | | | | Increase the VM timeout. If it is too small, the VM does not exit before the code has a chance to create the problematic condition that causes the timeout to be ignored. Fixes: https://github.com/nodejs/node/issues/24120 PR-URL: https://github.com/nodejs/node/pull/24251 Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* test: use URL fixtures under test/fixtures/wpt/url/resourcesJoyee Cheung2018-11-091-1/+3
| | | | | | | | | | | | | | | | | | | | | | Removes the following files: - test/fixtures/url-tests.js - test/fixtures/url-setter-tests.js - test/fixtures/url-toascii.js in favor of: - test/fixtures/wpt/url/resources/urltestdata.json - test/fixtures/wpt/url/resources/setters_tests.json - test/fixtures/wpt/url/resources/toascii.json Also removes dependency of `fixtures/url-tests.js` in http2 tests and use `fixtures/person-large.jpg` instead since they are just looking for a big enough file to transfer. PR-URL: https://github.com/nodejs/node/pull/24035 Refs: https://github.com/nodejs/node/issues/23192 Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
* test: fix flaky VM timeout test on Raspberry PiRich Trott2018-11-072-6/+6
| | | | | | | | | | | | Increase the timeouts based on platform. This required adjusting common.platformTimeout() to deal with bigint. Fixes: https://github.com/nodejs/node/issues/24120 PR-URL: https://github.com/nodejs/node/pull/24238 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
* test: remove unused catch bindingscjihrig2018-11-061-2/+2
| | | | | | PR-URL: https://github.com/nodejs/node/pull/24079 Reviewed-By: Sam Roberts <vieuxtech@gmail.com> Reviewed-By: Wyatt Preul <wpreul@gmail.com>
* module: support multi-dot file extensionGeoffrey Booth2018-11-011-18/+0
| | | | | | | | Support multi-dot file extensions like '.coffee.md' in Module.load. PR-URL: https://github.com/nodejs/node/pull/23416 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: John-David Dalton <john.david.dalton@gmail.com>
* test: mark test-vm-timeout-* known issue tests flakyJames M Snell2018-10-241-0/+3
| | | | | | | | | | These are known issues that can be flaky on certain platforms because they rely entirely on timing differences. PR-URL: https://github.com/nodejs/node/pull/23743 Refs: https://github.com/nodejs/node/issues/3020 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
View Lorry Controller Status