Skip to content

[test-optimization] [SDTEST-1784] Co-relate Playwright tests with RUM sessions#5509

Merged
Mariovido merged 33 commits intomasterfrom
mario.vidal/correlate_playwright_rum
Apr 3, 2025
Merged

[test-optimization] [SDTEST-1784] Co-relate Playwright tests with RUM sessions#5509
Mariovido merged 33 commits intomasterfrom
mario.vidal/correlate_playwright_rum

Conversation

@Mariovido
Copy link
Copy Markdown
Contributor

@Mariovido Mariovido commented Apr 1, 2025

What does this PR do?

This PRs co-relate Playwright E2E reporting to RUM sessions. We will co-relate them using a cookie that will contain the trace_id of the span.

Motivation

There have been some Feature Requests where some customers wanted to have something similar to what we have now with the Cypress and Selenium E2E tests.

Plugin Checklist

Additional Notes

Example on how will be shown:
Screenshot 2025-04-02 at 11 31 52

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 1, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 79.18%. Comparing base (b94052c) to head (5d60ddb).
Report is 8 commits behind head on master.

Files with missing lines Patch % Lines
...ages/datadog-instrumentations/src/helpers/hooks.js 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5509      +/-   ##
==========================================
- Coverage   79.18%   79.18%   -0.01%     
==========================================
  Files         512      512              
  Lines       23158    23159       +1     
==========================================
  Hits        18338    18338              
- Misses       4820     4821       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2025

Overall package size

Self size: 9.2 MB
Deduped: 101.5 MB
No deduping: 102.02 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | @datadog/libdatadog | 0.5.0 | 29.83 MB | 29.83 MB | | @datadog/native-appsec | 8.5.1 | 19.26 MB | 19.27 MB | | @datadog/native-iast-taint-tracking | 3.3.0 | 13.77 MB | 13.78 MB | | @datadog/pprof | 5.6.0 | 9.79 MB | 10.16 MB | | @opentelemetry/core | 1.30.1 | 908.66 kB | 7.16 MB | | protobufjs | 7.4.0 | 2.77 MB | 5.42 MB | | @datadog/wasm-js-rewriter | 3.1.0 | 2.37 MB | 2.52 MB | | @datadog/native-metrics | 3.1.0 | 1.06 MB | 1.46 MB | | @opentelemetry/api | 1.8.0 | 1.21 MB | 1.21 MB | | import-in-the-middle | 1.13.1 | 117.64 kB | 839.26 kB | | source-map | 0.7.4 | 226 kB | 226 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | lru-cache | 7.18.3 | 133.92 kB | 133.92 kB | | pprof-format | 2.1.0 | 111.69 kB | 111.69 kB | | @datadog/sketches-js | 2.1.1 | 109.9 kB | 109.9 kB | | lodash.sortby | 4.7.0 | 75.76 kB | 75.76 kB | | ignore | 5.3.2 | 53.63 kB | 53.63 kB | | istanbul-lib-coverage | 3.2.0 | 29.34 kB | 29.34 kB | | rfdc | 1.4.1 | 27.15 kB | 27.15 kB | | @isaacs/ttlcache | 1.4.1 | 25.2 kB | 25.2 kB | | tlhunter-sorted-set | 0.1.0 | 24.94 kB | 24.94 kB | | dc-polyfill | 0.1.6 | 24.56 kB | 24.56 kB | | shell-quote | 1.8.2 | 23.54 kB | 23.54 kB | | limiter | 1.1.5 | 23.17 kB | 23.17 kB | | retry | 0.13.1 | 18.85 kB | 18.85 kB | | semifies | 1.0.0 | 15.84 kB | 15.84 kB | | jest-docblock | 29.7.0 | 8.99 kB | 12.76 kB | | crypto-randomuuid | 1.0.0 | 11.18 kB | 11.18 kB | | ttl-set | 1.0.0 | 4.61 kB | 9.69 kB | | path-to-regexp | 0.1.12 | 6.6 kB | 6.6 kB | | koalas | 1.0.2 | 6.47 kB | 6.47 kB | | module-details-from-path | 1.0.3 | 4.47 kB | 4.47 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

@datadog-datadog-prod-us1
Copy link
Copy Markdown

datadog-datadog-prod-us1 Bot commented Apr 1, 2025

Datadog Report

Branch report: mario.vidal/correlate_playwright_rum
Commit report: e700a1b
Test service: dd-trace-js-integration-tests

✅ 0 Failed, 922 Passed, 0 Skipped, 14m 9.98s Total Time

@pr-commenter
Copy link
Copy Markdown

pr-commenter Bot commented Apr 1, 2025

Benchmarks

Benchmark execution time: 2025-04-02 13:49:58

Comparing candidate commit 6daf14469747006c1610b119ec344b79e2af7983 in PR branch mario.vidal/correlate_playwright_rum with baseline commit b94052c in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 947 metrics, 16 unstable metrics.

@Mariovido Mariovido marked this pull request as ready for review April 1, 2025 19:01
@Mariovido Mariovido requested review from a team as code owners April 1, 2025 19:01
@Mariovido Mariovido requested a review from a team as a code owner April 1, 2025 19:01
anmarchenko
anmarchenko previously approved these changes Apr 2, 2025
@Mariovido Mariovido force-pushed the mario.vidal/correlate_playwright_rum branch from 213155e to 5964b96 Compare April 2, 2025 12:01
@Mariovido Mariovido marked this pull request as draft April 2, 2025 13:52
@Mariovido Mariovido force-pushed the mario.vidal/correlate_playwright_rum branch from 6daf144 to 65f5ae3 Compare April 2, 2025 13:53
@Mariovido Mariovido marked this pull request as ready for review April 2, 2025 14:09
@Mariovido Mariovido merged commit 21e0408 into master Apr 3, 2025
427 checks passed
@Mariovido Mariovido deleted the mario.vidal/correlate_playwright_rum branch April 3, 2025 09:55
@wconti27 wconti27 mentioned this pull request Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants