Skip to content
This repository was archived by the owner on Apr 22, 2023. It is now read-only.
This repository was archived by the owner on Apr 22, 2023. It is now read-only.

test/simple/test-child-process-fork-dgram.js is broken on SmartOS #8046

@misterdjules

Description

@misterdjules

Actually, it seems we're just lucky that it works on any platform. It's possible that either only the child or the master will handle UDP messages. That what happens on SmartOS. PR #8045 fixes the issue currently, but it's some band-aid on a test waiting to break.

I'll rephrase what I included in the PR's description:

What are we trying to test with this test case? Do we absolutely need both the child and the parent to always handle UDP messages? Could we remove the UDP 'message' event handler as soon as we receive one dgram in the child or the master, so that the one that did not receive a message yet will be the only one that can receive it? This would guarantee (well, as much as UDP can guarantee anything...) that both processes will receive at least one message.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions