Repository navigation
fix(test-optimization): restore pre-defer suite reporting - #10017
Conversation
Overall package sizeSelf size: 8.42 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 3.3.3 | 125.43 kB | 445.14 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | dc-polyfill | 0.1.11 | 25.74 kB | 25.74 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
|
✅ All CI checks and tests passed. 🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: c6599ed | Docs | View more details | Give us feedback! |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8118ae7569
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
![datadog-official[bot]](https://avatars.githubusercontent.com/in/1358509?s=60&v=4){kind=small}
There was a problem hiding this comment.
Finishing a suite span does not export it while its parent module and session spans remain open. Normal framework runs therefore keep suites buffered until finalization, so the change does not provide immediate suite reporting.
🤖 Datadog Autotest · Commit 8118ae7 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
BenchmarksBenchmark execution time: 2026-08-28 08:24:25 Comparing candidate commit c6599ed in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 2295 metrics, 15 unstable metrics.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #10017 +/- ##
==========================================
- Coverage 98.58% 98.58% -0.01%
==========================================
Files 991 991
Lines 149573 149346 -227
Branches 13089 13066 -23
==========================================
- Hits 147460 147236 -224
+ Misses 2113 2110 -3 Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6043445034
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bc5752e692
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5c3073991a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
juan-fernandez
changed the title
|
Codex Review: Didn't find any major issues. 🎉 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
There was a problem hiding this comment.
More details
Suite spans now finish at each framework boundary and use normal trace buffering. The changed callers and exporter contain no remaining deferred-suite references.
🤖 Datadog Autotest · Commit 9aa623d · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
…rt-test-suites-immediately # Conflicts: # packages/dd-trace/src/ci-visibility/exporters/ci-visibility-exporter.js
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c65f73a7e6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. Another round soon, please! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
2 participants
What does this PR do?
Restores test-suite reporting to its behavior before deferred-suite storage was introduced. Framework integrations finish suite spans at their natural lifecycle boundary, while normal trace buffering decides when the completed trace is sent. The exporter no longer retains, rewrites, or separately replays suite events.
This does not add a suite-level partial-flush boundary or promise immediate delivery. Mocha keeps its pre-existing framework-error behavior: if reporter finalization fails while suite spans are still buffered, the suite, module, and session receive the framework error.
Motivation
Deferred-suite storage added cross-framework lifecycle plumbing and late span mutation that proved undesirable. Returning to the established reporting path removes that machinery without changing the normal batching behavior each framework already had.
This change is extracted from #9987 so the lifecycle rollback can be reviewed independently while #9987 remains the combined E2E validation branch.
Additional Notes
Verification:
git diff --check