Skip to content

test(integration): Add support for snapshots in integration tests - #7145

Merged
billyvg merged 5 commits into
developfrom
test-replay-add-snapshot-tests
Feb 13, 2023
Merged

test(integration): Add support for snapshots in integration tests#7145
billyvg merged 5 commits into
developfrom
test-replay-add-snapshot-tests

Conversation

@billyvg

Copy link
Copy Markdown
Member
  • adds yarn test:update-snapshots script to update snapshots easily. This will run tests across all browsers and update the snapshots accordingly.

@github-actions

github-actionsBot commented Feb 10, 2023

Copy link
Copy Markdown
Contributor

Replay SDK metrics 🚀

Plain+Sentry+Replay
RevisionValueValueDiffRatioValueDiffRatio
LCPThis PR f7c0a2f79.31 ms98.63 ms+19.32 ms+24.36 %125.45 ms+46.14 ms+58.17 %
Previous cb1981871.11 ms90.74 ms+19.63 ms+27.60 %128.27 ms+57.16 ms+80.38 %
CLSThis PR f7c0a2f0.06 ms0.06 ms-0.00 ms-0.04 %0.06 ms+0.00 ms+0.09 %
Previous cb198180.06 ms0.06 ms+0.00 ms+0.46 %0.06 ms+0.00 ms+0.46 %
CPUThis PR f7c0a2f18.10 %18.91 %+0.81 pp+4.50 %24.47 %+6.37 pp+35.21 %
Previous cb1981812.50 %12.42 %-0.08 pp-0.62 %24.45 %+11.95 pp+95.64 %
JS heap avgThis PR f7c0a2f1.94 MB2.05 MB+110.4 kB+5.69 %2.86 MB+921.47 kB+47.47 %
Previous cb198181.94 MB1.99 MB+48.2 kB+2.48 %3.02 MB+1.07 MB+55.35 %
JS heap maxThis PR f7c0a2f2.3 MB2.58 MB+272.82 kB+11.84 %3.36 MB+1.06 MB+45.86 %
Previous cb198182.31 MB2.56 MB+255.3 kB+11.07 %4.51 MB+2.21 MB+95.74 %
netTxThis PR f7c0a2f0 B0 B0 Bn/a2.23 kB+2.23 kBn/a
Previous cb198180 B0 B0 Bn/a2.52 kB+2.52 kBn/a
netRxThis PR f7c0a2f0 B0 B0 Bn/a41 B+41 Bn/a
Previous cb198180 B0 B0 Bn/a41 B+41 Bn/a
netCountThis PR f7c0a2f000n/a1+1n/a
Previous cb19818000n/a1+1n/a
netTimeThis PR f7c0a2f0.00 ms0.00 ms0.00 msn/a207.30 ms+207.30 msn/a
Previous cb198180.00 ms0.00 ms0.00 msn/a111.50 ms+111.50 msn/a

Previous results on branch: develop

RevisionLCPCLSCPUJS heap avgJS heap maxnetTxnetRxnetCountnetTime
cb19818+57.16 ms+0.00 ms+11.95 pp+1.07 MB+2.21 MB+2.52 kB+41 B+1+111.50 ms
ee301c3+71.07 ms-0.00 ms+12.64 pp+1.07 MB+2.22 MB+2.55 kB+41 B+1+94.67 ms
93c4759+61.10 ms-0.00 ms+12.72 pp+1.08 MB+2.19 MB+2.57 kB+41 B+1+116.75 ms
274f489+63.60 ms-0.00 ms+11.56 pp+1.08 MB+2.2 MB+2.56 kB+41 B+1+116.60 ms
4827b60+58.67 ms+0.00 ms+18.38 pp+1.07 MB+2.22 MB+2.6 kB+41 B+1+91.21 ms
c3806eb+79.85 ms-0.00 ms+12.10 pp+1.05 MB+2.16 MB+2.54 kB+41 B+1+93.58 ms
b49d9f7+52.79 ms-0.00 ms+15.46 pp+1.07 MB+2.21 MB+2.59 kB+41 B+1+70.48 ms
7e57cb7+47.89 ms+0.00 ms+10.59 pp+1.09 MB+2.21 MB+2.59 kB+41 B+1+72.93 ms
538c3a6+63.63 ms+0.00 ms+14.14 pp+1.07 MB+2.16 MB+2.6 kB+41 B+1+112.08 ms
fc7b716+52.44 ms+0.00 ms+14.74 pp+1.09 MB+2.21 MB+2.68 kB+41 B+1+89.64 ms

*) pp - percentage points - an absolute difference between two percentages.
Last updated: Mon, 13 Feb 2023 13:49:57 GMT

@billyvg
billyvgforce-pushed the test-replay-add-snapshot-tests branch from 47a4ea7 to 45137d3CompareFebruary 10, 2023 15:44
@github-actions

github-actionsBot commented Feb 10, 2023

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize
@sentry/browser - ES5 CDN Bundle (gzipped + minified)20.07 KB (0%)
@sentry/browser - ES5 CDN Bundle (minified)62.2 KB (0%)
@sentry/browser - ES6 CDN Bundle (gzipped + minified)18.7 KB (0%)
@sentry/browser - ES6 CDN Bundle (minified)55.34 KB (0%)
@sentry/browser - Webpack (gzipped + minified)20.43 KB (+0.01% 🔺)
@sentry/browser - Webpack (minified)66.78 KB (0%)
@sentry/react - Webpack (gzipped + minified)20.46 KB (+0.01% 🔺)
@sentry/nextjs Client - Webpack (gzipped + minified)47.85 KB (0%)
@sentry/browser + @sentry/tracing - ES5 CDN Bundle (gzipped + minified)26.99 KB (-0.01% 🔽)
@sentry/browser + @sentry/tracing - ES6 CDN Bundle (gzipped + minified)25.24 KB (0%)
@sentry/replay ES6 CDN Bundle (gzipped + minified)42.49 KB (0%)
@sentry/replay - Webpack (gzipped + minified)36.89 KB (0%)
@sentry/browser + @sentry/tracing + @sentry/replay - ES6 CDN Bundle (gzipped + minified)60.17 KB (0%)
@sentry/browser + @sentry/replay - ES6 CDN Bundle (gzipped + minified)53.73 KB (0%)

* adds `yarn test:update-snapshots` script to update snapshots easily. This will run tests across all browsers and update the snapshots accordingly.
@billyvg
billyvgforce-pushed the test-replay-add-snapshot-tests branch from 041d09e to 2ec2598CompareFebruary 10, 2023 18:15
@billyvg
billyvg marked this pull request as ready for review February 10, 2023 21:00
@billyvg
billyvg requested a review from Lms24February 10, 2023 21:00
Comment threadpackages/integration-tests/package.json Outdated
@billyvg
billyvg merged commit 2c514ed into developFeb 13, 2023
@billyvg
billyvg deleted the test-replay-add-snapshot-tests branch February 13, 2023 20:15
AbhiPrasad pushed a commit that referenced this pull request Feb 14, 2023
)
* adds `yarn test:update-snapshots` script to update snapshots easily. This will run tests across all browsers and update the snapshots accordingly.
ramchaik pushed a commit to ramchaik/sentry-javascript that referenced this pull request Feb 14, 2023
…tsentry#7145)
* adds `yarn test:update-snapshots` script to update snapshots easily. This will run tests across all browsers and update the snapshots accordingly.
Sign up for freeto 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

@billyvg@Lms24