Skip to content

[RUM-10753] Configure JUnit test tasks to log FAILED and STANDARD_ERROR events#3163

Merged
hamorillo merged 1 commit into
developfrom
hector.morilloprieto/RUM-10753
Jan 29, 2026
Merged

[RUM-10753] Configure JUnit test tasks to log FAILED and STANDARD_ERROR events#3163
hamorillo merged 1 commit into
developfrom
hector.morilloprieto/RUM-10753

Conversation

@hamorillo

Copy link
Copy Markdown
Contributor

What does this PR do?

This PR configures JUnit test tasks to log FAILED events and STANDARD_ERROR output, enabling visibility of diagnostic information (like Forge seeds) when tests fail in CI.

You can see an example of how it will look like here.

image

Motivation

Before this PR, when a test failed in CI, we didn’t have access to the Forge seed, so it wasn’t possible to reproduce the same test. This is especially important when dealing with flaky tests.

Additional Notes

This change will make the Gradle tasks more verbose, but in my opinion this is acceptable, since we’ll only look at the output when tests are failing, so it shouldn’t affect us much.

In addition, if you run ./local_ci.sh --test, you will see a number of tests that write to STANDARD_ERROR or throw exceptions but still pass. It could be a good idea to take a look at them and verify that the tests are working as expected.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Configure JUnit test tasks to log FAILED events and STANDARD_ERROR output, enabling visibility of diagnostic information (like Forge seeds) when tests fail in CI.
@hamorillo
hamorillo force-pushed the hector.morilloprieto/RUM-10753 branch from c3bb45b to 4d40871 Compare January 28, 2026 14:51
@datadog-official

datadog-official Bot commented Jan 28, 2026

Copy link
Copy Markdown

🎯 Code Coverage
Patch Coverage: 100.00%
Overall Coverage: 66.09% (+0.01%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 4d40871 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.95%. Comparing base (b54d77f) to head (4d40871).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3163      +/-   ##
===========================================
- Coverage    70.98%   70.95%   -0.02%     
===========================================
  Files          912      912              
  Lines        33548    33548              
  Branches      5640     5640              
===========================================
- Hits         23811    23803       -8     
- Misses        8158     8165       +7     
- Partials      1579     1580       +1     

see 38 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hamorillo
hamorillo marked this pull request as ready for review January 28, 2026 15:56
@hamorillo
hamorillo requested a review from a team as a code owner January 28, 2026 15:56
@hamorillo
hamorillo merged commit 39d8b4a into develop Jan 29, 2026
26 checks passed
@hamorillo
hamorillo deleted the hector.morilloprieto/RUM-10753 branch January 29, 2026 07:40
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.

3 participants