Skip to content

test: verify network tracking end to end - #8722

Open
denrase wants to merge 3 commits into
mainfrom
test/verify-ios-27-network-swizzling
Open

test: verify network tracking end to end#8722
denrase wants to merge 3 commits into
mainfrom
test/verify-ios-27-network-swizzling

Conversation

@denrase

@denrase denrase commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

📜 Description

  • Add a real URLSession request test that verifies network breadcrumb URL, method, status code, and duration.
  • Add a delayed local response and verify the resulting HTTP span includes the server delay.
  • Include both end-to-end tests in the local test-server plan.
  • Keep failed-request envelope snapshots isolated from unrelated network breadcrumbs.

💡 Motivation and Context

Verify that network breadcrumbs and HTTP performance span timing remain correct with potential Swift-backed networking changes in iOS 27.

Refs #8127

💚 How did you test it?

  • swift test --package-path test-server
  • Network breadcrumb end-to-end test on an iOS 27 simulator
  • HTTP span timing end-to-end test on an iOS 27 simulator using test-without-building

📝 Checklist

  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.
  • If I added a new public API, I also added it to the SentryObjC wrapper.

#skip-changelog

Add a real URLSession request test that waits for the public
beforeBreadcrumb callback and verifies URL, method, status, and timing.

Include it in the local server plan and isolate the failed-request snapshot
from unrelated breadcrumbs.

Refs #8127
@denrase denrase self-assigned this Aug 7, 2026
@denrase denrase added the run-full-ci Allows gated GitHub Action workflows to run for a labelled pull request label Aug 7, 2026
@denrase
denrase marked this pull request as ready for review August 7, 2026 13:25
@denrase
denrase marked this pull request as draft August 7, 2026 13:40
@denrase denrase removed the run-full-ci Allows gated GitHub Action workflows to run for a labelled pull request label Aug 7, 2026
Add a delayed local response and verify the network span includes the
server delay when running through URLSession swizzling.

Refs #8127
@denrase denrase changed the title test: Verify network breadcrumbs end to end test: verify network tracking end to end Aug 7, 2026
@denrase
denrase marked this pull request as ready for review August 7, 2026 14:27
@denrase denrase added the run-full-ci Allows gated GitHub Action workflows to run for a labelled pull request label Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-full-ci Allows gated GitHub Action workflows to run for a labelled pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant