Skip to content

Conversation

@Leiyks
Copy link
Contributor

@Leiyks Leiyks commented Jan 20, 2026

Description

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented Jan 20, 2026

⚠️ Tests

Fix all issues with Cursor

⚠️ Warnings

❄️ 1 New flaky test detected

testUserLoginSuccessEvent from tests/Integrations/WordPress/V5_9.DDTrace\Tests\Integrations\WordPress\V5_9\AutomatedLoginEventsTest (Datadog) (Fix with Cursor)
DDTrace\Tests\Integrations\WordPress\V5_9\AutomatedLoginEventsTest::testUserLoginSuccessEvent
Failed asserting that 0 matches expected 1.

tests/Integrations/WordPress/AutomatedLoginEventsTestSuite.php:50
tests/Common/RetryTraitVersionGeneric.php:28

🧪 25 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: 696fa5330000000073261c9ac245c4ae
tid: 696fa53300000000
hexProcessTraceId: 73261c9ac245c4ae
hexProcessSpanId: f16c5acc0c1f55ad
processTraceId: 8297350814486611118
processSpanId: 17396379293048591789

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: 696fa58d00000000d8329fbf9b89cdb7
tid: 696fa58d00000000
hexProcessTraceId: d8329fbf9b89cdb7
hexProcessSpanId: 9e6a7d9b3c537f24
processTraceId: 15578689706325036471
processSpanId: 11415074311209451300

phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106
View all
This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 90aade4 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@codecov-commenter
Copy link

codecov-commenter commented Jan 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.91%. Comparing base (01cc88c) to head (90aade4).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3588      +/-   ##
==========================================
- Coverage   62.01%   61.91%   -0.11%     
==========================================
  Files         140      140              
  Lines       13311    13311              
  Branches     1762     1762              
==========================================
- Hits         8255     8241      -14     
- Misses       4267     4280      +13     
- Partials      789      790       +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 01cc88c...90aade4. 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.

Signed-off-by: Alexandre Rulleau <[email protected]>
@Leiyks Leiyks force-pushed the leiyks/fix-verify-alpine-jobs branch from 693ca9d to 90aade4 Compare January 20, 2026 15:43
@Leiyks Leiyks marked this pull request as ready for review January 20, 2026 15:58
@Leiyks Leiyks requested a review from a team as a code owner January 20, 2026 15:58
@Leiyks Leiyks merged commit 0f0c13a into master Jan 20, 2026
1804 of 1921 checks passed
@Leiyks Leiyks deleted the leiyks/fix-verify-alpine-jobs branch January 20, 2026 16:06
@github-actions github-actions bot added this to the 1.16.0 milestone Jan 20, 2026
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.

4 participants