沃巴蒂
ff22ed3a8b
本次同步 09-12 至 09-28 的代码变更，主要内容： 新增功能 - 修改密码：后端新增 ChangePasswordRequest DTO 与UserController/UserService 对应接口，前端新增 ChangePasswordDialog 并在 Header 接入入口 - 文件预览组件重构：file-preview 目录拆分出 VideoPreview（支持 blobUrl 直传）、 ImagePreview、AudioPreview，index.tsx 统一调度各类型预览器 - 新增 my_ops 算子模块：markitdown_parser、mineru_pdf_parser、text_quality_evaluator - 新增 tools/minio-file-server MinIO 文件服务 功能增强 - 数据清洗：cleaning_task_service/scheduler/routes 重构， data_juicer_executor 与 wrapper 增强，OperatorTable/LogsTable/TaskDetail 同步调整 - 数据标注：annotation sync.py、project.py 接口与schema 更新 - 数据采集、算子市场、首页与菜单路由相关页面调整 - DataJuicer 算子接入（useOperatorOperations、dj_ops_schema） 结构调整 - deepflow/ 更名为 deep-flow/（419 个文件重命名） - 新增 helm 部署编排、GitHub Actions 之外的自建镜像构建脚本 未纳入本次提交 - scripts-backup/、runtime/python-executor-backup/、components_backup/（备份目录） - *.backup*、__pycache__ 产物 - zh_core_web_sm-3.8.0-py3-none-any.whl（46MB 构建产物） - DataX/（内嵌独立仓库，远程亦未收录其源码）
223 lines
7.6 KiB
Python
223 lines
7.6 KiB
Python
# Copyright (c) 2025 Bytedance Ltd. and/or its affiliates
|
|
# SPDX-License-Identifier: MIT
|
|
|
|
import os
|
|
from unittest.mock import patch
|
|
|
|
import pytest
|
|
|
|
from src.tools.python_repl import python_repl_tool
|
|
|
|
|
|
class TestPythonReplTool:
|
|
@patch.dict(os.environ, {"ENABLE_PYTHON_REPL": "true"})
|
|
@patch("src.tools.python_repl.repl")
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_successful_code_execution(self, mock_logger, mock_repl):
|
|
# Arrange
|
|
code = "print('Hello, World!')"
|
|
expected_output = "Hello, World!\n"
|
|
mock_repl.run.return_value = expected_output
|
|
|
|
# Act
|
|
result = python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_repl.run.assert_called_once_with(code)
|
|
mock_logger.info.assert_called_with("Code execution successful")
|
|
assert "Successfully executed:" in result
|
|
assert code in result
|
|
assert expected_output in result
|
|
|
|
@patch.dict(os.environ, {"ENABLE_PYTHON_REPL": "true"})
|
|
@patch("src.tools.python_repl.repl")
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_invalid_input_type(self, mock_logger, mock_repl):
|
|
# Arrange
|
|
invalid_code = 123
|
|
|
|
# Act & Assert - expect ValidationError when passing invalid input
|
|
with pytest.raises(Exception): # Could be ValidationError or similar
|
|
python_repl_tool(invalid_code)
|
|
|
|
mock_repl.run.assert_not_called()
|
|
|
|
@patch.dict(os.environ, {"ENABLE_PYTHON_REPL": "true"})
|
|
@patch("src.tools.python_repl.repl")
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_code_execution_with_error_in_result(self, mock_logger, mock_repl):
|
|
# Arrange
|
|
code = "invalid_function()"
|
|
error_result = "NameError: name 'invalid_function' is not defined"
|
|
mock_repl.run.return_value = error_result
|
|
|
|
# Act
|
|
result = python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_repl.run.assert_called_once_with(code)
|
|
mock_logger.error.assert_called_with(error_result)
|
|
assert "Error executing code:" in result
|
|
assert code in result
|
|
assert error_result in result
|
|
|
|
@patch.dict(os.environ, {"ENABLE_PYTHON_REPL": "true"})
|
|
@patch("src.tools.python_repl.repl")
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_code_execution_with_exception_in_result(self, mock_logger, mock_repl):
|
|
# Arrange
|
|
code = "1/0"
|
|
exception_result = "ZeroDivisionError: division by zero"
|
|
mock_repl.run.return_value = exception_result
|
|
|
|
# Act
|
|
result = python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_repl.run.assert_called_once_with(code)
|
|
mock_logger.error.assert_called_with(exception_result)
|
|
assert "Error executing code:" in result
|
|
assert code in result
|
|
assert exception_result in result
|
|
|
|
@patch.dict(os.environ, {"ENABLE_PYTHON_REPL": "true"})
|
|
@patch("src.tools.python_repl.repl")
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_code_execution_raises_exception(self, mock_logger, mock_repl):
|
|
# Arrange
|
|
code = "print('test')"
|
|
exception = RuntimeError("REPL failed")
|
|
mock_repl.run.side_effect = exception
|
|
|
|
# Act
|
|
result = python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_repl.run.assert_called_once_with(code)
|
|
mock_logger.error.assert_called_with(repr(exception))
|
|
assert "Error executing code:" in result
|
|
assert code in result
|
|
assert repr(exception) in result
|
|
|
|
@patch.dict(os.environ, {"ENABLE_PYTHON_REPL": "true"})
|
|
@patch("src.tools.python_repl.repl")
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_successful_execution_with_calculation(self, mock_logger, mock_repl):
|
|
# Arrange
|
|
code = "result = 2 + 3\nprint(result)"
|
|
expected_output = "5\n"
|
|
mock_repl.run.return_value = expected_output
|
|
|
|
# Act
|
|
result = python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_repl.run.assert_called_once_with(code)
|
|
mock_logger.info.assert_any_call("Executing Python code")
|
|
mock_logger.info.assert_any_call("Code execution successful")
|
|
assert "Successfully executed:" in result
|
|
assert code in result
|
|
assert expected_output in result
|
|
|
|
@patch.dict(os.environ, {"ENABLE_PYTHON_REPL": "true"})
|
|
@patch("src.tools.python_repl.repl")
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_empty_string_code(self, mock_logger, mock_repl):
|
|
# Arrange
|
|
code = ""
|
|
mock_repl.run.return_value = ""
|
|
|
|
# Act
|
|
result = python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_repl.run.assert_called_once_with(code)
|
|
mock_logger.info.assert_called_with("Code execution successful")
|
|
assert "Successfully executed:" in result
|
|
|
|
@patch.dict(os.environ, {"ENABLE_PYTHON_REPL": "true"})
|
|
@patch("src.tools.python_repl.repl")
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_logging_calls(self, mock_logger, mock_repl):
|
|
# Arrange
|
|
code = "x = 1"
|
|
mock_repl.run.return_value = ""
|
|
|
|
# Act
|
|
python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_logger.info.assert_any_call("Executing Python code")
|
|
mock_logger.info.assert_any_call("Code execution successful")
|
|
|
|
# New tests for configuration behavior
|
|
@patch.dict(os.environ, {"ENABLE_PYTHON_REPL": "false"})
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_tool_disabled(self, mock_logger):
|
|
# Arrange
|
|
code = "print('test')"
|
|
|
|
# Act
|
|
result = python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_logger.warning.assert_called_with(
|
|
"Python REPL tool is disabled. Please enable it in environment configuration."
|
|
)
|
|
assert "Tool disabled:" in result
|
|
assert "Python REPL tool is disabled" in result
|
|
|
|
@patch.dict(os.environ, {}, clear=True)
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_tool_disabled_by_default(self, mock_logger):
|
|
# Arrange - remove any existing ENABLE_PYTHON_REPL variable
|
|
if "ENABLE_PYTHON_REPL" in os.environ:
|
|
del os.environ["ENABLE_PYTHON_REPL"]
|
|
code = "print('test')"
|
|
|
|
# Act
|
|
result = python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_logger.warning.assert_called_with(
|
|
"Python REPL tool is disabled. Please enable it in environment configuration."
|
|
)
|
|
assert "Tool disabled:" in result
|
|
|
|
@pytest.mark.parametrize("env_value", ["true", "True", "TRUE", "1", "yes", "on"])
|
|
@patch("src.tools.python_repl.repl")
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_tool_enabled_with_various_truthy_values(
|
|
self, mock_logger, mock_repl, env_value
|
|
):
|
|
# Arrange
|
|
with patch.dict(os.environ, {"ENABLE_PYTHON_REPL": env_value}):
|
|
code = "print('enabled')"
|
|
expected_output = "enabled\n"
|
|
mock_repl.run.return_value = expected_output
|
|
|
|
# Act
|
|
result = python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_repl.run.assert_called_once_with(code)
|
|
assert "Successfully executed:" in result
|
|
|
|
@pytest.mark.parametrize(
|
|
"env_value", ["false", "False", "FALSE", "0", "no", "off", ""]
|
|
)
|
|
@patch("src.tools.python_repl.logger")
|
|
def test_tool_disabled_with_various_falsy_values(self, mock_logger, env_value):
|
|
# Arrange
|
|
with patch.dict(os.environ, {"ENABLE_PYTHON_REPL": env_value}):
|
|
code = "print('disabled')"
|
|
|
|
# Act
|
|
result = python_repl_tool(code)
|
|
|
|
# Assert
|
|
mock_logger.warning.assert_called_with(
|
|
"Python REPL tool is disabled. Please enable it in environment configuration."
|
|
)
|
|
assert "Tool disabled:" in result
|