Skip to content

test(harness): stabilize runtime cold-start test#3727

Merged
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/stabilize-harness-runtime-test
Jul 22, 2026
Merged

test(harness): stabilize runtime cold-start test#3727
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/stabilize-harness-runtime-test

Conversation

@pauldambra

Copy link
Copy Markdown
Member

Problem

The harness runtime integration test performs a cold import and full native Pi runtime initialization. Under concurrent CI load, that valid setup can exceed Vitest's default five-second timeout and fail unrelated pull requests.

Why: The test needs enough time to exercise the real runtime consistently in CI without weakening timeout coverage across the rest of the harness suite.

Changes

Give only the cold-start runtime test a 15-second timeout. All other harness tests retain Vitest's default timeout.

How did you test this?

  • pnpm exec biome check --write packages/harness/src/runtime.test.ts
  • pnpm --filter @posthog/shared build
  • pnpm --filter @posthog/harness test (661 tests passed)
  • pnpm --filter @posthog/harness typecheck
  • git diff --check

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Created with PostHog Code

Generated-By: PostHog Code
Task-Id: f5d6c417-32e8-4a2b-bfdc-98f33b60c450
@trunk-io

trunk-io Bot commented Jul 22, 2026

Copy link
Copy Markdown

😎 Merged directly without going through the merge queue, as the queue was empty and the PR was up to date with the target branch - details.

@pauldambra pauldambra added the Stamphog This will request an autostamp by stamphog on small changes label Jul 22, 2026
@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 77b8a47.

@pauldambra
pauldambra marked this pull request as ready for review July 22, 2026 20:34

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Diff matches description exactly — only adds a 15s timeout option to one existing test, no logic changes, purely test-file scope.

  • 👍 on the PR from chatgpt-codex-connector[bot].
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 0L, 0F substantive, 72L/1F incl. docs/generated/snapshots — within ceiling
tier T0 auto-approve: T0-deterministic (72L, 1F, single-area, test)
stamphog 2.0.0b3 .stamphog/policy.yml @ aebec27 · reviewed head 77b8a47

@trunk-io
trunk-io Bot merged commit 5599f91 into main Jul 22, 2026
41 checks passed
@trunk-io
trunk-io Bot deleted the posthog-code/stabilize-harness-runtime-test branch July 22, 2026 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant