node icon indicating copy to clipboard operation
node copied to clipboard

test: refactor test-cluster-master-error to be more resilient

Open addaleax opened this issue 4 years ago • 12 comments

Using the filesystem for communication circumvents the issue described in the comment where a new process could end up picking up the process ids that the test case checks for.

Checklist
  • [x] make -j4 test (UNIX), or vcbuild test (Windows) passes
  • [x] tests and/or benchmarks are included
  • [x] commit message follows commit guidelines

addaleax avatar Oct 11 '20 14:10 addaleax

CI: https://ci.nodejs.org/job/node-test-pull-request/33526/

nodejs-github-bot avatar Oct 11 '20 14:10 nodejs-github-bot

Interesting that this timed out in one of the Windows tests … might take me a while to get back to that, but it makes me suspect that there’s a chance this is a genuine bug here (since it is flaky both before and after this PR)

addaleax avatar Oct 12 '20 08:10 addaleax

So... On windows, I've seen some flakiness before when calling sync file operations in process exit but that's been on much larger outputs. I'll see if I can investigate tomorrow

jasnell avatar Oct 15 '20 04:10 jasnell

CI: https://ci.nodejs.org/job/node-test-pull-request/34189/

nodejs-github-bot avatar Nov 08 '20 11:11 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/34265/

nodejs-github-bot avatar Nov 10 '20 04:11 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/34433/

nodejs-github-bot avatar Nov 17 '20 14:11 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/35455/

nodejs-github-bot avatar Jan 11 '21 23:01 nodejs-github-bot

@addaleax This needs a rebase.

aduh95 avatar Jan 18 '21 20:01 aduh95

CI: https://ci.nodejs.org/job/node-test-pull-request/58954/

nodejs-github-bot avatar May 05 '24 12:05 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/58956/

nodejs-github-bot avatar May 05 '24 14:05 nodejs-github-bot

It looks like this commit is making the test less resilient on Windows – I don't know if I did something wrong when rebasing or what. I'm going to add the stalled label, if someone cares to pick it up and fine what's wrong with it, that'd be awesome, otherwise we can keep the current test as is.

aduh95 avatar May 05 '24 19:05 aduh95

This issue/PR was marked as stalled, it will be automatically closed in 30 days. If it should remain open, please leave a comment explaining why it should remain open.

github-actions[bot] avatar May 05 '24 19:05 github-actions[bot]

Closing this because it has stalled. Feel free to reopen if this issue/PR is still relevant, or to ping the collaborator who labelled it stalled if you have any questions.

github-actions[bot] avatar Jun 12 '24 00:06 github-actions[bot]