Skip to content

tests can break when run in parallel #32

Closed

Description

@andrewzah

I'm packaging codex-relay in nix currently, and I had to disable some of the issue_26_ tests as they bind to the same port and thus break.

  # flakey: both issue_26 tests bind the same mock-upstream port
  checkFlags = [
    "--skip=issue_26_streaming_reasoning_content_emits_responses_reasoning_events"
  ];

Activity

  1. mountain commented on Jul 2, 2026

    @mountain
    Collaborator

    Fix in progress: #33 — tests no longer share ports in parallel. The mock upstream and relay subprocess now bind an ephemeral port (port 0) instead of the racy bind/drop/rebind pick_port() pattern, so the issue_26_* collision is gone and the skip flags shouldn't be needed anymore.

  2. added a commit that references this issue on Jul 2, 2026
  3. added a commit that references this issue on Sep 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions