Skip to content

TEST: skip when macos+uv and adding uv venv tests#6387

Merged
ekzhu merged 2 commits intomicrosoft:mainfrom
SongChiYoung:TEST/macos_venv_with_uv_error_when_using_venv
Apr 24, 2025
Merged

TEST: skip when macos+uv and adding uv venv tests#6387
ekzhu merged 2 commits intomicrosoft:mainfrom
SongChiYoung:TEST/macos_venv_with_uv_error_when_using_venv

Conversation

@SongChiYoung
Copy link
Contributor

Why are these changes needed?

The pytest tests test_local_executor_with_custom_venv and test_local_executor_with_custom_venv_in_local_relative_path located in packages/autogen-ext/tests/code_executors/test_commandline_code_executor.py fail when run on macOS (aarch64) using a Python interpreter managed by uv (following the project's recommended development setup).

The failure occurs during the creation of a nested virtual environment using Python's standard venv.EnvBuilder. Specifically, the attempt to run ensurepip inside the newly created venv fails immediately with a SIGABRT signal. The root cause appears to be a dynamic library loading error (dyld error) where the Python executable inside the newly created venv cannot find its required libpythonX.Y.dylib shared library.

So, when MacOS + uv case, skipping that test.
And, adding uv-venv case

Related issue number

Closes #6341

Checks

@ekzhu ekzhu merged commit 0c9fd64 into microsoft:main Apr 24, 2025
60 checks passed
@codecov
Copy link

codecov bot commented Apr 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.36%. Comparing base (cbd8745) to head (83f0caf).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6387   +/-   ##
=======================================
  Coverage   78.36%   78.36%           
=======================================
  Files         221      221           
  Lines       16020    16020           
=======================================
  Hits        12554    12554           
  Misses       3466     3466           
Flag Coverage Δ
unittests 78.36% <ø> (ø)

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.

peterj added a commit to kagent-dev/autogen that referenced this pull request Apr 26, 2025
…n0425

* upstream/main:
  Update website for v0.5.5 (microsoft#6401)
  Document custom message types in teams API docs (microsoft#6400)
  [doc] Clarify selector prompt for SelectorGroupChat (microsoft#6399)
  Add functional termination condition (microsoft#6398)
  Update: implement return_value_as_string for McpToolAdapter  (microsoft#6380)
  Update version to 0.5.5 (microsoft#6397)
  Add example using autogen-core and FastAPI for handoff multi-agent design pattern with streaming and UI (microsoft#6391)
  Update agent documentation  (microsoft#6394)
  AssistantAgent to support Workbench (microsoft#6393)
  TEST: skip when macos+uv and adding uv venv tests (microsoft#6387)
  Add guide for workbench and mcp & bug fixes for create_mcp_server_session (microsoft#6392)

Signed-off-by: Peter Jausovec <peter.jausovec@solo.io>
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.

macOS/uv: venv tests fail (dyld error), not resolved by symlinks=False

2 participants