Skip to content

👷 [e2e] group ignored browser log tags by matched pattern#4389

Merged
thomas-lebeau merged 1 commit into
mainfrom
thomas.lebeau/fix-browser-log-ignore-tag
Mar 25, 2026
Merged

👷 [e2e] group ignored browser log tags by matched pattern#4389
thomas-lebeau merged 1 commit into
mainfrom
thomas.lebeau/fix-browser-log-ignore-tag

Conversation

@thomas-lebeau
Copy link
Copy Markdown
Collaborator

@thomas-lebeau thomas-lebeau commented Mar 25, 2026

Motivation

When E2E tests encounter ignored console log messages (e.g. "Layout was forced before the page was fully loaded."), they add a test.fixme tag to flag them. The tag was including the full log.message, which contains dynamic content like {file: "http://172.28.56.32:9324/datadog-rum.js" line: 1}. This made each occurrence unique, preventing proper grouping in test reporting.

Changes

Use the matched ignore pattern as the test.fixme tag value instead of the full log message. This ensures all occurrences of the same warning type are grouped together.

Test instructions

No functional change — just affects how the test.fixme tag value is reported.

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.
  • Updated documentation and/or relevant AGENTS.md file

Use the matched ignore pattern instead of the full log message in the
addTag call, and fix the "Unnexpected" typo.
@thomas-lebeau thomas-lebeau requested a review from a team as a code owner March 25, 2026 12:55
@thomas-lebeau thomas-lebeau changed the title 🐛 fix(e2e): group ignored browser log tags by matched pattern 👷 group ignored browser log tags by matched pattern Mar 25, 2026
@cit-pr-commenter-54b7da
Copy link
Copy Markdown

cit-pr-commenter-54b7da Bot commented Mar 25, 2026

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 176.49 KiB 176.49 KiB 0 B 0.00%
Rum Profiler 6.16 KiB 6.16 KiB 0 B 0.00%
Rum Recorder 27.46 KiB 27.46 KiB 0 B 0.00%
Logs 56.85 KiB 56.85 KiB 0 B 0.00%
Rum Slim 132.13 KiB 132.13 KiB 0 B 0.00%
Worker 23.63 KiB 23.63 KiB 0 B 0.00%
🚀 CPU Performance
Action Name Base CPU Time (ms) Local CPU Time (ms) 𝚫%
RUM - add timing 0.0031 0.0029 -6.45%
RUM - start view 0.0144 0.0138 -4.17%
RUM - start/stop session replay recording 0.0009 0.0008 -11.11%
🧠 Memory Performance
Action Name Base Memory Consumption Local Memory Consumption 𝚫
RUM - add global context 27.21 KiB 26.09 KiB -1.11 KiB
RUM - add action 101.06 KiB 53.06 KiB -48.00 KiB
RUM - add timing 26.90 KiB 26.46 KiB -450 B
RUM - add error 96.23 KiB 55.89 KiB -40.34 KiB
RUM - start/stop session replay recording 25.68 KiB 26.03 KiB +362 B
RUM - start view 472.59 KiB 462.10 KiB -10.50 KiB
Logs - log message 44.62 KiB 44.62 KiB +0 B

🔗 RealWorld

@datadog-official
Copy link
Copy Markdown

datadog-official Bot commented Mar 25, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 77.31% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 44ca9f2 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

@thomas-lebeau thomas-lebeau changed the title 👷 group ignored browser log tags by matched pattern 👷 [e2e] group ignored browser log tags by matched pattern Mar 25, 2026
@thomas-lebeau thomas-lebeau merged commit 131433c into main Mar 25, 2026
22 checks passed
@thomas-lebeau thomas-lebeau deleted the thomas.lebeau/fix-browser-log-ignore-tag branch March 25, 2026 14:53
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants