Skip to content

Commit

Permalink
test(ext/node): disable flaky dgram tests (#27549)
Browse files Browse the repository at this point in the history
Closes #27316
  • Loading branch information
kt3k authored and dsherret committed Jan 9, 2025
1 parent 00719ec commit ee5e1db
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 122 deletions.
8 changes: 5 additions & 3 deletions tests/node_compat/config.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -368,9 +368,11 @@
"test-dgram-send-callback-multi-buffer.js",
"test-dgram-send-callback-recursive.js",
"test-dgram-send-default-host.js",
"test-dgram-send-empty-array.js",
"test-dgram-send-empty-buffer.js",
"test-dgram-send-empty-packet.js",
// TODO(kt3k): These tests are flaky on macOS CI.
// https://github.com/denoland/deno/issues/27316
// "test-dgram-send-empty-array.js",
// "test-dgram-send-empty-buffer.js",
// "test-dgram-send-empty-packet.js",
"test-dgram-send-error.js",
"test-dgram-send-invalid-msg-type.js",
"test-dgram-send-multi-buffer-copy.js",
Expand Down
5 changes: 4 additions & 1 deletion tests/node_compat/runner/TODO.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<!-- deno-fmt-ignore-file -->
# Remaining Node Tests

1155 tests out of 3681 have been ported from Node 20.11.1 (31.38% ported, 69.14% remaining).
1152 tests out of 3681 have been ported from Node 20.11.1 (31.30% ported, 69.22% remaining).

NOTE: This file should not be manually edited. Please edit `tests/node_compat/config.json` and run `deno task setup` in `tests/node_compat/runner` dir instead.

Expand Down Expand Up @@ -499,6 +499,9 @@ NOTE: This file should not be manually edited. Please edit `tests/node_compat/co
- [parallel/test-dgram-multicast-set-interface.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-dgram-multicast-set-interface.js)
- [parallel/test-dgram-multicast-setTTL.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-dgram-multicast-setTTL.js)
- [parallel/test-dgram-send-address-types.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-dgram-send-address-types.js)
- [parallel/test-dgram-send-empty-array.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-dgram-send-empty-array.js)
- [parallel/test-dgram-send-empty-buffer.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-dgram-send-empty-buffer.js)
- [parallel/test-dgram-send-empty-packet.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-dgram-send-empty-packet.js)
- [parallel/test-dgram-send-queue-info.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-dgram-send-queue-info.js)
- [parallel/test-dgram-sendto.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-dgram-sendto.js)
- [parallel/test-dgram-setBroadcast.js](https://github.com/nodejs/node/tree/v20.11.1/test/parallel/test-dgram-setBroadcast.js)
Expand Down
32 changes: 0 additions & 32 deletions tests/node_compat/test/parallel/test-dgram-send-empty-array.js

This file was deleted.

50 changes: 0 additions & 50 deletions tests/node_compat/test/parallel/test-dgram-send-empty-buffer.js

This file was deleted.

36 changes: 0 additions & 36 deletions tests/node_compat/test/parallel/test-dgram-send-empty-packet.js

This file was deleted.

0 comments on commit ee5e1db

Please sign in to comment.