Skip to content

fix(harness): stabilize OAuth callback tests#3397

Merged
tatoalo merged 1 commit into
mainfrom
posthog-code/stabilize-harness-oauth-tests
Jul 13, 2026
Merged

fix(harness): stabilize OAuth callback tests#3397
tatoalo merged 1 commit into
mainfrom
posthog-code/stabilize-harness-oauth-tests

Conversation

@tatoalo

@tatoalo tatoalo commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Problem

The harness OAuth callback tests use real loopback HTTP servers and can exceed Vitest's default 5-second timeout when CI is under load

Generated-By: PostHog Code
Task-Id: 03956753-fc87-4c81-8f32-24b5ab7d98aa
@trunk-io

trunk-io Bot commented Jul 13, 2026

Copy link
Copy Markdown

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 1b5b140.

@tatoalo tatoalo self-assigned this Jul 13, 2026
@tatoalo tatoalo marked this pull request as ready for review July 13, 2026 14:13
@tatoalo tatoalo added the Stamphog This will request an autostamp by stamphog on small changes label Jul 13, 2026

@stamphog stamphog 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.

The change is a trivial test timeout bump in OAuth tests, but it matched the auth deny-list and was classified as T2-never, requiring human review. Get a review from a teammate before re-requesting.

@stamphog stamphog Bot removed the Stamphog This will request an autostamp by stamphog on small changes label Jul 13, 2026
@greptile-apps

greptile-apps Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(harness): stabilize OAuth callback t..." | Re-trigger Greptile

@tatoalo tatoalo requested a review from a team July 13, 2026 14:19
@tatoalo tatoalo merged commit 863768a into main Jul 13, 2026
32 checks passed
@tatoalo tatoalo deleted the posthog-code/stabilize-harness-oauth-tests branch July 13, 2026 14:31
charlesvien pushed a commit to ricardo-leiva/code that referenced this pull request Jul 13, 2026
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.

2 participants