[PR #4240] [MERGED] test: enable cloudflare smoke test #5272

Closed
opened 2026-03-13 12:16:26 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/4240
Author: @himself65
Created: 8/26/2025
Status: Merged
Merged: 8/26/2025
Merged by: @himself65

Base: canaryHead: himself65/2025/08/26/cloudflare


📝 Commits (1)

  • c3e9b4f test: enable cloudflare smoke test

📊 Changes

33 files changed (+8961 additions, -1304 deletions)

View changed files

e2e/smoke/test/cloudflare.spec.ts (+35 -0)
📝 e2e/smoke/test/fixtures/cloudflare/.gitignore (+63 -50)
📝 e2e/smoke/test/fixtures/cloudflare/drizzle.config.ts (+0 -0)
📝 e2e/smoke/test/fixtures/cloudflare/drizzle/0000_clean_vector.sql (+0 -0)
📝 e2e/smoke/test/fixtures/cloudflare/drizzle/meta/0000_snapshot.json (+0 -0)
📝 e2e/smoke/test/fixtures/cloudflare/drizzle/meta/_journal.json (+0 -0)
e2e/smoke/test/fixtures/cloudflare/package.json (+22 -0)
📝 e2e/smoke/test/fixtures/cloudflare/src/auth-schema.ts (+0 -0)
📝 e2e/smoke/test/fixtures/cloudflare/src/db.ts (+0 -0)
📝 e2e/smoke/test/fixtures/cloudflare/src/index.ts (+22 -1)
📝 e2e/smoke/test/fixtures/cloudflare/test/apply-migrations.ts (+0 -0)
📝 e2e/smoke/test/fixtures/cloudflare/test/env.d.ts (+0 -0)
📝 e2e/smoke/test/fixtures/cloudflare/test/index.test.ts (+0 -0)
e2e/smoke/test/fixtures/cloudflare/tsconfig.json (+19 -0)
📝 e2e/smoke/test/fixtures/cloudflare/vitest.config.ts (+1 -1)
e2e/smoke/test/fixtures/cloudflare/worker-configuration.d.ts (+8658 -0)
e2e/smoke/test/fixtures/cloudflare/wrangler.json (+18 -0)
integration-tests/bun/README.md (+0 -15)
integration-tests/bun/auth.ts (+0 -12)
integration-tests/bun/client.ts (+0 -13)

...and 13 more files

📄 Description

Summary by cubic

Add a Cloudflare smoke test that builds and sanity-checks our Workers-based OpenAuth example. Replaces the old integration tests with a self-contained e2e fixture using Wrangler, Miniflare, and D1.

  • New Features

    • Added e2e/smoke/test/cloudflare-build.spec.ts to run the Cloudflare fixture and verify it exits cleanly.
    • Introduced a Cloudflare fixture with wrangler.json, Workers type defs, D1 migrations, and a Hono + better-auth + drizzle setup.
    • Included a README with setup and migration steps.
  • Refactors

    • Removed integration-tests/bun and integration-tests/cloudflare; tests now live under e2e/smoke/test/fixtures/cloudflare.
    • Updated vitest config to use wrangler.json; tweaked tsconfig includes.
    • Adjusted workspace packages and added a resolution (miniflare>zod) to support the new test.

🔄 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/4240 **Author:** [@himself65](https://github.com/himself65) **Created:** 8/26/2025 **Status:** ✅ Merged **Merged:** 8/26/2025 **Merged by:** [@himself65](https://github.com/himself65) **Base:** `canary` ← **Head:** `himself65/2025/08/26/cloudflare` --- ### 📝 Commits (1) - [`c3e9b4f`](https://github.com/better-auth/better-auth/commit/c3e9b4f565a4e82b6f53508f82fba30d0b434af8) test: enable cloudflare smoke test ### 📊 Changes **33 files changed** (+8961 additions, -1304 deletions) <details> <summary>View changed files</summary> ➕ `e2e/smoke/test/cloudflare.spec.ts` (+35 -0) 📝 `e2e/smoke/test/fixtures/cloudflare/.gitignore` (+63 -50) 📝 `e2e/smoke/test/fixtures/cloudflare/drizzle.config.ts` (+0 -0) 📝 `e2e/smoke/test/fixtures/cloudflare/drizzle/0000_clean_vector.sql` (+0 -0) 📝 `e2e/smoke/test/fixtures/cloudflare/drizzle/meta/0000_snapshot.json` (+0 -0) 📝 `e2e/smoke/test/fixtures/cloudflare/drizzle/meta/_journal.json` (+0 -0) ➕ `e2e/smoke/test/fixtures/cloudflare/package.json` (+22 -0) 📝 `e2e/smoke/test/fixtures/cloudflare/src/auth-schema.ts` (+0 -0) 📝 `e2e/smoke/test/fixtures/cloudflare/src/db.ts` (+0 -0) 📝 `e2e/smoke/test/fixtures/cloudflare/src/index.ts` (+22 -1) 📝 `e2e/smoke/test/fixtures/cloudflare/test/apply-migrations.ts` (+0 -0) 📝 `e2e/smoke/test/fixtures/cloudflare/test/env.d.ts` (+0 -0) 📝 `e2e/smoke/test/fixtures/cloudflare/test/index.test.ts` (+0 -0) ➕ `e2e/smoke/test/fixtures/cloudflare/tsconfig.json` (+19 -0) 📝 `e2e/smoke/test/fixtures/cloudflare/vitest.config.ts` (+1 -1) ➕ `e2e/smoke/test/fixtures/cloudflare/worker-configuration.d.ts` (+8658 -0) ➕ `e2e/smoke/test/fixtures/cloudflare/wrangler.json` (+18 -0) ➖ `integration-tests/bun/README.md` (+0 -15) ➖ `integration-tests/bun/auth.ts` (+0 -12) ➖ `integration-tests/bun/client.ts` (+0 -13) _...and 13 more files_ </details> ### 📄 Description <!-- This is an auto-generated description by cubic. --> ## Summary by cubic Add a Cloudflare smoke test that builds and sanity-checks our Workers-based OpenAuth example. Replaces the old integration tests with a self-contained e2e fixture using Wrangler, Miniflare, and D1. - **New Features** - Added e2e/smoke/test/cloudflare-build.spec.ts to run the Cloudflare fixture and verify it exits cleanly. - Introduced a Cloudflare fixture with wrangler.json, Workers type defs, D1 migrations, and a Hono + better-auth + drizzle setup. - Included a README with setup and migration steps. - **Refactors** - Removed integration-tests/bun and integration-tests/cloudflare; tests now live under e2e/smoke/test/fixtures/cloudflare. - Updated vitest config to use wrangler.json; tweaked tsconfig includes. - Adjusted workspace packages and added a resolution (miniflare>zod) to support the new test. <!-- End of auto-generated description by cubic. --> --- <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-03-13 12:16:26 -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#5272