Skip to content
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

internal(sample-rn): Add Detox for integration/e2e tests of the rn sample (#4535) #4614

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

krystofwoldrich
Copy link
Member

@krystofwoldrich krystofwoldrich commented Mar 3, 2025

Copy link
Contributor

github-actions bot commented Mar 3, 2025

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 441.90 ms 436.64 ms -5.25 ms
Size 17.75 MiB 20.12 MiB 2.37 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
79976dd 456.94 ms 439.17 ms -17.76 ms
4161236 442.79 ms 433.02 ms -9.77 ms
db44eaf 437.65 ms 436.06 ms -1.59 ms
e1ea4a8 506.82 ms 510.48 ms 3.66 ms
5dff5ee 462.32 ms 456.26 ms -6.06 ms
d361d38 354.10 ms 381.69 ms 27.59 ms
e5bc97b 438.96 ms 437.39 ms -1.57 ms
4a6664f 548.79 ms 585.00 ms 36.21 ms
5571a20 410.55 ms 441.06 ms 30.51 ms
e73d82f 475.82 ms 506.55 ms 30.73 ms

App size

Revision Plain With Sentry Diff
79976dd 17.75 MiB 20.11 MiB 2.36 MiB
4161236 17.75 MiB 20.11 MiB 2.37 MiB
db44eaf 17.74 MiB 20.08 MiB 2.35 MiB
e1ea4a8 17.74 MiB 20.08 MiB 2.34 MiB
5dff5ee 17.75 MiB 20.11 MiB 2.36 MiB
d361d38 17.73 MiB 19.81 MiB 2.08 MiB
e5bc97b 17.74 MiB 20.08 MiB 2.34 MiB
4a6664f 17.73 MiB 19.94 MiB 2.21 MiB
5571a20 17.73 MiB 19.93 MiB 2.19 MiB
e73d82f 17.73 MiB 20.07 MiB 2.33 MiB

Copy link
Contributor

github-actions bot commented Mar 3, 2025

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 387.53 ms 391.51 ms 3.98 ms
Size 7.15 MiB 8.39 MiB 1.23 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
8bda0cc+dirty 383.08 ms 417.40 ms 34.32 ms
b6f8ea2+dirty 397.51 ms 457.40 ms 59.88 ms
63ed251+dirty 485.02 ms 531.16 ms 46.14 ms
7f6950b+dirty 432.72 ms 476.91 ms 44.19 ms
dadc233+dirty 363.19 ms 370.37 ms 7.18 ms
de59d3a+dirty 439.90 ms 454.20 ms 14.30 ms
22e31b6+dirty 295.75 ms 346.73 ms 50.98 ms
15c80ab+dirty 276.38 ms 327.54 ms 51.17 ms
0677344+dirty 288.40 ms 391.44 ms 103.04 ms
60d7316+dirty 475.43 ms 535.26 ms 59.84 ms

App size

Revision Plain With Sentry Diff
8bda0cc+dirty 7.15 MiB 8.38 MiB 1.23 MiB
b6f8ea2+dirty 7.15 MiB 8.38 MiB 1.23 MiB
63ed251+dirty 7.15 MiB 8.35 MiB 1.20 MiB
7f6950b+dirty 7.15 MiB 8.37 MiB 1.22 MiB
dadc233+dirty 7.15 MiB 8.04 MiB 910.84 KiB
de59d3a+dirty 7.15 MiB 8.38 MiB 1.23 MiB
22e31b6+dirty 7.15 MiB 8.10 MiB 981.29 KiB
15c80ab+dirty 7.15 MiB 8.09 MiB 966.13 KiB
0677344+dirty 7.15 MiB 8.07 MiB 949.80 KiB
60d7316+dirty 7.15 MiB 8.38 MiB 1.22 MiB

Copy link
Contributor

github-actions bot commented Mar 3, 2025

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1233.41 ms 1226.48 ms -6.93 ms
Size 2.63 MiB 3.75 MiB 1.12 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
686b3bc+dirty 1210.89 ms 1221.14 ms 10.25 ms
3261206+dirty 1219.49 ms 1224.10 ms 4.61 ms
1d86dd6+dirty 1249.71 ms 1279.16 ms 29.45 ms
d7401ac+dirty 1252.38 ms 1275.04 ms 22.66 ms
13f280b+dirty 1229.25 ms 1226.10 ms -3.15 ms
6e8584e+dirty 1274.50 ms 1296.82 ms 22.32 ms
a5d86e1+dirty 1242.47 ms 1239.94 ms -2.53 ms
5852d77+dirty 1210.16 ms 1222.60 ms 12.44 ms
d2c32bb+dirty 1223.69 ms 1229.49 ms 5.80 ms
18ce5e8+dirty 1247.78 ms 1242.85 ms -4.92 ms

App size

Revision Plain With Sentry Diff
686b3bc+dirty 2.63 MiB 3.74 MiB 1.11 MiB
3261206+dirty 2.63 MiB 3.69 MiB 1.06 MiB
1d86dd6+dirty 2.36 MiB 2.89 MiB 535.43 KiB
d7401ac+dirty 2.36 MiB 2.83 MiB 481.14 KiB
13f280b+dirty 2.36 MiB 3.10 MiB 753.43 KiB
6e8584e+dirty 2.36 MiB 2.88 MiB 533.17 KiB
a5d86e1+dirty 2.36 MiB 3.10 MiB 752.65 KiB
5852d77+dirty 2.63 MiB 3.74 MiB 1.11 MiB
d2c32bb+dirty 2.36 MiB 3.08 MiB 737.22 KiB
18ce5e8+dirty 2.36 MiB 3.12 MiB 778.65 KiB

Copy link
Contributor

github-actions bot commented Mar 3, 2025

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1218.81 ms 1211.89 ms -6.93 ms
Size 3.19 MiB 4.32 MiB 1.13 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
686b3bc+dirty 1219.65 ms 1231.73 ms 12.08 ms
3261206+dirty 1223.00 ms 1226.06 ms 3.06 ms
1d86dd6+dirty 1289.25 ms 1293.36 ms 4.11 ms
d7401ac+dirty 1288.10 ms 1289.54 ms 1.44 ms
13f280b+dirty 1217.73 ms 1229.08 ms 11.35 ms
6e8584e+dirty 1271.71 ms 1281.26 ms 9.55 ms
a5d86e1+dirty 1230.35 ms 1222.47 ms -7.88 ms
5852d77+dirty 1226.88 ms 1236.38 ms 9.50 ms
d2c32bb+dirty 1244.00 ms 1245.77 ms 1.77 ms
18ce5e8+dirty 1244.67 ms 1242.96 ms -1.72 ms

App size

Revision Plain With Sentry Diff
686b3bc+dirty 3.19 MiB 4.31 MiB 1.12 MiB
3261206+dirty 3.19 MiB 4.25 MiB 1.07 MiB
1d86dd6+dirty 2.92 MiB 3.44 MiB 538.27 KiB
d7401ac+dirty 2.92 MiB 3.40 MiB 488.06 KiB
13f280b+dirty 2.92 MiB 3.66 MiB 758.67 KiB
6e8584e+dirty 2.92 MiB 3.44 MiB 536.52 KiB
a5d86e1+dirty 2.92 MiB 3.66 MiB 757.10 KiB
5852d77+dirty 3.19 MiB 4.30 MiB 1.12 MiB
d2c32bb+dirty 2.92 MiB 3.64 MiB 742.84 KiB
18ce5e8+dirty 2.92 MiB 3.69 MiB 789.94 KiB

<?xml version="1.0" encoding="utf-8"?>
<network-security-config>
<domain-config cleartextTrafficPermitted="true">
<domain includeSubdomains="true">10.0.2.2</domain>
Copy link
Collaborator

Choose a reason for hiding this comment

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

i: We could add a comment mentioning this is the host machine address


it('Shows Bottom Tab Bar', async () => {
await expect(element(by.text('Performance'))).toBeVisible();
});
Copy link
Collaborator

Choose a reason for hiding this comment

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

What are we testing here, exactly, that the sample app launched?
Will this pass if there is an error right after the screen is rendered?

matrix:
include:
- platform: ios
runs-on: macos-15
Copy link
Collaborator

Choose a reason for hiding this comment

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

are there any issues with latest?

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