🧪 Add global test configuration and common fixtures for improved test, refactor test files to use pytest and improve mock handling

- Converted unittest test cases to pytest format across multiple test files.
- Simplified mock setups using pytest-mock for better readability and maintainability.
- Removed unnecessary imports and legacy code to streamline test execution.
- Enhanced test coverage for utility functions in various modules.

Author: Zhi-a

test/backend/agents/test_create_agent_info.py

@@ -5,9 +5,9 @@
 from pathlib import Path
 from unittest.mock import AsyncMock, MagicMock, patch, Mock, PropertyMock
 
-from test.common.env_test_utils import bootstrap_env
+from test.common.test_mocks import bootstrap_test_env
 
-env_state = bootstrap_env()
+env_state = bootstrap_test_env()
 consts_const = env_state["mock_const"]
 TEST_ROOT = Path(__file__).resolve().parents[2]
 PROJECT_ROOT = TEST_ROOT.parent

test/backend/app/test_image_app.py

@@ -8,9 +8,9 @@
 if str(TEST_ROOT) not in sys.path:
     sys.path.append(str(TEST_ROOT))
 
-from test.common.env_test_utils import bootstrap_env
+from test.common.test_mocks import bootstrap_test_env
 
-helpers_env = bootstrap_env()
+helpers_env = bootstrap_test_env()
 
 
 helpers_env["mock_const"].DATA_PROCESS_SERVICE = "http://mock-data-process-service"

test/backend/app/test_knowledge_summary_app.py

@@ -12,12 +12,7 @@
     if path not in sys.path:
         sys.path.insert(0, path)
 
-# Patch environment variables before any imports that might use them
-os.environ.setdefault('MINIO_ENDPOINT', 'http://localhost:9000')
-os.environ.setdefault('MINIO_ACCESS_KEY', 'minioadmin')
-os.environ.setdefault('MINIO_SECRET_KEY', 'minioadmin')
-os.environ.setdefault('MINIO_REGION', 'us-east-1')
-os.environ.setdefault('MINIO_DEFAULT_BUCKET', 'test-bucket')
+# Environment variables are now configured in conftest.py
 
 # Mock external dependencies
 sys.modules['boto3'] = MagicMock()

test/backend/app/test_mock_user_management_app.py

@@ -7,12 +7,7 @@
 current_dir = os.path.dirname(os.path.abspath(__file__))
 sys.path.insert(0, os.path.join(current_dir, "../../../backend"))
 
-# Patch environment variables before any imports that might use them
-os.environ.setdefault('MINIO_ENDPOINT', 'http://localhost:9000')
-os.environ.setdefault('MINIO_ACCESS_KEY', 'minioadmin')
-os.environ.setdefault('MINIO_SECRET_KEY', 'minioadmin')
-os.environ.setdefault('MINIO_REGION', 'us-east-1')
-os.environ.setdefault('MINIO_DEFAULT_BUCKET', 'test-bucket')
+# Environment variables are now configured in conftest.py
 
 boto3_mock = MagicMock()
 minio_client_mock = MagicMock()

test/backend/app/test_model_managment_app.py

@@ -17,11 +17,7 @@
     sys.path.insert(0, BACKEND_ROOT)
 
 # Patch environment variables before any imports that might use them
-os.environ.setdefault('MINIO_ENDPOINT', 'http://localhost:9000')
-os.environ.setdefault('MINIO_ACCESS_KEY', 'minioadmin')
-os.environ.setdefault('MINIO_SECRET_KEY', 'minioadmin')
-os.environ.setdefault('MINIO_REGION', 'us-east-1')
-os.environ.setdefault('MINIO_DEFAULT_BUCKET', 'test-bucket')
+# Environment variables are now configured in conftest.py
 
 # Patch storage factory and MinIO config validation to avoid errors during initialization
 # These patches must be started before any imports that use MinioClient

test/backend/app/test_vectordatabase_app.py

@@ -18,12 +18,7 @@
 backend_dir = os.path.abspath(os.path.join(current_dir, "../../../backend"))
 sys.path.insert(0, backend_dir)
 
-# Patch environment variables before any imports that might use them
-os.environ.setdefault('MINIO_ENDPOINT', 'http://localhost:9000')
-os.environ.setdefault('MINIO_ACCESS_KEY', 'minioadmin')
-os.environ.setdefault('MINIO_SECRET_KEY', 'minioadmin')
-os.environ.setdefault('MINIO_REGION', 'us-east-1')
-os.environ.setdefault('MINIO_DEFAULT_BUCKET', 'test-bucket')
+# Environment variables are now configured in conftest.py
 
 boto3_mock = MagicMock()
 minio_client_mock = MagicMock()

test/backend/database/test_attachment_db.py

@@ -14,21 +14,10 @@
 sys.path.insert(0, os.path.abspath(os.path.join(
     os.path.dirname(__file__), '..', '..', '..')))
 
-# Mock environment variables before imports
-os.environ.setdefault('MINIO_ENDPOINT', 'http://localhost:9000')
-os.environ.setdefault('MINIO_ACCESS_KEY', 'minioadmin')
-os.environ.setdefault('MINIO_SECRET_KEY', 'minioadmin')
-os.environ.setdefault('MINIO_REGION', 'us-east-1')
-os.environ.setdefault('MINIO_DEFAULT_BUCKET', 'test-bucket')
-
 # Mock consts module
 consts_mock = MagicMock()
 consts_mock.const = MagicMock()
-consts_mock.const.MINIO_ENDPOINT = os.environ.get('MINIO_ENDPOINT', 'http://localhost:9000')
-consts_mock.const.MINIO_ACCESS_KEY = os.environ.get('MINIO_ACCESS_KEY', 'minioadmin')
-consts_mock.const.MINIO_SECRET_KEY = os.environ.get('MINIO_SECRET_KEY', 'minioadmin')
-consts_mock.const.MINIO_REGION = os.environ.get('MINIO_REGION', 'us-east-1')
-consts_mock.const.MINIO_DEFAULT_BUCKET = os.environ.get('MINIO_DEFAULT_BUCKET', 'test-bucket')
+# Environment variables are now configured in conftest.py
 
 sys.modules['consts'] = consts_mock
 sys.modules['consts.const'] = consts_mock.const