Skip to content

Conversation

@krystofwoldrich
Copy link
Contributor

@krystofwoldrich krystofwoldrich commented Feb 24, 2025

Add more detox tests.

Covers transaction creation

#skip-changelog

@github-actions
Copy link
Contributor

github-actions bot commented Feb 24, 2025

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 419.13 ms 451.88 ms 32.75 ms
Size 17.75 MiB 20.12 MiB 2.37 MiB

Baseline results on branch: capture-app-start-errors

Startup times

Revision Plain With Sentry Diff
454f971 436.27 ms 478.76 ms 42.48 ms
208f4af 439.23 ms 427.31 ms -11.92 ms
1a89392 412.59 ms 424.69 ms 12.10 ms
555070f 438.67 ms 428.30 ms -10.37 ms
c8578fb 397.17 ms 415.72 ms 18.55 ms
b6d933c 442.52 ms 461.82 ms 19.30 ms
8d0daf6 422.36 ms 449.40 ms 27.04 ms
b75148e 440.04 ms 421.36 ms -18.68 ms
18a5066 427.89 ms 436.47 ms 8.57 ms
5625ce7 476.74 ms 488.24 ms 11.50 ms

App size

Revision Plain With Sentry Diff
454f971 17.75 MiB 20.11 MiB 2.37 MiB
208f4af 17.75 MiB 20.11 MiB 2.37 MiB
1a89392 17.75 MiB 20.11 MiB 2.37 MiB
555070f 17.75 MiB 20.11 MiB 2.37 MiB
c8578fb 17.75 MiB 20.11 MiB 2.36 MiB
b6d933c 17.75 MiB 20.11 MiB 2.37 MiB
8d0daf6 17.75 MiB 20.11 MiB 2.36 MiB
b75148e 17.75 MiB 20.11 MiB 2.37 MiB
18a5066 17.75 MiB 20.11 MiB 2.37 MiB
5625ce7 17.75 MiB 20.11 MiB 2.37 MiB

Previous results on branch: kw-test-transactions

Startup times

Revision Plain With Sentry Diff
22661b1 404.40 ms 435.05 ms 30.64 ms

App size

Revision Plain With Sentry Diff
22661b1 17.75 MiB 20.12 MiB 2.37 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Feb 24, 2025

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 363.56 ms 355.34 ms -8.22 ms
Size 7.15 MiB 8.39 MiB 1.24 MiB

Baseline results on branch: capture-app-start-errors

Startup times

Revision Plain With Sentry Diff
b6d933c+dirty 398.43 ms 456.62 ms 58.19 ms
555070f+dirty 388.25 ms 424.44 ms 36.19 ms
454f971+dirty 439.04 ms 423.88 ms -15.16 ms
b75148e+dirty 428.91 ms 461.26 ms 32.35 ms
5625ce7+dirty 358.15 ms 416.65 ms 58.50 ms
18a5066+dirty 370.06 ms 414.10 ms 44.04 ms
208f4af+dirty 346.93 ms 402.77 ms 55.84 ms
8d0daf6+dirty 393.58 ms 394.84 ms 1.26 ms
1a89392+dirty 425.56 ms 530.65 ms 105.09 ms
c8578fb+dirty 374.17 ms 362.06 ms -12.11 ms

App size

Revision Plain With Sentry Diff
b6d933c+dirty 7.15 MiB 8.38 MiB 1.23 MiB
555070f+dirty 7.15 MiB 8.38 MiB 1.23 MiB
454f971+dirty 7.15 MiB 8.38 MiB 1.23 MiB
b75148e+dirty 7.15 MiB 8.38 MiB 1.23 MiB
5625ce7+dirty 7.15 MiB 8.38 MiB 1.23 MiB
18a5066+dirty 7.15 MiB 8.38 MiB 1.23 MiB
208f4af+dirty 7.15 MiB 8.38 MiB 1.23 MiB
8d0daf6+dirty 7.15 MiB 8.38 MiB 1.23 MiB
1a89392+dirty 7.15 MiB 8.38 MiB 1.23 MiB
c8578fb+dirty 7.15 MiB 8.38 MiB 1.23 MiB

Previous results on branch: kw-test-transactions

Startup times

Revision Plain With Sentry Diff
22661b1+dirty 440.71 ms 463.57 ms 22.86 ms

App size

Revision Plain With Sentry Diff
22661b1+dirty 7.15 MiB 8.39 MiB 1.24 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Feb 24, 2025

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1226.36 ms 1221.90 ms -4.46 ms
Size 3.19 MiB 4.32 MiB 1.13 MiB

Baseline results on branch: capture-app-start-errors

Startup times

