[WIP] Request manager in Swift
:scroll: Description
:bulb: Motivation and Context
:green_heart: How did you test it?
:pencil: Checklist
You have to check all boxes before merging:
- [ ] I added tests to verify the changes.
- [ ] No new PII added or SDK only sends newly added PII if
sendDefaultPIIis 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.
#skip-changelog
Codecov Report
Attention: Patch coverage is 73.91304% with 6 lines in your changes missing coverage. Please review.
Project coverage is 8.305%. Comparing base (
95cfd20) to head (632ca2c). Report is 1 commits behind head on main.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| ...es/Swift/Tools/SentryQueueableRequestManager.swift | 73.913% | 6 Missing :warning: |
:exclamation: There is a different number of reports uploaded between BASE (95cfd20) and HEAD (632ca2c). Click for more details.
HEAD has 3 uploads less than BASE
Flag BASE (95cfd20) HEAD (632ca2c) 4 1
Additional details and impacted files
@@ Coverage Diff @@
## main #5452 +/- ##
=============================================
- Coverage 86.221% 8.305% -77.916%
=============================================
Files 399 367 -32
Lines 34647 26812 -7835
Branches 14983 57 -14926
=============================================
- Hits 29873 2227 -27646
- Misses 4726 24585 +19859
+ Partials 48 0 -48
| Files with missing lines | Coverage Δ | |
|---|---|---|
| Sources/Sentry/SentrySpotlightTransport.m | 0.000% <ø> (-79.311%) |
:arrow_down: |
| Sources/Sentry/SentryTransportFactory.m | 81.578% <ø> (-18.422%) |
:arrow_down: |
| ...es/Swift/Tools/SentryQueueableRequestManager.swift | 73.913% <73.913%> (ø) |
... and 390 files with indirect coverage changes
Continue to review full report in Codecov by Sentry.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 95cfd20...632ca2c. Read the comment docs.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
Performance metrics :rocket:
| Plain | With Sentry | Diff | |
|---|---|---|---|
| Startup time | 1212.65 ms | 1241.02 ms | 28.37 ms |
| Size | 23.75 KiB | 847.77 KiB | 824.02 KiB |
Baseline results on branch: main
Startup times
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| c5232eadff3e28f984b91f2f5ce629ad7381aad1 | 1226.06 ms | 1246.57 ms | 20.50 ms |
| 464117d76c03363a8a5699b3edd07c57398c9518 | 1240.46 ms | 1252.91 ms | 12.45 ms |
| 94e1968f3d4f4f1e1889efef776458ac3b0157b2 | 1230.22 ms | 1253.33 ms | 23.11 ms |
| 3ea21f5004b1b347fe430434d623860de393f15f | 1250.80 ms | 1258.88 ms | 8.08 ms |
| b1783fd3e447cebe342a4bc2c49e7ea0db3d282e | 1216.67 ms | 1238.75 ms | 22.08 ms |
| 794f87ff4fbac22a85fdfb6fa78d2f10479f03c2 | 1225.78 ms | 1243.46 ms | 17.68 ms |
| 17afc4b3c47807a1cf8010aab1faffd6c99ae97a | 1228.94 ms | 1251.10 ms | 22.16 ms |
| 6e31b7c796f20edaf39ba795667596e7bf876c43 | 1230.80 ms | 1246.29 ms | 15.49 ms |
| 25e00aa395feb46c07df40f02805b43c8fd716cd | 1236.98 ms | 1250.98 ms | 14.00 ms |
| 4c00f8ce3aa9f8bb8560977533e170613df5d397 | 1231.62 ms | 1237.76 ms | 6.14 ms |
App size
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| c5232eadff3e28f984b91f2f5ce629ad7381aad1 | 22.85 KiB | 413.98 KiB | 391.13 KiB |
| 464117d76c03363a8a5699b3edd07c57398c9518 | 21.58 KiB | 705.40 KiB | 683.82 KiB |
| 94e1968f3d4f4f1e1889efef776458ac3b0157b2 | 21.58 KiB | 614.74 KiB | 593.15 KiB |
| 3ea21f5004b1b347fe430434d623860de393f15f | 22.84 KiB | 402.63 KiB | 379.78 KiB |
| b1783fd3e447cebe342a4bc2c49e7ea0db3d282e | 22.31 KiB | 818.59 KiB | 796.28 KiB |
| 794f87ff4fbac22a85fdfb6fa78d2f10479f03c2 | 20.76 KiB | 401.37 KiB | 380.61 KiB |
| 17afc4b3c47807a1cf8010aab1faffd6c99ae97a | 20.76 KiB | 436.25 KiB | 415.49 KiB |
| 6e31b7c796f20edaf39ba795667596e7bf876c43 | 21.58 KiB | 614.65 KiB | 593.07 KiB |
| 25e00aa395feb46c07df40f02805b43c8fd716cd | 23.76 KiB | 821.43 KiB | 797.67 KiB |
| 4c00f8ce3aa9f8bb8560977533e170613df5d397 | 20.76 KiB | 419.62 KiB | 398.86 KiB |
Jumping in here even though it is still in draft, what made you rewrite the file in Swift?
👀