test(wave191): 补全direct_construction测试用例，增加完整构造场景

- 拆分test_direct_construction为minimal和full两个用例
- minimal用例验证默认值行为
- full用例验证所有字段均可独立赋值
- 新增EditingMode导入

tests/unit/domain/test_edit_template.py

@@ -0,0 +1,281 @@
+"""edit_template 剪辑模板实体单测."""
+
+from datetime import datetime, timezone
+
+import pytest
+from domain.edit_template import EditTemplate, EditTemplateStatus
+from domain.editing_mode import EditingMode
+
+# ── EditTemplateStatus 枚举 ──────────────────────────────────────────────────
+
+
+class TestEditTemplateStatus:
+    """EditTemplateStatus 枚举"""
+
+    def test_enum_values(self):
+        assert EditTemplateStatus.ACTIVE.value == "active"
+        assert EditTemplateStatus.INACTIVE.value == "inactive"
+
+    def test_is_str_enum(self):
+        assert isinstance(EditTemplateStatus.ACTIVE, str)
+        assert EditTemplateStatus.ACTIVE == "active"
+
+    def test_from_string(self):
+        assert EditTemplateStatus("active") == EditTemplateStatus.ACTIVE
+        assert EditTemplateStatus("inactive") == EditTemplateStatus.INACTIVE
+
+    def test_invalid_raises(self):
+        with pytest.raises(ValueError):
+            EditTemplateStatus("deleted")
+
+
+# ── EditTemplate.create 工厂方法 ────────────────────────────────────────────
+
+
+class TestEditTemplateCreate:
+    """EditTemplate.create 工厂方法"""
+
+    def test_minimal_create(self):
+        t = EditTemplate.create("测试模板")
+        assert t.id is not None
+        assert len(t.id) == 32  # uuid4 hex
+        assert t.name == "测试模板"
+        assert t.description == ""
+        assert t.template_type == "default"
+        assert t.editing_mode == "one_take"
+        assert t.config == {}
+        assert t.preview_url == ""
+        assert t.sort_weight == 0
+        assert t.status == EditTemplateStatus.ACTIVE
+        assert t.version == 1
+
+    def test_unique_ids(self):
+        t1 = EditTemplate.create("模板A")
+        t2 = EditTemplate.create("模板B")
+        assert t1.id != t2.id
+
+    def test_custom_fields(self):
+        t = EditTemplate.create(
+            "自定义模板",
+            description="这是一个自定义模板",
+            template_type="story",
+            editing_mode="one_take",
+            config={"key": "value"},
+            preview_url="https://example.com/preview.mp4",
+            sort_weight=100,
+            status=EditTemplateStatus.INACTIVE,
+            version=2,
+        )
+        assert t.name == "自定义模板"
+        assert t.description == "这是一个自定义模板"
+        assert t.template_type == "story"
+        assert t.editing_mode == "one_take"
+        assert t.config == {"key": "value"}
+        assert t.preview_url == "https://example.com/preview.mp4"
+        assert t.sort_weight == 100
+        assert t.status == EditTemplateStatus.INACTIVE
+        assert t.version == 2
+
+    def test_name_stripped(self):
+        t = EditTemplate.create("  带空格的模板  ")
+        assert t.name == "带空格的模板"
+
+    def test_empty_name_raises(self):
+        with pytest.raises(ValueError, match="名称"):
+            EditTemplate.create("")
+
+    def test_whitespace_only_name_raises(self):
+        with pytest.raises(ValueError):
+            EditTemplate.create("   ")
+
+    def test_invalid_editing_mode_raises(self):
+        with pytest.raises(ValueError, match="editing_mode"):
+            EditTemplate.create("测试", editing_mode="invalid_mode")
+
+    def test_empty_editing_mode_falls_back_to_default(self):
+        t = EditTemplate.create("测试", editing_mode="")
+        assert t.editing_mode == "one_take"
+
+    def test_whitespace_editing_mode_falls_back(self):
+        t = EditTemplate.create("测试", editing_mode="   ")
+        assert t.editing_mode == "one_take"
+
+    def test_editing_mode_stripped(self):
+        t = EditTemplate.create("测试", editing_mode=" one_take ")
+        assert t.editing_mode == "one_take"
+
+    def test_description_stripped(self):
+        t = EditTemplate.create("测试", description="  描述  ")
+        assert t.description == "描述"
+
+    def test_template_type_stripped(self):
+        t = EditTemplate.create("测试", template_type="  vlog  ")
+        assert t.template_type == "vlog"
+
+    def test_empty_template_type_falls_back(self):
+        t = EditTemplate.create("测试", template_type="")
+        assert t.template_type == "default"
+
+    def test_none_config_becomes_empty_dict(self):
+        t = EditTemplate.create("测试", config=None)
+        assert t.config == {}
+        assert isinstance(t.config, dict)
+
+    def test_preview_url_stripped(self):
+        t = EditTemplate.create("测试", preview_url="  https://x.com/a.mp4  ")
+        assert t.preview_url == "https://x.com/a.mp4"
+
+    def test_timestamps_are_utc(self):
+        t = EditTemplate.create("测试")
+        assert t.created_at.tzinfo is not None
+        assert t.updated_at.tzinfo is not None
+
+    def test_created_at_equals_updated_at_on_create(self):
+        t = EditTemplate.create("测试")
+        # 创建时两个时间应该非常接近
+        diff = abs((t.updated_at - t.created_at).total_seconds())
+        assert diff < 1.0
+
+
+# ── 状态操作 ─────────────────────────────────────────────────────────────────
+
+
+class TestEditTemplateStatusOperations:
+    """EditTemplate 状态操作"""
+
+    def test_activate_sets_active(self):
+        t = EditTemplate.create("测试", status=EditTemplateStatus.INACTIVE)
+        t.activate()
+        assert t.status == EditTemplateStatus.ACTIVE
+        assert t.is_active is True
+
+    def test_deactivate_sets_inactive(self):
+        t = EditTemplate.create("测试")
+        t.deactivate()
+        assert t.status == EditTemplateStatus.INACTIVE
+        assert t.is_active is False
+
+    def test_is_active_true(self):
+        t = EditTemplate.create("测试")
+        assert t.is_active is True
+
+    def test_is_active_false(self):
+        t = EditTemplate.create("测试", status=EditTemplateStatus.INACTIVE)
+        assert t.is_active is False
+
+    def test_activate_updates_updated_at(self):
+        t = EditTemplate.create("测试", status=EditTemplateStatus.INACTIVE)
+        old_updated = t.updated_at
+        t.activate()
+        assert t.updated_at >= old_updated
+
+    def test_deactivate_updates_updated_at(self):
+        t = EditTemplate.create("测试")
+        old_updated = t.updated_at
+        t.deactivate()
+        assert t.updated_at >= old_updated
+
+
+# ── 版本操作 ─────────────────────────────────────────────────────────────────
+
+
+class TestEditTemplateVersion:
+    """EditTemplate 版本操作"""
+
+    def test_bump_version_increments(self):
+        t = EditTemplate.create("测试")
+        assert t.version == 1
+        t.bump_version()
+        assert t.version == 2
+
+    def test_bump_version_multiple(self):
+        t = EditTemplate.create("测试", version=5)
+        t.bump_version()
+        t.bump_version()
+        t.bump_version()
+        assert t.version == 8
+
+    def test_bump_version_updates_updated_at(self):
+        t = EditTemplate.create("测试")
+        old_updated = t.updated_at
+        t.bump_version()
+        assert t.updated_at >= old_updated
+
+
+# ── dataclass 基础特性 ───────────────────────────────────────────────────────
+
+
+class TestEditTemplateBasics:
+    """EditTemplate 基础特性"""
+
+    def test_slots_no_extra_attrs(self):
+        t = EditTemplate.create("测试")
+        with pytest.raises(AttributeError):
+            t.nonexistent_field = "value"
+
+    def test_direct_construction_minimal(self):
+        # 最小构造：仅必填字段 + 状态，其余走默认值
+        t = EditTemplate(
+            id="custom_id",
+            name="直接构造",
+            status=EditTemplateStatus.ACTIVE,
+        )
+        assert t.id == "custom_id"
+        assert t.name == "直接构造"
+        assert t.status == EditTemplateStatus.ACTIVE
+        # 默认值检查
+        assert t.description == ""
+        assert t.config == {}
+        assert t.version == 1
+        assert t.editing_mode == EditingMode.ONE_TAKE.value
+        assert isinstance(t.created_at, datetime)
+        assert isinstance(t.updated_at, datetime)
+
+    def test_direct_construction_full(self):
+        # 完整构造：所有字段都传
+        now = datetime(2025, 1, 1, tzinfo=timezone.utc)
+        t = EditTemplate(
+            id="full_id",
+            name="完整构造",
+            description="测试描述",
+            template_type="custom",
+            editing_mode=EditingMode.PIP.value,
+            config={"key": "value"},
+            preview_url="https://example.com/preview.jpg",
+            sort_weight=100,
+            status=EditTemplateStatus.INACTIVE,
+            version=3,
+            created_at=now,
+            updated_at=now,
+        )
+        assert t.id == "full_id"
+        assert t.name == "完整构造"
+        assert t.description == "测试描述"
+        assert t.template_type == "custom"
+        assert t.editing_mode == EditingMode.PIP.value
+        assert t.config == {"key": "value"}
+        assert t.preview_url == "https://example.com/preview.jpg"
+        assert t.sort_weight == 100
+        assert t.status == EditTemplateStatus.INACTIVE
+        assert t.version == 3
+        assert t.created_at == now
+        assert t.updated_at == now
+
+    def test_config_is_independent(self):
+        # 不同实例的 config 应该是独立的 dict
+        t1 = EditTemplate.create("模板1")
+        t2 = EditTemplate.create("模板2")
+        t1.config["key"] = "value"
+        assert "key" not in t2.config
+
+    def test_equality(self):
+        # 两个不同实例即使内容相同也不等（id不同）
+        t1 = EditTemplate.create("同名模板")
+        t2 = EditTemplate.create("同名模板")
+        assert t1 != t2
+
+    def test_same_id_equal(self):
+        now = datetime.now(timezone.utc)
+        t1 = EditTemplate(id="same", name="同名", created_at=now, updated_at=now)
+        t2 = EditTemplate(id="same", name="同名", created_at=now, updated_at=now)
+        assert t1 == t2

tests/unit/domain/test_verification_code.py

@@ -1,280 +0,0 @@
-"""VerificationCode 单元测试."""
-
-from __future__ import annotations
-
-from datetime import datetime, timedelta, timezone
-from unittest.mock import patch
-
-import pytest
-from domain.verification_code import VerificationCode
-
-
-class TestVerificationCodeCreate:
-    """create() 工厂方法测试."""
-
-    def test_create_basic(self):
-        vc = VerificationCode.create("test@example.com", "email_login")
-        assert vc.id is not None
-        assert len(vc.id) == 32
-        assert vc.recipient == "test@example.com"
-        assert vc.code_type == "email_login"
-        assert len(vc.code) == 6
-        assert vc.code.isdigit()
-        assert vc.used_at is None
-        assert vc.attempts == 0
-        assert vc.created_at is not None
-        assert vc.expires_at > vc.created_at
-
-    def test_create_recipient_stripped(self):
-        vc = VerificationCode.create("  test@example.com  ", "email_login")
-        assert vc.recipient == "test@example.com"
-
-    def test_create_custom_code(self):
-        vc = VerificationCode.create("test@example.com", "email_login", custom_code="123456")
-        assert vc.code == "123456"
-
-    def test_create_custom_ttl(self):
-        fixed_now = datetime(2026, 1, 1, 12, 0, 0, tzinfo=timezone.utc)
-        with patch("domain.verification_code.datetime") as mock_dt:
-            mock_dt.now.return_value = fixed_now
-            mock_dt.side_effect = lambda *a, **kw: datetime(*a, **kw)
-            vc = VerificationCode.create("test@example.com", "email_login", ttl_seconds=60)
-        assert vc.expires_at == fixed_now + timedelta(seconds=60)
-
-    def test_create_default_ttl_300(self):
-        fixed_now = datetime(2026, 1, 1, 12, 0, 0, tzinfo=timezone.utc)
-        with patch("domain.verification_code.datetime") as mock_dt:
-            mock_dt.now.return_value = fixed_now
-            mock_dt.side_effect = lambda *a, **kw: datetime(*a, **kw)
-            vc = VerificationCode.create("test@example.com", "email_login")
-        assert vc.expires_at == fixed_now + timedelta(seconds=300)
-
-    def test_create_unique_ids(self):
-        vc1 = VerificationCode.create("a@b.com", "email_login")
-        vc2 = VerificationCode.create("a@b.com", "email_login")
-        assert vc1.id != vc2.id
-
-    def test_create_unique_codes(self):
-        codes = set()
-        for _ in range(20):
-            vc = VerificationCode.create("a@b.com", "email_login")
-            codes.add(vc.code)
-        # 20个随机6位码几乎肯定不都一样
-        assert len(codes) > 1
-
-    def test_create_phone_recipient(self):
-        vc = VerificationCode.create("13800138000", "phone_login")
-        assert vc.recipient == "13800138000"
-        assert vc.code_type == "phone_login"
-
-    def test_create_all_code_types(self):
-        for ct in ["email_bind", "phone_bind", "email_login", "phone_login", "reset_password"]:
-            vc = VerificationCode.create("test@example.com", ct)
-            assert vc.code_type == ct
-
-
-class TestVerificationCodeIsExpired:
-    """is_expired 属性测试."""
-
-    def test_not_expired_future(self):
-        future = datetime.now(timezone.utc) + timedelta(hours=1)
-        vc = VerificationCode(
-            id="1",
-            recipient="a@b.com",
-            code="123456",
-            code_type="email_login",
-            expires_at=future,
-        )
-        assert vc.is_expired is False
-
-    def test_expired_past(self):
-        past = datetime.now(timezone.utc) - timedelta(hours=1)
-        vc = VerificationCode(
-            id="1",
-            recipient="a@b.com",
-            code="123456",
-            code_type="email_login",
-            expires_at=past,
-        )
-        assert vc.is_expired is True
-
-    def test_expired_boundary_exact(self):
-        # 用mock固定时间，expires_at等于当前时间不算过期
-        fixed_now = datetime(2026, 1, 1, 12, 0, 0, tzinfo=timezone.utc)
-        with patch("domain.verification_code.datetime") as mock_dt:
-            mock_dt.now.return_value = fixed_now
-            mock_dt.side_effect = lambda *a, **kw: datetime(*a, **kw)
-            vc = VerificationCode(
-                id="1",
-                recipient="a@b.com",
-                code="123456",
-                code_type="email_login",
-                expires_at=fixed_now,
-            )
-            assert vc.is_expired is False
-
-
-class TestVerificationCodeIsUsed:
-    """is_used 属性测试."""
-
-    def test_not_used_default(self):
-        vc = VerificationCode.create("a@b.com", "email_login")
-        assert vc.is_used is False
-
-    def test_is_used_after_mark(self):
-        vc = VerificationCode.create("a@b.com", "email_login")
-        vc.mark_used()
-        assert vc.is_used is True
-
-
-class TestVerificationCodeIsValid:
-    """is_valid 属性测试."""
-
-    def test_valid_fresh(self):
-        future = datetime.now(timezone.utc) + timedelta(hours=1)
-        vc = VerificationCode(
-            id="1",
-            recipient="a@b.com",
-            code="123456",
-            code_type="email_login",
-            expires_at=future,
-        )
-        assert vc.is_valid is True
-
-    def test_invalid_expired(self):
-        past = datetime.now(timezone.utc) - timedelta(hours=1)
-        vc = VerificationCode(
-            id="1",
-            recipient="a@b.com",
-            code="123456",
-            code_type="email_login",
-            expires_at=past,
-        )
-        assert vc.is_valid is False
-
-    def test_invalid_used(self):
-        future = datetime.now(timezone.utc) + timedelta(hours=1)
-        vc = VerificationCode(
-            id="1",
-            recipient="a@b.com",
-            code="123456",
-            code_type="email_login",
-            expires_at=future,
-        )
-        vc.mark_used()
-        assert vc.is_valid is False
-
-    def test_invalid_expired_and_used(self):
-        past = datetime.now(timezone.utc) - timedelta(hours=1)
-        vc = VerificationCode(
-            id="1",
-            recipient="a@b.com",
-            code="123456",
-            code_type="email_login",
-            expires_at=past,
-        )
-        vc.mark_used()
-        assert vc.is_valid is False
-
-
-class TestVerificationCodeMarkUsed:
-    """mark_used 方法测试."""
-
-    def test_mark_used_sets_timestamp(self):
-        vc = VerificationCode.create("a@b.com", "email_login")
-        assert vc.used_at is None
-        before = datetime.now(timezone.utc)
-        vc.mark_used()
-        after = datetime.now(timezone.utc)
-        assert vc.used_at is not None
-        assert before <= vc.used_at <= after
-
-    def test_mark_used_twice_overwrites(self):
-        vc = VerificationCode.create("a@b.com", "email_login")
-        vc.mark_used()
-        first = vc.used_at
-        # 时间足够短，一般不会不同，但确保可以重复调用
-        vc.mark_used()
-        assert vc.used_at is not None
-
-
-class TestVerificationCodeIncrementAttempts:
-    """increment_attempts 方法测试."""
-
-    def test_default_zero(self):
-        vc = VerificationCode.create("a@b.com", "email_login")
-        assert vc.attempts == 0
-
-    def test_increment_once(self):
-        vc = VerificationCode.create("a@b.com", "email_login")
-        vc.increment_attempts()
-        assert vc.attempts == 1
-
-    def test_increment_multiple(self):
-        vc = VerificationCode.create("a@b.com", "email_login")
-        for _i in range(5):
-            vc.increment_attempts()
-        assert vc.attempts == 5
-
-
-class TestVerificationCodeBasics:
-    """基础构造和 slots 测试."""
-
-    def test_direct_construction(self):
-        now = datetime.now(timezone.utc)
-        vc = VerificationCode(
-            id="abc123",
-            recipient="test@test.com",
-            code="000000",
-            code_type="email_bind",
-            expires_at=now + timedelta(minutes=5),
-            used_at=None,
-            attempts=0,
-            created_at=now,
-        )
-        assert vc.id == "abc123"
-        assert vc.recipient == "test@test.com"
-        assert vc.code == "000000"
-
-    def test_slots_no_extra_attrs(self):
-        vc = VerificationCode.create("a@b.com", "email_login")
-        with pytest.raises((AttributeError, TypeError)):
-            vc.new_field = "value"  # type: ignore[attr-defined]
-
-    def test_equality_same_id(self):
-        now = datetime(2026, 1, 1, 12, 0, 0, tzinfo=timezone.utc)
-        vc1 = VerificationCode(
-            id="same",
-            recipient="a@b.com",
-            code="111",
-            code_type="email_login",
-            expires_at=now,
-            created_at=now,
-        )
-        vc2 = VerificationCode(
-            id="same",
-            recipient="a@b.com",
-            code="111",
-            code_type="email_login",
-            expires_at=now,
-            created_at=now,
-        )
-        assert vc1 == vc2
-
-    def test_equality_different_id(self):
-        now = datetime.now(timezone.utc)
-        vc1 = VerificationCode(
-            id="id1",
-            recipient="a@b.com",
-            code="111",
-            code_type="email_login",
-            expires_at=now,
-        )
-        vc2 = VerificationCode(
-            id="id2",
-            recipient="a@b.com",
-            code="111",
-            code_type="email_login",
-            expires_at=now,
-        )
-        assert vc1 != vc2