getsentry / getsentry/sentry-javascript

cloudflare: Two uncovered spots

Đang mở
#24,041 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
Cloudflare Workers 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ả

* `src/executionContext.ts`: 0% statements, 0% branches.
* `src/withSentry.ts`: 33% branch coverage (see C2).

**Work item.** Add unit tests for both. `executionContext.ts` should be cheap.

**Prior art ***(related history only)*. Three open issues cover Cloudflare test infrastructure but not these two files: [#22523]() "Add tests with @cloudflare/vitest-pool-workers", [#23610]() "Create E2E tests that use real Cloudflare Workers", and [#20874]() "Create canary tests for Cloudflare E2E and integration tests".

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

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

Hướng nghiên cứu

Start by reading src/executionContext.ts and src/withSentry.ts, then review the coverage report, especially the C2 branch in withSentry.ts. Add unit tests for the uncovered statements and branches in both files, with executionContext.ts treated as the smaller task. Done means both files have tests covering the identified gaps.

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
cloud, 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
Khá rõ ràng
Mức phù hợp với người mới
70/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.