Skip to content

test: deflake async-hooks/test-improper-order on AIX #58567

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
gulbaki wants to merge 7 commits into from
Closed

test: deflake async-hooks/test-improper-order on AIX #58567

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
5aac532
test: deflake async-hooks/test-improper-order on AIX #58562
bakigul1 Jun 3, 2025
1a80d8a
Update test-improper-order.js
gulbaki Jun 3, 2025
0d35e9b
update test-improper-order.js
gulbaki Jun 3, 2025
0427368
test: improve exit handling in async-hooks/test-improper-order
gulbaki Jun 4, 2025
4a23c0f
test: improve flakiness detection on stack corruption tests
gulbaki Jun 6, 2025
381034a
Apply suggestions from code review
gulbaki Jun 8, 2025
e9f8488
fix lint
gulbaki Jun 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Apply suggestions from code review 
Co-authored-by: Darshan Sen <raisinten@gmail.com>
  • Loading branch information
@gulbaki @RaisinTen
gulbaki and RaisinTen authored Jun 8, 2025
commit 381034ac93f297c5dc2f39f12735904b6dfaef6a
6 changes: 3 additions & 3 deletions test/async-hooks/test-improper-order.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,9 @@ if (process.argv[2] === 'child') {

child.on('close', common.mustCall((code, signal) => {
if ((common.isAIX ||
(common.isLinux && process.arch === 'x64')) &&
signal === 'SIGABRT') {
// XXX: The child process could be aborted due to unknown reasons. Work around it.
(common.isLinux && process.arch === 'x64')) &&
signal === 'SIGABRT') {
// XXX: The child process could be aborted due to unknown reasons. Work around it.
Comment thread
RaisinTen marked this conversation as resolved.
Resolved
} else {
assert.strictEqual(signal, null);
assert.strictEqual(code, 1);
Expand Down