getsentry / getsentry/sentry-javascript
node-integration-tests: drop static pins from non-tracing suites
- 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ả
Follow the porting recipe in the parent issue.
Largest group by file count and the smallest by effort. Roughly 90 suite directories carry the `traceLifecycle: 'static'` pin without ever asserting on a transaction envelope, so the pin just needs to go.
## Suites (pin removal only)
* `public-api/*` except the span surface dirs owned by the trace semantics sub-issue: `addBreadcrumb`, `captureException`, `captureMessage`, `configureScope`, `LocalVariables`, `logger`, `metrics`, `OnUncaughtException`, `setContext`, `setExtra`, `setExtras`, `setTag`, `setTags`, `setUser`, `withMonitor`, `withScope`
* `sessions`, `cron/*`, `client-reports/periodic-send`
* Logging: `pino`, `winston`, `consola`, `integrations/console/filter`
* Runtime: `anr`, `system-error`, `thread-blocked-native`, `child-process`, `modules`, `esm/*`, `contextLines/*`, `breadcrumbs/process-thread`, `node-runtime-metrics`, `bun-runtime-metrics`
* `vercel/sigterm-flush`
## Suites that still need an assertion rewrite
These 7 do assert on transaction envelopes:
* `public-api/onUnhandledRejectionIntegration`
* `client-reports/drop-reasons/*` (4 dirs)
* `featureFlags/featureFlagsIntegration/onSpan` and `featureFlags/growthbook/onSpan`
* `fs-instrumentation`
* `aws-serverless/aws-integration` (fold `aws-serverless/aws-integration-streamed` back into it)
Splitting this issue into a mechanical pin-removal PR and a smaller rewrite PR is fine.
Hướng dẫn đóng góp
Hướng nghiên cứu
Start with the parent issue's porting recipe, then search the listed node-integration-tests suite directories for traceLifecycle: 'static'. Remove the pin from the non-tracing suites named under “pin removal only”; leave the seven assertion-rewrite groups for separate work. Done means those suites no longer carry the unnecessary static pin while the listed envelope-asserting suites remain unchanged.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- node.js, typescript
- Lĩnh vực
- testing-qa
- Loại issue
- Tái cấu trúc
- Độ 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
- 74/100