getsentry / getsentry/sentry-javascript

node-integration-tests: port GraphQL suites to span streaming

Offen
#24,139 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
javascript Task
Vorherrschende Sprache
TypeScript
Sterne
8.7k
Forks
1.8k
Ø Merge
1 T. 17 Std.
Gemergte PRs (30 T.)
523

Beschreibung

Follow the porting recipe in the parent issue.

Small group, but the span names change the most: GraphQL phase spans now use the low cardinality fallback name and carry `graphql.processing.type`, and the segment name update from `useOperationNameForRootSpan` lands on the segment span instead of the transaction name.

## Suites

* `tracing/apollo-graphql`, `tracing/apollo-graphql/resolvers`, `tracing/apollo-graphql/useOperationNameForRootSpan`
* `tracing/graphql-tracing-channel`, `tracing/graphql-tracing-channel/resolvers`, `tracing/graphql-tracing-channel/useOperationNameForRootSpan`
* `aws-serverless/graphql/useOperationNameForRootSpan`

No `-streamed` twins exist for this group.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Start with the parent issue's porting recipe, then inspect the listed suites under tracing/apollo-graphql, tracing/graphql-tracing-channel, and aws-serverless/graphql. Update the suites for the specified GraphQL span names, processing type, and segment-name behavior; done means all listed suites are ported and their tests pass.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
graphql, node.js, typescript
Bereich
testing-qa
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
68/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.