getsentry / getsentry/sentry-javascript

hono: Only the Node entry has SDK tests

Đang mở
#24,050 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
Bun javascript Tests
Ngôn ngữ chính
TypeScript
Star
8.7k
Fork
1.8k
Merge trung bình
1 ngày 17 giờ
Pull request đã merge (30 ngày)
523

Mô tả

`hono/test/` has `node/sdk.test.ts` but no `bun/sdk.test.ts`, `deno/sdk.test.ts` or `cloudflare/sdk.test.ts`. All four `init` functions carry the `buildFilteredIntegrations` + `LOW_QUALITY_TRANSACTION_PATTERNS` logic, and the double-init divergence in H1 is exactly the kind of thing a shared test would have caught.

**Work item.** Add one parameterized SDK test that runs against all four entries and asserts: sdk name and packages, `ignoreSpans` merging, Hono removed from defaults, and double-init behavior.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Start with hono/test/node/sdk.test.ts and locate the corresponding init entries for Bun, Deno, and Cloudflare. Add one parameterized SDK test covering the four listed behaviors across all entries, then run the Hono SDK tests. Done means each entry passes the sdk name and packages, ignoreSpans merging, Hono default removal, and double-init assertions.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
typescript
Lĩnh vực
testing
Loại issue
Tính năng
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Sôi nổi
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
76/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.