getsentry / getsentry/sentry-javascript

Add tests with @cloudflare/vitest-pool-workers

未关闭
#22,523 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
Cloudflare Workers Tests
主要语言
TypeScript
星标
8.7k
派生
1.8k
平均合并
1 天 17 小时
30 天内合并 PR
515

描述

### Description

For now our `cloudflare-integration-tests` do run with `wrangler` directly, which is not a bad thing overall. However, wrangler runs with ESBuild inside and does tree-shaking - when running tests within vitest, then there is no tree-shaking and it would include everything. In #22519 there was a case where `worker_threads` were included, which weren't available in an older compatibility version. To not break older compatibility versions by accident tests with `@cloudflare/vitest-pool-workers` would be required to prevent such things.

贡献指南

打开贡献指南

调研方向

Start by locating the cloudflare-integration-tests and reviewing how they currently run through wrangler. Read #22519 for the worker_threads compatibility case, then determine how @cloudflare/vitest-pool-workers should run equivalent tests; done means the tests cover this regression without breaking older compatibility versions.

由索引模型根据 Issue 内容生成。

评估

技术栈
typescript
领域
testing-qa
Issue 类型
功能
难度
4/5
预计耗时
3-5 天
活跃度
冷清
描述清晰度
需要澄清
新手友好度
45/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。