Commit Graph

215 Commits

Author SHA1 Message Date
jakecastelli
0ed9a43242
test: mark test-pipe-file-to-http as flaky
PR-URL: https://github.com/nodejs/node/pull/53751
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
2024-07-23 14:12:40 +00:00
Abdirahim Musse
e55587e398
test: un-set inspector-async-hook-setup-at-inspect-brk as flaky
There was a commit that got merged that should have fixed the issue

ref: https://github.com/nodejs/node/issues/50222#issuecomment-1962680957
PR-URL: https://github.com/nodejs/node/pull/53692
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
2024-07-04 20:45:42 +00:00
Michaël Zasso
7ad0cc3e57
build: remove support for 32-bit Windows
Closes: https://github.com/nodejs/node/issues/42543
PR-URL: https://github.com/nodejs/node/pull/53184
Fixes: https://github.com/nodejs/node/issues/42543
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Stefan Stojanovic <stefan.stojanovic@janeasystems.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
2024-05-30 14:28:47 +00:00
Madhuri
b41ab9d116
test: move test-http-server-request-timeouts-mixed to sequential
PR-URL: https://github.com/nodejs/node/pull/45722
Fixes: https://github.com/nodejs/node/issues/43465
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
2024-05-11 20:07:28 +02:00
Chengzhong Wu
1291d3d5c4
test: skip unstable shadow realm gc tests
PR-URL: https://github.com/nodejs/node/pull/52855
Refs: https://github.com/nodejs/node/issues/52273
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
2024-05-08 21:41:40 +00:00
Antoine du Hamel
7c3dce0e4f
test: mark test-error-serdes as flaky
PR-URL: https://github.com/nodejs/node/pull/52739
Refs: https://github.com/nodejs/node/issues/52630
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
2024-04-29 01:10:30 +02:00
Michael Dawson
02388eb357
test: mark test as flaky
Refs: https://github.com/nodejs/node/issues/51862

Ran into this a few times myself this week and also
seems to be a top failer in the reliability reports

Signed-off-by: Michael Dawson <midawson@redhat.com>
PR-URL: https://github.com/nodejs/node/pull/52671
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
2024-04-26 20:57:48 +00:00
Michaël Zasso
9efc84a2cb
test: mark test-worker-arraybuffer-zerofill as flaky
Refs: https://github.com/nodejs/node/issues/52274
PR-URL: https://github.com/nodejs/node/pull/51362
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
2024-03-31 15:37:43 +02:00
Michaël Zasso
84c2e712eb
test: mark some GC-related tests as flaky
Refs: https://github.com/nodejs/node/issues/52273
PR-URL: https://github.com/nodejs/node/pull/51362
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
2024-03-31 15:37:41 +02:00
Luigi Pinca
5c97f411d8
test: remove flaky designation
Searching the automatically-generated test failure reports in the issues
at https://github.com/nodejs/reliability/issues, I don't see
`test-worker-message-port-message-before-close` failing any time
recently.

PR-URL: https://github.com/nodejs/node/pull/51736
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
2024-03-02 11:46:24 +00:00
Luigi Pinca
c1fa929f39
test: remove test-fs-stat-bigint flaky designation
Refs: https://github.com/nodejs/node/issues/31727#issuecomment-985589968
PR-URL: https://github.com/nodejs/node/pull/51735
Reviewed-By: Michael Dawson <midawson@redhat.com>
2024-02-22 18:33:14 +00:00
Shi Pujin
2eaee2820e test: skip test-http-correct-hostname on loong64
PR-URL: https://github.com/nodejs/node/pull/51663
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
2024-02-20 10:51:14 -05:00
Luigi Pinca
6db72fcae4
test: remove test-cli-node-options flaky designation
The last trace of failure dates back to 2023-09-24.

Fixes: https://github.com/nodejs/node/issues/50295
PR-URL: https://github.com/nodejs/node/pull/51716
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
2024-02-17 06:44:18 +00:00
Luigi Pinca
78273ed0d1
test: remove test-domain-error-types flaky designation
There is no recent trace of failure for this test.

Fixes: https://github.com/nodejs/node/issues/38063
PR-URL: https://github.com/nodejs/node/pull/51717
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
2024-02-16 19:27:04 +00:00
Luigi Pinca
0f80e29419
test: remove duplicate entry for flaky test
Refs: https://github.com/nodejs/node/blob/5acd42ed9e65/test/parallel/parallel.status#L28-L30
PR-URL: https://github.com/nodejs/node/pull/51654
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
2024-02-05 17:55:45 +00:00
Luigi Pinca
415aa77083
test: remove test-crypto-keygen flaky designation
Refs: https://github.com/nodejs/node/issues/41206#issuecomment-1858836845
PR-URL: https://github.com/nodejs/node/pull/51567
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
2024-02-03 09:42:22 +00:00
Luigi Pinca
a7eb72d23d
test: remove test-fs-rmdir-recursive flaky designation
Refs: https://github.com/nodejs/node/issues/41201#issuecomment-1858836768
PR-URL: https://github.com/nodejs/node/pull/51566
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
2024-02-01 07:13:03 +00:00
Luigi Pinca
6ab15a1aa8
test: remove test-file-write-stream4 flaky designation
Recent failures seem to be only caused by insufficient space left on the
device.

PR-URL: https://github.com/nodejs/node/pull/51472
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
2024-01-17 19:17:14 +00:00
Joyee Cheung
5ac658102d
test: deflake test-diagnostics-channel-memory-leak
PR-URL: https://github.com/nodejs/node/pull/50572
Refs: 0fd478bcda
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
2023-12-15 00:35:40 +01:00
Joyee Cheung
94156e35df
test: skip test-diagnostics-channel-memory-leak.js
There is currently no reliable way to detect this leak because:

1. We cannot reliably get a reference to the channel from the
  API to detect finalization without creating another strong reference.
2. This test does gc() and then checks memory usage - however the
  use of gc() disables code aging which can actually lead to increased
  memory usage overall, as it is not intended to be used to lower
  memory usage in the first place.
3. The implementation of diagnostics channels relies on ephemeron gc
  which is inefficient, it's not reliable to use the typical "create
  a lot of objects and see if it crashes" trick to check leaks.

Skip the test for now until we find a way to test it reliably.

To avoid flakiness in the CI, it's better to remove an unreliable
test altogether.

PR-URL: https://github.com/nodejs/node/pull/50327
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
2023-11-03 21:50:03 +00:00
Yagiz Nizipli
35becbc9a9 test: set inspector async stack test as flaky
https://github.com/nodejs/node/issues/50243
PR-URL: https://github.com/nodejs/node/pull/50244
Refs: https://github.com/nodejs/node/issues/50243
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Ruy Adorno <ruyadorno@google.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
2023-10-20 10:27:50 -04:00
StefanStojanovic
b4cc3e27c7 test: set test-worker-nearheaplimit-deadlock flaky
PR-URL: https://github.com/nodejs/node/pull/50277
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
2023-10-20 09:40:52 -04:00
Yagiz Nizipli
5ab70b3325 test: set test-cli-node-options as flaky
PR-URL: https://github.com/nodejs/node/pull/50296
Refs: https://github.com/nodejs/node/issues/50295
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
Reviewed-By: Michael Dawson <midawson@redhat.com>
2023-10-20 09:28:36 -04:00
Yagiz Nizipli
e0bbe56630
test: set inspector async hook test as flaky
Ref: https://github.com/nodejs/node/issues/50222
PR-URL: https://github.com/nodejs/node/pull/50252
Refs: https://github.com/nodejs/node/issues/50222
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
2023-10-18 22:19:39 +00:00
Yagiz Nizipli
b3313bf03f
test: set parallel http server test as flaky
Ref: https://github.com/nodejs/node/issues/43465
PR-URL: https://github.com/nodejs/node/pull/50227
Refs: https://github.com/nodejs/node/issues/43465
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
2023-10-18 18:13:04 +00:00
Stefan Stojanovic
dbd0ffa125
test: set test-worker-nearheaplimit-deadlock flaky
The test is only flaky on x86 Windows.

Fixes: https://github.com/nodejs/node/issues/50220
PR-URL: https://github.com/nodejs/node/pull/50238
Refs: https://github.com/nodejs/node/pull/49962
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
Reviewed-By: Michael Dawson <midawson@redhat.com>
2023-10-18 15:23:16 +00:00
Yagiz Nizipli
aa8282e08e
test: set test-runner-watch-mode as flaky
PR-URL: https://github.com/nodejs/node/pull/50221
Refs: https://github.com/nodejs/node/issues/49985
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
2023-10-18 15:01:51 +00:00
Michaël Zasso
c13c98dd38
test: skip test-tick-processor-arguments on SmartOS
Refs: https://github.com/nodejs/node/issues/50050
PR-URL: https://github.com/nodejs/node/pull/49639
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
2023-10-10 08:28:10 +02:00
Richard Lau
6c9625dca4
test: update skip for moved test-wasm-web-api
`test-wasm-web-api` was moved from `test/parallel` to `test/es-modules`.
Update the status files for parallel and es-modules accordingly.

Refs: https://github.com/nodejs/node/pull/49869
Refs: https://github.com/nodejs/node/pull/47299
Refs: https://github.com/nodejs/node/issues/47297
PR-URL: https://github.com/nodejs/node/pull/49958
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
2023-09-30 18:10:14 +00:00
Luigi Pinca
f05d6d090c
Revert "test: mark test-runner-output as flaky"
This reverts commit 717e233cd9.

Refs: https://github.com/nodejs/node/commit/fef7927cc3a7
PR-URL: https://github.com/nodejs/node/pull/49905
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Michael Dawson <midawson@redhat.com>
2023-09-30 15:43:14 +00:00
Joyee Cheung
717e233cd9
test: mark test-runner-output as flaky
This has been flaky on many platforms for months. Mark it as
flaky for now to avoid blocking the CI.

PR-URL: https://github.com/nodejs/node/pull/49854
Refs: https://github.com/nodejs/node/issues/49853
Refs: https://github.com/nodejs/reliability/issues/673
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
2023-09-25 17:13:28 +00:00
Joyee Cheung
dac5f29691
test: skip test-child-process-stdio-reuse-readable-stdio on Windows
It is flaky due to the same cause of test-child-process-pipe-dataflow
being flaky - cygwin quirks - so skip it on Windows too.

Drive-by: remove the skip mark of test-child-process-pipe-dataflow
in the status file and directly skip it in the test with a comment.

PR-URL: https://github.com/nodejs/node/pull/49621
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
2023-09-14 21:23:46 +00:00
Joyee Cheung
b289c6df11
test: mark test-runner-watch-mode as flaky
PR-URL: https://github.com/nodejs/node/pull/49627
Refs: https://github.com/nodejs/node/issues/49605
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
2023-09-14 14:55:19 +00:00
Joyee Cheung
b0edb28b63
test: skip test-child-process-pipe-dataflow.js on Windows
This is constantly failing on Windows now that the CI is never green
there. To give CI at least some green space, mark it as SKIP, because
we've been practically ignoring the failure for months anyway, and
will probably just continue doing that.

PR-URL: https://github.com/nodejs/node/pull/49563
Refs: https://github.com/nodejs/node/issues/48300
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
2023-09-11 11:29:21 +00:00
Luigi Pinca
08638b74ec
Revert "test: remove test-crypto-keygen flaky designation"
This reverts commit 893c000046.

Refs: https://github.com/nodejs/node/pull/48575#issuecomment-1620267554
PR-URL: https://github.com/nodejs/node/pull/48652
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Matthew Aitken <maitken033380023@gmail.com>
2023-07-05 16:30:30 +00:00
Luigi Pinca
893c000046 test: remove test-crypto-keygen flaky designation
PR-URL: https://github.com/nodejs/node/pull/48575
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
2023-06-29 08:47:12 +00:00
Luigi Pinca
300634d33d test: remove test-timers-immediate-queue flaky designation
PR-URL: https://github.com/nodejs/node/pull/48575
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
2023-06-29 08:47:11 +00:00
Luigi Pinca
c308054aaa
test: mark test-child-process-stdio-reuse-readable-stdio flaky
Refs: https://github.com/nodejs/node/issues/48300#issuecomment-1604531463
Refs: https://github.com/nodejs/reliability/issues/597#issue-1772339022
PR-URL: https://github.com/nodejs/node/pull/48537
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
2023-06-25 11:08:07 +00:00
Moshe Atlow
c6e7e42733
test: mark test-child-process-pipe-dataflow as flaky
PR-URL: https://github.com/nodejs/node/pull/48334
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
2023-06-04 20:58:45 +00:00
Abdirahim Musse
959142a465
test: skip test-http-pipeline-flood on IBM i
PR-URL: https://github.com/nodejs/node/pull/48048
Refs: https://github.com/nodejs/node/issues/48047
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
2023-05-22 11:50:10 +00:00
Yagiz Nizipli
62fd6bc4aa
test: move test-cluster-primary-error flaky test
PR-URL: https://github.com/nodejs/node/pull/48039
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Deokjin Kim <deokjin81.kim@gmail.com>
2023-05-17 13:26:05 +00:00
Yagiz Nizipli
08e1564474
test: mark test/parallel/test-file-write-stream4 as flaky
PR-URL: https://github.com/nodejs/node/pull/47423
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
2023-04-10 17:46:07 +00:00
Michaël Zasso
65b79aba0f
test: skip test-wasm-web-api on ARM
This test is flaky on ARM with V8 >= 11.2.
Skip it so we can update V8 before the release of Nodejs 20.0.0.

PR-URL: https://github.com/nodejs/node/pull/47299
Refs: https://github.com/nodejs/node/pull/47251
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
2023-03-30 12:09:32 +02:00
Stefan Stojanovic
13767a4675
test: fix test-child-process-exec-cwd
Refs: https://github.com/nodejs/node/pull/47020
Refs: https://github.com/nodejs/build/issues/3046
PR-URL: https://github.com/nodejs/node/pull/47235
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
2023-03-27 11:20:08 +00:00
Stefan Stojanovic
f08655532b
test: skip broken tests win arm64
PR-URL: https://github.com/nodejs/node/pull/47020
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
2023-03-22 10:54:49 +00:00
Vita Batrla
21fcfcdd5d
test: fix test-net-connect-reset-until-connected
Fixes: https://github.com/nodejs/node/issues/43446
PR-URL: https://github.com/nodejs/node/pull/46781
Reviewed-By: theanarkh <theratliter@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
2023-02-25 18:53:11 +00:00
Rich Trott
badc631d77
test: remove flaky designation for test-worker-http2-stream-terminate
Closes: https://github.com/nodejs/node/issues/43084
PR-URL: https://github.com/nodejs/node/pull/45438
Fixes: https://github.com/nodejs/node/issues/43084
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
2022-11-14 17:19:48 +00:00
Rich Trott
6a22b77bba
test: fix flaky test-repl-sigint-nested-eval
There is a race condition where process.kill can be sent before the
target is ready to receive the signal.

Fixes: https://github.com/nodejs/node/issues/41123
PR-URL: https://github.com/nodejs/node/pull/45354
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
2022-11-13 02:56:08 +00:00
Richard Lau
261176b04d
test: mark connection leak test flaky on IBM i
Mark `test-http-pipeline-requests-connection-leak` flaky on IBM i.

PR-URL: https://github.com/nodejs/node/pull/44215
Refs: https://github.com/nodejs/node/issues/43509
Refs: https://github.com/nodejs/node/issues/39683
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
2022-08-14 04:30:11 +00:00
Luigi Pinca
b2f4a55770
test: remove test-http-client-response-timeout flaky designation
The issue was fixed by https://github.com/nodejs/node/pull/43890.

Fixes: https://github.com/nodejs/node/issues/43680
PR-URL: https://github.com/nodejs/node/pull/44145
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>
2022-08-07 12:26:50 +00:00