Skip to content

test: stabilize timing-sensitive integration fixtures - #65

Merged
davidahmann merged 1 commit into
mainfrom
codex/stabilize-github-adapter-ci
Sep 22, 2026
Merged

davidahmann merged 1 commit into
mainfrom
codex/stabilize-github-adapter-ci

Conversation

@davidahmann

@davidahmann davidahmann commented Sep 22, 2026 •

Copy link
Copy Markdown
Owner

Problem

Main CI exposed two timing-sensitive test assumptions under parallel runner load:

  • concurrent fake gh processes could JSON-parse the shared call log while another process was appending to it
  • the OCI preflight test could consume its 100 ms deadline during setup and exercise command timeout instead of image-inspection timeout

Change

  • count fake-provider endpoint occurrences from raw log text before appending the current call
  • give the OCI preflight fixture one second to enter image inspection, with a separate three-second abort fallback

Production deadlines and runtime behavior are unchanged.

Validation

  • GitHub adapter test passed twice consecutively
  • OCI deadline test passed three times consecutively
  • npm run check
  • exact-head local maintainer review: ready, zero findings
  • DCO check passed

Signed-off-by: David Ahmann <46606159+davidahmann@users.noreply.github.com>
@davidahmann
davidahmann force-pushed the codex/stabilize-github-adapter-ci branch from 22ab62c to d293409 Compare September 22, 2026 00:19
@davidahmann davidahmann changed the title test: stabilize concurrent GitHub adapter fixture test: stabilize timing-sensitive integration fixtures Sep 22, 2026
@davidahmann
davidahmann merged commit 1392e78 into main Sep 22, 2026
6 checks passed
@davidahmann
davidahmann deleted the codex/stabilize-github-adapter-ci branch September 22, 2026 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant