[PR #8210] [MERGED] fix(ci): increase test timeout for sso, api-key, oauth-provider and add CI job timeout #24721

Closed
opened 2026-04-15 22:31:51 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/8210
Author: @himself65
Created: 2/28/2026
Status: Merged
Merged: 2/28/2026
Merged by: @himself65

Base: canaryHead: fix/ci-test-timeout-prevention


📝 Commits (1)

  • c44d6af fix(ci): increase test timeout for sso, api-key, oauth-provider and add CI job timeout

📊 Changes

4 files changed (+4 additions, -0 deletions)

View changed files

📝 .github/workflows/ci.yml (+1 -0)
📝 packages/api-key/vitest.config.ts (+1 -0)
📝 packages/oauth-provider/vitest.config.ts (+1 -0)
📝 packages/sso/vitest.config.ts (+1 -0)

📄 Description

Summary

  • Add testTimeout: 10_000 to packages/sso, packages/api-key, and packages/oauth-provider vitest configs to prevent flaky CI timeout failures on large test suites (matching the pattern already applied to better-auth, scim, and stripe)
  • Add timeout-minutes: 30 to the CI test job to catch hung tests early instead of waiting for GitHub's default 6-hour timeout

Test plan

  • CI passes on this PR
  • Verify sso, api-key, and oauth-provider tests no longer hit the default 5s timeout

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/better-auth/better-auth/pull/8210 **Author:** [@himself65](https://github.com/himself65) **Created:** 2/28/2026 **Status:** ✅ Merged **Merged:** 2/28/2026 **Merged by:** [@himself65](https://github.com/himself65) **Base:** `canary` ← **Head:** `fix/ci-test-timeout-prevention` --- ### 📝 Commits (1) - [`c44d6af`](https://github.com/better-auth/better-auth/commit/c44d6af06e8cb946412112d1b9b1de0205160905) fix(ci): increase test timeout for sso, api-key, oauth-provider and add CI job timeout ### 📊 Changes **4 files changed** (+4 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/ci.yml` (+1 -0) 📝 `packages/api-key/vitest.config.ts` (+1 -0) 📝 `packages/oauth-provider/vitest.config.ts` (+1 -0) 📝 `packages/sso/vitest.config.ts` (+1 -0) </details> ### 📄 Description ## Summary - Add `testTimeout: 10_000` to `packages/sso`, `packages/api-key`, and `packages/oauth-provider` vitest configs to prevent flaky CI timeout failures on large test suites (matching the pattern already applied to `better-auth`, `scim`, and `stripe`) - Add `timeout-minutes: 30` to the CI `test` job to catch hung tests early instead of waiting for GitHub's default 6-hour timeout ## Test plan - [ ] CI passes on this PR - [ ] Verify sso, api-key, and oauth-provider tests no longer hit the default 5s timeout --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-15 22:31:51 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#24721