FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add Cloudflare Workers Vitest test setup by sounmind · Pull Request #44 · DaleStudy/github · GitHub

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 commented May 2, 2026
edited
Loading

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 install 후 bun 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

Copy link
Copy Markdown

Deploying with    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 marked this pull request as ready for review May 2, 2026 00:58
sounmind requested a review from SamTheKorean May 2, 2026 01:05
sounmind self-assigned this May 2, 2026
sounmind requested review from a team and removed request for SamTheKorean May 2, 2026 01:05
soobing self-requested a review May 2, 2026 01:32

SamTheKorean 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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

sounmind merged commit 7b63ecc into main May 2, 2026
2 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL