andre/crewAI
1
0
Fork 0
mirror of https://github.com/crewAIInc/crewAI.git synced 2026-05-02 15:52:34 +00:00
Code Issues Actions 6 Packages Projects Releases Wiki Activity

fix: fix tool publisher logger when available_exports is found

Browse Source
This commit is contained in:
Lucas Gomide
2025-06-04 11:50:50 -03:00
parent 552921cf83
commit c263cc7244
2 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/crewai/cli/tools/main.py
View File

@@ -89,7 +89,7 @@ class ToolCommand(BaseCommand, PlusAPIMixin):
if available_exports: if available_exports:
console.print( console.print(
f"[green]Found these tools to publish: {', '.join(available_exports)}[/green]" f"[green]Found these tools to publish: {', '.join([e['name'] for e in available_exports])}[/green]"
) )
with tempfile.TemporaryDirectory() as temp_build_dir: with tempfile.TemporaryDirectory() as temp_build_dir:

12
tests/cli/tools/test_main.py
View File

@@ -165,7 +165,7 @@ def test_publish_when_not_in_sync(mock_is_synced, capsys, tool_command):
) )
@patch("crewai.cli.plus_api.PlusAPI.publish_tool") @patch("crewai.cli.plus_api.PlusAPI.publish_tool")
@patch("crewai.cli.tools.main.git.Repository.is_synced", return_value=False) @patch("crewai.cli.tools.main.git.Repository.is_synced", return_value=False)
@patch("crewai.cli.tools.main.extract_available_exports", return_value=["SampleTool"]) @patch("crewai.cli.tools.main.extract_available_exports", return_value=[{"name": "SampleTool"}])
def test_publish_when_not_in_sync_and_force( def test_publish_when_not_in_sync_and_force(
mock_available_exports, mock_available_exports,
mock_is_synced, mock_is_synced,
@@ -200,7 +200,7 @@ def test_publish_when_not_in_sync_and_force(
version="1.0.0", version="1.0.0",
description="A sample tool", description="A sample tool",
encoded_file=unittest.mock.ANY, encoded_file=unittest.mock.ANY,
available_exports=["SampleTool"], available_exports=[{"name": "SampleTool"}],
) )
@@ -216,7 +216,7 @@ def test_publish_when_not_in_sync_and_force(
) )
@patch("crewai.cli.plus_api.PlusAPI.publish_tool") @patch("crewai.cli.plus_api.PlusAPI.publish_tool")
@patch("crewai.cli.tools.main.git.Repository.is_synced", return_value=True) @patch("crewai.cli.tools.main.git.Repository.is_synced", return_value=True)
@patch("crewai.cli.tools.main.extract_available_exports", return_value=["SampleTool"]) @patch("crewai.cli.tools.main.extract_available_exports", return_value=[{"name": "SampleTool"}])
def test_publish_success( def test_publish_success(
mock_available_exports, mock_available_exports,
mock_is_synced, mock_is_synced,
@@ -251,7 +251,7 @@ def test_publish_success(
version="1.0.0", version="1.0.0",
description="A sample tool", description="A sample tool",
encoded_file=unittest.mock.ANY, encoded_file=unittest.mock.ANY,
available_exports=["SampleTool"], available_exports=[{"name": "SampleTool"}],
) )
@@ -266,7 +266,7 @@ def test_publish_success(
read_data=b"sample tarball content", read_data=b"sample tarball content",
) )
@patch("crewai.cli.plus_api.PlusAPI.publish_tool") @patch("crewai.cli.plus_api.PlusAPI.publish_tool")
@patch("crewai.cli.tools.main.extract_available_exports", return_value=["SampleTool"]) @patch("crewai.cli.tools.main.extract_available_exports", return_value=[{"name": "SampleTool"}])
def test_publish_failure( def test_publish_failure(
mock_available_exports, mock_available_exports,
mock_publish, mock_publish,
@@ -304,7 +304,7 @@ def test_publish_failure(
read_data=b"sample tarball content", read_data=b"sample tarball content",
) )
@patch("crewai.cli.plus_api.PlusAPI.publish_tool") @patch("crewai.cli.plus_api.PlusAPI.publish_tool")
@patch("crewai.cli.tools.main.extract_available_exports", return_value=["SampleTool"]) @patch("crewai.cli.tools.main.extract_available_exports", return_value=[{"name": "SampleTool"}])
def test_publish_api_error( def test_publish_api_error(
mock_available_exports, mock_available_exports,
mock_publish, mock_publish,