Skip to content

test(aws-serverless): Add callback-style and trace-propagation Lambda e2e coverage - #21757

Closed
nicohrubec wants to merge 1 commit into
developfrom
nh/aws-lambda-tests
Closed

test(aws-serverless): Add callback-style and trace-propagation Lambda e2e coverage#21757
nicohrubec wants to merge 1 commit into
developfrom
nh/aws-lambda-tests

Conversation

@nicohrubec

Copy link
Copy Markdown
Member

Adds e2e coverage for AWS Lambda handler paths the existing async-only suite missed, ahead of the AwsLambda instrumentation streamline (#stacked):

  • Callback-style handler ((event, context, callback)) — success transaction + error capture (mechanism auto.function.aws_serverless.otel).
  • Trace propagation — invoking with an incoming sentry-trace/baggage and asserting the Lambda transaction continues the trace_id/parent_span_id.

These pass against the current instrumentation, so they act as a behavior-preserving guard for the streamline PR stacked on top.

🤖 Generated with Claude Code

… e2e coverage
Adds e2e cases for callback-style handlers (success + error) and trace
continuation from incoming headers, covering paths the existing async-only
suite missed.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser27.47 kB+0.27%+73 B 🔺
@sentry/browser - with treeshaking flags25.91 kB+0.28%+72 B 🔺
@sentry/browser (incl. Tracing)45.97 kB+0.59%+267 B 🔺
@sentry/browser (incl. Tracing + Span Streaming)47.72 kB-0.46%-218 B 🔽
@sentry/browser (incl. Tracing, Profiling)50.76 kB+0.51%+255 B 🔺
@sentry/browser (incl. Tracing, Replay)85.22 kB+0.35%+295 B 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags74.81 kB+0.38%+282 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas)89.91 kB+0.34%+298 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback)102.57 kB+0.27%+268 B 🔺
@sentry/browser (incl. Feedback)44.66 kB+0.22%+95 B 🔺
@sentry/browser (incl. sendFeedback)32.26 kB+0.18%+55 B 🔺
@sentry/browser (incl. FeedbackAsync)37.4 kB+0.24%+88 B 🔺
@sentry/browser (incl. Metrics)28.54 kB+0.27%+75 B 🔺
@sentry/browser (incl. Logs)28.78 kB+0.26%+74 B 🔺
@sentry/browser (incl. Metrics & Logs)29.47 kB+0.24%+69 B 🔺
@sentry/react29.27 kB+0.27%+78 B 🔺
@sentry/react (incl. Tracing)48.28 kB+0.58%+275 B 🔺
@sentry/vue32.63 kB+0.64%+207 B 🔺
@sentry/vue (incl. Tracing)47.84 kB+0.52%+243 B 🔺
@sentry/svelte27.5 kB+0.28%+76 B 🔺
CDN Bundle29.89 kB+0.33%+96 B 🔺
CDN Bundle (incl. Tracing)47.89 kB-0.64%-305 B 🔽
CDN Bundle (incl. Logs, Metrics)31.44 kB+0.34%+105 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics)49.24 kB-0.53%-258 B 🔽
CDN Bundle (incl. Replay, Logs, Metrics)70.78 kB+0.23%+158 B 🔺
CDN Bundle (incl. Tracing, Replay)85.4 kB-0.15%-123 B 🔽
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)86.68 kB-0.11%-93 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback)91.19 kB-0.2%-179 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)92.45 kB-0.19%-172 B 🔽
CDN Bundle - uncompressed88.94 kB+0.41%+356 B 🔺
CDN Bundle (incl. Tracing) - uncompressed145.03 kB-0.54%-774 B 🔽
CDN Bundle (incl. Logs, Metrics) - uncompressed93.65 kB+0.39%+355 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed149 kB-0.52%-775 B 🔽
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed218.62 kB+0.24%+507 B 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed264.05 kB-0.24%-621 B 🔽
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed268 kB-0.24%-622 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed277.75 kB-0.23%-621 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed281.69 kB-0.23%-622 B 🔽
@sentry/nextjs (client)50.67 kB+0.44%+220 B 🔺
@sentry/sveltekit (client)46.37 kB+0.54%+248 B 🔺
@sentry/core/server76.5 kB+0.56%+424 B 🔺
@sentry/core/browser63.63 kB+0.66%+413 B 🔺
@sentry/node-core61.51 kB-0.35%-216 B 🔽
@sentry/node122.65 kB-4.68%-6.02 kB 🔽
@sentry/node - without tracing73.55 kB-0.75%-554 B 🔽
@sentry/aws-serverless84.74 kB-0.85%-718 B 🔽
@sentry/cloudflare (withSentry) - minified176.01 kB+1.05%+1.82 kB 🔺
@sentry/cloudflare (withSentry)437.76 kB+0.55%+2.36 kB 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection)69.95 kBaddedadded
@sentry/node/light50.4 kBaddedadded

View base workflow run

@nicohrubec

Copy link
Copy Markdown
MemberAuthor

Combining into #21758 — the changes are small enough to ship as a single PR.

@nicohrubec
nicohrubec deleted the nh/aws-lambda-tests branch June 24, 2026 14:16
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@nicohrubec