Skip to content

Fix flaky test and add more info in case of failure#3650

Merged
estringana merged 1 commit intomasterfrom
estringana/fix-flaky-test
Feb 13, 2026
Merged

Fix flaky test and add more info in case of failure#3650
estringana merged 1 commit intomasterfrom
estringana/fix-flaky-test

Conversation

@estringana
Copy link
Contributor

Description

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented Feb 13, 2026

⚠️ Tests

Fix all issues with Cursor

⚠️ Warnings

🧪 1026 Tests failed

    testSearchPhpBinaries from integration.DDTrace\Tests\Integration\PHPInstallerTest (Fix with Cursor)

testSimplePushAndProcess from laravel-58-test.DDTrace\Tests\Integrations\Laravel\V5_8\QueueTest (Datadog) (Fix with Cursor)
DDTrace\Tests\Integrations\Laravel\V5_8\QueueTest::testSimplePushAndProcess
Test code or tested code printed unexpected output: spanLinksTraceId: 698f4dbf000000003af3c56c960ae4d0
tid: 698f4dbf00000000
hexProcessTraceId: 3af3c56c960ae4d0
hexProcessSpanId: 6b26a9e6fadd8969
processTraceId: 4247955943704945872
processSpanId: 7721045420689688937

phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106
testSimplePushAndProcess from laravel-8x-test.DDTrace\Tests\Integrations\Laravel\V8_x\QueueTest (Datadog) (Fix with Cursor)
DDTrace\Tests\Integrations\Laravel\V8_x\QueueTest::testSimplePushAndProcess
Test code or tested code printed unexpected output: spanLinksTraceId: 698f4d8e000000006523c45a9783682b
tid: 698f4d8e00000000
hexProcessTraceId: 6523c45a9783682b
hexProcessSpanId: 76ad7bbad770eb5a
processTraceId: 7287884515383666731
processSpanId: 8551627309855075162

phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106
View all

ℹ️ Info

❄️ No new flaky tests detected

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

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.21%. Comparing base (ee75439) to head (1a736e0).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3650      +/-   ##
==========================================
+ Coverage   62.11%   62.21%   +0.10%     
==========================================
  Files         141      141              
  Lines       13387    13387              
  Branches     1753     1753              
==========================================
+ Hits         8315     8329      +14     
+ Misses       4273     4260      -13     
+ Partials      799      798       -1     

see 3 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee75439...1a736e0. Read the comment docs.

🚀 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.

@Leiyks Leiyks marked this pull request as ready for review February 13, 2026 17:57
@Leiyks Leiyks requested a review from a team as a code owner February 13, 2026 17:57
@estringana estringana merged commit c31cf60 into master Feb 13, 2026
2068 of 2070 checks passed
@estringana estringana deleted the estringana/fix-flaky-test branch February 13, 2026 17:58
@github-actions github-actions bot added this to the 1.17.0 milestone Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants