Skip to content

Use stub Executor for WorkerParameters forgery factory#2949

Merged
0xnm merged 1 commit into
developfrom
nogorodnikov/use-stub-executor-for-workerparameters-forgery
Oct 20, 2025
Merged

Use stub Executor for WorkerParameters forgery factory#2949
0xnm merged 1 commit into
developfrom
nogorodnikov/use-stub-executor-for-workerparameters-forgery

Conversation

@0xnm

@0xnm 0xnm commented Oct 17, 2025

Copy link
Copy Markdown
Member

What does this PR do?

Tiny improvement: avoid creating a real Executor spinning a real thread in the forgery factory.

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)

@0xnm
0xnm requested review from a team as code owners October 17, 2025 15:09
@0xnm
0xnm force-pushed the nogorodnikov/use-stub-executor-for-workerparameters-forgery branch from dc5dfdd to e8721de Compare October 17, 2025 15:12
@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Oct 17, 2025

Copy link
Copy Markdown

🎯 Code Coverage
Patch Coverage: 100.00%
Total Coverage: 100.00% (+29.16%)

View detailed report

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

@codecov-commenter

codecov-commenter commented Oct 17, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.81%. Comparing base (641c42d) to head (e8721de).
⚠️ Report is 1345 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2949      +/-   ##
===========================================
- Coverage    71.02%   70.81%   -0.21%     
===========================================
  Files          829      829              
  Lines        30381    30381              
  Branches      5183     5183              
===========================================
- Hits         21578    21514      -64     
- Misses        7347     7392      +45     
- Partials      1456     1475      +19     

see 32 files with indirect coverage changes

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

@0xnm
0xnm merged commit ad12432 into develop Oct 20, 2025
25 checks passed
@0xnm
0xnm deleted the nogorodnikov/use-stub-executor-for-workerparameters-forgery branch October 20, 2025 07:34
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.

5 participants