Skip to content

Add Cloudflare Workers Vitest test setup#44

Merged
sounmind merged 1 commit into
mainfrom
feat/cloudflare-vitest-tests
May 2, 2026
Merged

Add Cloudflare Workers Vitest test setup#44
sounmind merged 1 commit into
mainfrom
feat/cloudflare-vitest-tests

Conversation

@sounmind

@sounmind sounmind commented May 2, 2026

Copy link
Copy Markdown
Member

Summary

  • Cloudflare Workers Vitest integration으로 테스트 환경을 전환했습니다.
  • package.json, bun.lock, vitest.config.js를 추가했습니다.
  • 기존 테스트를 bun:test에서 vitest로 마이그레이션했습니다.
  • utils/github.js, utils/cors.js 단위 테스트를 추가했습니다.
  • cloudflare:test / cloudflare:workers 기반 Worker runtime smoke test를 추가했습니다.
  • CI를 bun installbun run test 실행으로 변경했습니다.
  • README/AGENTS 테스트 문서를 새 환경에 맞게 갱신했습니다.

Closes #2

Verification

bun run test
10 test files passed
106 tests passed

Notes

  • utils/github.js의 PKCS1 private key 처리도 실제 지원되도록 보강하고 회귀 테스트를 추가했습니다.
  • 로컬 workflow 산출물(.claude/, .omc/, docs/)은 커밋에 포함하지 않았습니다.

The project is deployed as a Cloudflare Worker, so the test harness now uses Cloudflare's Vitest pool instead of Bun's built-in runner. Existing tests were migrated to Vitest, utility crypto/CORS coverage was added for issue #2, and a small Worker runtime smoke test proves wrangler vars and request handling load through the Cloudflare test environment.

Constraint: Cloudflare Workers runtime APIs and wrangler bindings should be exercised through @cloudflare/vitest-pool-workers

Rejected: Keep Bun's built-in test runner as primary | it does not exercise Workers runtime bindings and had incompatible module mock behavior locally

Rejected: Maintain parallel Bun and Vitest suites | unnecessary duplicate maintenance for this small Worker

Confidence: high

Scope-risk: moderate

Directive: Keep docs/superpowers artifacts out of product commits unless the user explicitly asks to version workflow docs

Tested: bun run test (10 files, 106 tests); git diff --check; no bun:test/mock.module/vi.module leftovers

Not-tested: GitHub Actions execution on remote runner
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
github 3ae4b15 Commit Preview URL

Branch Preview URL
May 02 2026, 12:57 AM

@sounmind sounmind marked this pull request as ready for review May 2, 2026 00:58
@sounmind sounmind requested a review from SamTheKorean May 2, 2026 01:05
@sounmind sounmind self-assigned this May 2, 2026
@sounmind sounmind requested review from a team and removed request for SamTheKorean May 2, 2026 01:05
@soobing soobing self-requested a review May 2, 2026 01:32

@SamTheKorean SamTheKorean left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

이제는 태스트를 레퍼런스삼아서 더 원할하게 기여할 수 있겠네요! 고생하셨습니다!

@sounmind sounmind merged commit 7b63ecc into main May 2, 2026
2 checks passed
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