designcourse icon indicating copy to clipboard operation
designcourse copied to clipboard

👷 [RUM-259] update test/app to use local tarballs

Open thomas-lebeau opened this issue 1 year ago • 2 comments

Motivation

With this PR, all e2e tests now run all npm setup using the packed version of the sdk packages, this will help catch cases were we deploy broken packages to npm.

Changes

  • add pack command to all packages' package.json
  • pack all packages before building test/app
  • use packed version of sdk in test/app dependencies

Testing

  • [x] Local
  • [ ] Staging
  • [ ] Unit
  • [x] End to end

I have gone over the contributing documentation.

thomas-lebeau avatar Feb 12 '25 15:02 thomas-lebeau

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.93%. Comparing base (515dc4e) to head (b19fe86).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3335   +/-   ##
=======================================
  Coverage   91.93%   91.93%           
=======================================
  Files         309      309           
  Lines        8105     8105           
  Branches     1838     1838           
=======================================
  Hits         7451     7451           
  Misses        654      654           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov-commenter avatar Feb 12 '25 15:02 codecov-commenter

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 146.96 KiB 146.96 KiB 0 B 0.00%
Logs 51.38 KiB 51.38 KiB 0 B 0.00%
Rum Slim 106.19 KiB 106.19 KiB 0 B 0.00%
Worker 23.59 KiB 23.59 KiB 0 B 0.00%
🚀 CPU Performance
Action Name Base Average Cpu Time (ms) Local Average Cpu Time (ms) 𝚫
addglobalcontext 0.004 0.003 -0.001
addaction 0.085 0.058 -0.027
addtiming 0.002 0.001 -0.001
adderror 0.093 0.057 -0.036
startstopsessionreplayrecording 0.025 0.020 -0.006
startview 0.619 0.463 -0.155
logmessage 0.043 0.031 -0.012
🧠 Memory Performance
Action Name Base Consumption Memory (bytes) Local Consumption Memory (bytes) 𝚫 (bytes)
addglobalcontext 26.60 KiB 28.75 KiB 2.16 KiB
addaction 53.04 KiB 58.34 KiB 5.30 KiB
addtiming 25.24 KiB 26.26 KiB 1.02 KiB
adderror 56.16 KiB 62.14 KiB 5.98 KiB
startstopsessionreplayrecording 24.85 KiB 24.10 KiB -759 B
startview 423.35 KiB 427.29 KiB 3.93 KiB
logmessage 52.58 KiB 59.57 KiB 6.99 KiB

🔗 RealWorld

cit-pr-commenter[bot] avatar Feb 12 '25 15:02 cit-pr-commenter[bot]