6ede33ccdb
Build and Push Docker Images / create_manifest (web, surfsense-web, , cpu) (push) Has been cancelled
Build and Push Docker Images / finalize_release (push) Has been cancelled
Obsidian Plugin Lint / lint (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_web, cpu, ./surfsense_web/Dockerfile, web, surfsense-web, ubuntu-24.04-arm, linux/arm64, arm64, , runner, false, cpu) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_web, cpu, ./surfsense_web/Dockerfile, web, surfsense-web, ubuntu-latest, linux/amd64, amd64, , runner, false, cpu) (push) Has been cancelled
Build and Push Docker Images / compute_version (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cpu, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-24.04-arm, linux/arm64, arm64, , production, false, cpu) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cpu, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-latest, linux/amd64, amd64, , production, false, cpu) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cu126, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-24.04-arm, linux/arm64, arm64, -cuda126, production, true, cuda126) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cu126, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-latest, linux/amd64, amd64, -cuda126, production, true, cuda126) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cu128, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-24.04-arm, linux/arm64, arm64, -cuda, production, true, cuda) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cu128, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-latest, linux/amd64, amd64, -cuda, production, true, cuda) (push) Has been cancelled
Build and Push Docker Images / verify_digests (push) Has been cancelled
Build and Push Docker Images / create_manifest (backend, surfsense-backend, , cpu) (push) Has been cancelled
Build and Push Docker Images / create_manifest (backend, surfsense-backend, -cuda, cuda) (push) Has been cancelled
Build and Push Docker Images / create_manifest (backend, surfsense-backend, -cuda126, cuda126) (push) Has been cancelled
88 lines
2.4 KiB
Python
88 lines
2.4 KiB
Python
"""Unit tests for ensure_folder_hierarchy_with_depth_validation."""
|
|
|
|
from unittest.mock import AsyncMock, MagicMock, patch
|
|
|
|
import pytest
|
|
|
|
pytestmark = pytest.mark.unit
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_creates_missing_folders_in_chain():
|
|
"""Should create all folders when none exist."""
|
|
from app.services.folder_service import (
|
|
ensure_folder_hierarchy_with_depth_validation,
|
|
)
|
|
|
|
session = AsyncMock()
|
|
# All lookups return None (no existing folders)
|
|
mock_result = MagicMock()
|
|
mock_result.scalar_one_or_none.return_value = None
|
|
session.execute.return_value = mock_result
|
|
|
|
folder_instances = []
|
|
|
|
def track_add(obj):
|
|
folder_instances.append(obj)
|
|
|
|
session.add = track_add
|
|
|
|
with (
|
|
patch(
|
|
"app.services.folder_service.validate_folder_depth", new_callable=AsyncMock
|
|
),
|
|
patch(
|
|
"app.services.folder_service.generate_folder_position",
|
|
new_callable=AsyncMock,
|
|
return_value="a0",
|
|
),
|
|
):
|
|
# Mock flush to assign IDs
|
|
call_count = 0
|
|
|
|
async def mock_flush():
|
|
nonlocal call_count
|
|
call_count += 1
|
|
if folder_instances:
|
|
folder_instances[-1].id = call_count
|
|
|
|
session.flush = mock_flush
|
|
|
|
segments = [
|
|
{"name": "Slack", "metadata": {"source": "slack"}},
|
|
{"name": "2025-03-15", "metadata": {"source": "slack"}},
|
|
]
|
|
|
|
result = await ensure_folder_hierarchy_with_depth_validation(
|
|
session, 1, segments
|
|
)
|
|
|
|
assert len(folder_instances) == 2
|
|
assert folder_instances[0].name == "Slack"
|
|
assert folder_instances[1].name == "2025-03-15"
|
|
assert result is folder_instances[-1]
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_reuses_existing_folder():
|
|
"""When a folder already exists, it should be reused, not created."""
|
|
from app.services.folder_service import (
|
|
ensure_folder_hierarchy_with_depth_validation,
|
|
)
|
|
|
|
session = AsyncMock()
|
|
|
|
existing_folder = MagicMock()
|
|
existing_folder.id = 42
|
|
|
|
mock_result = MagicMock()
|
|
mock_result.scalar_one_or_none.return_value = existing_folder
|
|
session.execute.return_value = mock_result
|
|
|
|
segments = [{"name": "Existing", "metadata": None}]
|
|
|
|
result = await ensure_folder_hierarchy_with_depth_validation(session, 1, segments)
|
|
|
|
assert result is existing_folder
|
|
session.add.assert_not_called()
|