Confirm bun-integration-tests need no span streaming port

Open Beginner friendly
#24,134 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
82/100
Issue type
Refactor
Clarity
Clearly specified
Activity status
Active
Tech stack
bun, typescript
Domain
testing-qa

Research direction

Inspect the five suites under dev-packages/bun-integration-tests: basic, fetch, hono-sdk, orchestrion-mysql, and orchestrion-postgres. Search them for traceLifecycle, STREAMED, transaction envelope assertions, and orchestrion span names, then run the Bun integration tests. Done means the listed checks are confirmed or any discovered gap is split into a follow-up issue.

Written by the indexing model from the issue text.

Description

javascript Task

Part of SDK-1425, one tracking issue per integration test package.

dev-packages/bun-integration-tests carries no traceLifecycle pin at all, so it already runs on the default streaming lifecycle. suites/fetch and suites/hono-sdk already assert on streamed spans.

This is a verification issue, not a port. Confirm and close, or split off whatever gap turns up.

What to check

  • No suite pins traceLifecycle: 'static' and none uses an env-gated STREAMED switch.
  • The 5 suites (basic, fetch, hono-sdk, orchestrion-mysql, orchestrion-postgres) assert on span v2 payloads, not on transaction envelopes. The remaining transaction: reference in suites/hono-sdk/test.ts is the error event's transaction field, which stays.
  • The orchestrion suites assert low cardinality db span names.
Dominant language
TypeScript
Stars
8.7k
Forks
1.9k
Avg merge
1d 17h
Merged PRs (30d)
521

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from getsentry/sentry-javascript

All issues in getsentry/sentry-javascript

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.