Revision Plain With Sentry Diff
18a5066+dirty 1244.20 ms 1251.54 ms 7.34 ms
b6d933c+dirty 1228.02 ms 1235.32 ms 7.30 ms
1a89392+dirty 1229.00 ms 1234.78 ms 5.78 ms
c8578fb+dirty 1225.49 ms 1224.29 ms -1.20 ms
208f4af+dirty 1213.08 ms 1223.82 ms 10.73 ms
454f971+dirty 1233.51 ms 1239.18 ms 5.67 ms
8d0daf6+dirty 1227.80 ms 1236.83 ms 9.03 ms
b75148e+dirty 1202.72 ms 1212.04 ms 9.32 ms
5625ce7+dirty 1219.73 ms 1223.80 ms 4.07 ms
555070f+dirty 1223.61 ms 1227.57 ms 3.96 ms

App size

Revision Plain With Sentry Diff
18a5066+dirty 3.19 MiB 4.26 MiB 1.08 MiB
b6d933c+dirty 3.19 MiB 4.26 MiB 1.08 MiB
1a89392+dirty 3.19 MiB 4.26 MiB 1.08 MiB
c8578fb+dirty 3.19 MiB 4.30 MiB 1.12 MiB
208f4af+dirty 3.19 MiB 4.25 MiB 1.06 MiB
454f971+dirty 3.19 MiB 4.31 MiB 1.12 MiB
8d0daf6+dirty 3.19 MiB 4.30 MiB 1.12 MiB
b75148e+dirty 3.19 MiB 4.25 MiB 1.07 MiB
5625ce7+dirty 3.19 MiB 4.26 MiB 1.08 MiB
555070f+dirty 3.19 MiB 4.25 MiB 1.06 MiB

Previous results on branch: kw-test-transactions

Startup times

Revision Plain With Sentry Diff
22661b1+dirty 1219.53 ms 1222.61 ms 3.08 ms

App size

Revision Plain With Sentry Diff
22661b1+dirty 3.19 MiB 4.32 MiB 1.13 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Feb 24, 2025

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1215.35 ms 1237.69 ms 22.34 ms
Size 2.63 MiB 3.75 MiB 1.12 MiB

Baseline results on branch: capture-app-start-errors

Startup times

Revision Plain With Sentry Diff
18a5066+dirty 1218.00 ms 1212.27 ms -5.73 ms
b6d933c+dirty 1217.71 ms 1220.58 ms 2.87 ms
1a89392+dirty 1220.12 ms 1216.69 ms -3.43 ms
c8578fb+dirty 1218.55 ms 1217.23 ms -1.32 ms
208f4af+dirty 1209.44 ms 1217.13 ms 7.70 ms
454f971+dirty 1220.55 ms 1225.82 ms 5.27 ms
8d0daf6+dirty 1230.90 ms 1233.16 ms 2.27 ms
b75148e+dirty 1221.53 ms 1220.85 ms -0.68 ms
5625ce7+dirty 1226.98 ms 1217.77 ms -9.21 ms
555070f+dirty 1213.59 ms 1217.79 ms 4.20 ms

App size

Revision Plain With Sentry Diff
18a5066+dirty 2.63 MiB 3.70 MiB 1.06 MiB
b6d933c+dirty 2.63 MiB 3.70 MiB 1.06 MiB
1a89392+dirty 2.63 MiB 3.70 MiB 1.06 MiB
c8578fb+dirty 2.63 MiB 3.74 MiB 1.11 MiB
208f4af+dirty 2.63 MiB 3.69 MiB 1.05 MiB
454f971+dirty 2.63 MiB 3.74 MiB 1.11 MiB
8d0daf6+dirty 2.63 MiB 3.74 MiB 1.11 MiB
b75148e+dirty 2.63 MiB 3.69 MiB 1.06 MiB
5625ce7+dirty 2.63 MiB 3.70 MiB 1.06 MiB
555070f+dirty 2.63 MiB 3.69 MiB 1.05 MiB

Previous results on branch: kw-test-transactions

Startup times

Revision Plain With Sentry Diff
22661b1+dirty 1219.87 ms 1225.78 ms 5.90 ms

App size

Revision Plain With Sentry Diff
22661b1+dirty 2.63 MiB 3.75 MiB 1.12 MiB

@krystofwoldrich krystofwoldrich marked this pull request as ready for review February 24, 2025 16:34
Copy link
Contributor

@antonis antonis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀
Thank you for adding more tests Krystof 🙇

@krystofwoldrich krystofwoldrich merged commit ae342a3 into capture-app-start-errors Feb 25, 2025
68 checks passed
@krystofwoldrich krystofwoldrich deleted the kw-test-transactions branch February 25, 2025 11:46
Sign up for free to 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.

3 participants