Content-Length: 491289 | pFad | https://github.com/getsentry/sentry-react-native/pull/4588

98 test(e2e): Add auto init from JS tests by krystofwoldrich · Pull Request #4588 · getsentry/sentry-react-native · GitHub
Skip to content

test(e2e): Add auto init from JS tests #4588

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 26 commits into from
Feb 25, 2025

Conversation

krystofwoldrich
Copy link
Member

@krystofwoldrich krystofwoldrich commented Feb 24, 2025

Run the sample E2E test with auto init from JS.

The test suit is the same as we don't expect any differences between the manual native init and auto init from JS.

#skip-changelog

Copy link
Contributor

github-actions bot commented Feb 24, 2025

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 408.85 ms 417.96 ms 9.11 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-add-auto-init-tests-ios

Startup times

Revision Plain With Sentry Diff
2d4e016 436.04 ms 427.17 ms -8.88 ms
67f518f 436.47 ms 467.29 ms 30.82 ms

App size

Revision Plain With Sentry Diff
2d4e016 17.75 MiB 20.12 MiB 2.37 MiB
67f518f 17.75 MiB 20.12 MiB 2.37 MiB

Copy link
Contributor

github-actions bot commented Feb 24, 2025

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1214.36 ms 1216.94 ms 2.58 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-add-auto-init-tests-ios

Startup times

Revision Plain With Sentry Diff
67f518f+dirty 1213.96 ms 1221.86 ms 7.90 ms
2d4e016+dirty 1219.94 ms 1231.69 ms 11.76 ms

App size

Revision Plain With Sentry Diff
67f518f+dirty 2.63 MiB 3.75 MiB 1.12 MiB
2d4e016+dirty 2.63 MiB 3.75 MiB 1.12 MiB

Copy link
Contributor

github-actions bot commented Feb 24, 2025

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 395.08 ms 399.00 ms 3.92 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-add-auto-init-tests-ios

Startup times

Revision Plain With Sentry Diff
2d4e016+dirty 379.89 ms 386.67 ms 6.78 ms
67f518f+dirty 356.59 ms 346.38 ms -10.21 ms

App size

Revision Plain With Sentry Diff
2d4e016+dirty 7.15 MiB 8.39 MiB 1.24 MiB
67f518f+dirty 7.15 MiB 8.39 MiB 1.24 MiB

Copy link
Contributor

github-actions bot commented Feb 24, 2025

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1228.08 ms 1238.80 ms 10.72 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-add-auto-init-tests-ios

Startup times

Revision Plain With Sentry Diff
67f518f+dirty 1220.45 ms 1216.22 ms -4.23 ms
2d4e016+dirty 1226.82 ms 1236.27 ms 9.45 ms

App size

Revision Plain With Sentry Diff
67f518f+dirty 3.19 MiB 4.32 MiB 1.13 MiB
2d4e016+dirty 3.19 MiB 4.32 MiB 1.13 MiB

Copy link
Collaborator

@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.

Added a nit pick suggestion but overall LGTM 🎸
Thank you for adding those tests Krystof 🙇

Base automatically changed from kw-test-transactions to capture-app-start-errors February 25, 2025 11:46
@krystofwoldrich krystofwoldrich merged commit 770f9fb into capture-app-start-errors Feb 25, 2025
69 checks passed
@krystofwoldrich krystofwoldrich deleted the kw-add-auto-init-tests-ios branch February 25, 2025 13:25
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.

2 participants








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/getsentry/sentry-react-native/pull/4588

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy