Skip to content

test_docker_commandline_code_executor.py : 161.66s -> 108.07#6429

Merged
ekzhu merged 5 commits into
microsoft:mainfrom
SongChiYoung:TEST/docker_test_fast
Apr 29, 2025
Merged

test_docker_commandline_code_executor.py : 161.66s -> 108.07#6429
ekzhu merged 5 commits into
microsoft:mainfrom
SongChiYoung:TEST/docker_test_fast

Conversation

@SongChiYoung
Copy link
Copy Markdown
Contributor

Why are these changes needed?

Current autogen-ext's test is too slow.
So, I will search slow test case and makes more fast.

init docker executor function to module 180s->140s
reuse executor at some tests 140s->120s
Remove unnecessary start of docker 120s->110s

Related issue number

Part of #6376

Checks

@ekzhu ekzhu enabled auto-merge (squash) April 29, 2025 20:52
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.51%. Comparing base (bfdf816) to head (52daa96).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6429      +/-   ##
==========================================
- Coverage   78.54%   78.51%   -0.04%     
==========================================
  Files         225      225              
  Lines       16521    16521              
==========================================
- Hits        12977    12971       -6     
- Misses       3544     3550       +6     
Flag Coverage Δ
unittests 78.51% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@ekzhu ekzhu merged commit b0c13a4 into microsoft:main Apr 29, 2025
62 checks passed
ChrisBlaa pushed a commit to ChrisBlaa/autogen that referenced this pull request May 8, 2025
…ft#6429)

## Why are these changes needed?
Current autogen-ext's test is too slow.
So, I will search slow test case and makes more fast.

[init docker executor function to module
180s->140s](microsoft@a3cf70b)
[reuse executor at some tests
140s->120s](microsoft@ca15938)
[Remove unnecessary start of docker
120s->110s](microsoft@6124761)

## Related issue number

<!-- For example: "Closes microsoft#1234" -->
Part of microsoft#6376

## Checks

- [ ] I've included any doc changes needed for
<https://microsoft.github.io/autogen/>. See
<https://github.com/microsoft/autogen/blob/main/CONTRIBUTING.md> to
build and test documentation locally.
- [ ] I've added tests (if relevant) corresponding to the changes
introduced in this PR.
- [x] I've made sure all auto checks have passed.

---------

Co-authored-by: Eric Zhu <ekzhu@users.noreply.github.com>
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.

2 participants