test(boards): wire video record storage into multiuser test fixtures

After the board cover/count fix started reading from `video_records` and
`board_video_records`, the multiuser test fixtures that still set both
to `None` started erroring out — the boards router's catch-all turned the
AttributeError into a 404, cascading through every test that PATCHes or
GETs a board (auth, workflows, data-isolation suites). Swap the `None`
placeholders for real SqliteVideoRecordStorage / SqliteBoardVideoRecordStorage
instances (paralleling the existing image storage setup), and pin sane
defaults on the MagicMocks in `test_videos_multiuser.py` so the
get_dto cover/count lookups don't trip Pydantic validation.

Author: lstein

tests/app/routers/test_multiuser_authorization.py

@@ -68,12 +68,14 @@ def client():
 def mock_services() -> InvocationServices:
     from invokeai.app.services.board_image_records.board_image_records_sqlite import SqliteBoardImageRecordStorage
     from invokeai.app.services.board_records.board_records_sqlite import SqliteBoardRecordStorage
+    from invokeai.app.services.board_video_records.board_video_records_sqlite import SqliteBoardVideoRecordStorage
     from invokeai.app.services.boards.boards_default import BoardService
     from invokeai.app.services.bulk_download.bulk_download_default import BulkDownloadService
     from invokeai.app.services.client_state_persistence.client_state_persistence_sqlite import (
         ClientStatePersistenceSqlite,
     )
     from invokeai.app.services.image_records.image_records_sqlite import SqliteImageRecordStorage
+    from invokeai.app.services.video_records.video_records_sqlite import SqliteVideoRecordStorage
     from invokeai.app.services.images.images_default import ImageService
     from invokeai.app.services.invocation_cache.invocation_cache_memory import MemoryInvocationCache
     from invokeai.app.services.invocation_stats.invocation_stats_default import InvocationStatsService
@@ -118,8 +120,8 @@ def mock_services() -> InvocationServices:
         external_generation=None,  # type: ignore
         videos=None,  # type: ignore
         video_files=None,  # type: ignore
-        video_records=None,  # type: ignore
-        board_video_records=None,  # type: ignore
+        video_records=SqliteVideoRecordStorage(db=db),
+        board_video_records=SqliteBoardVideoRecordStorage(db=db),
         gallery=None,  # type: ignore
     )
 

tests/app/routers/test_videos_multiuser.py

@@ -83,7 +83,12 @@ def enable_multiuser_for_videos(monkeypatch: Any, mock_invoker: Invoker):
     # board_video_records is touched by remove_video_from_board; not exercised by the
     # list tests but stub it defensively so unrelated routes don't blow up.
     mock_invoker.services.board_video_records = MagicMock()
+    # The board service computes video_count + cover_video_name on every get_dto/update;
+    # an unconfigured MagicMock returns nested MagicMocks that fail Pydantic validation and
+    # the boards route swallows the exception as a 404. Pin sane defaults.
+    mock_invoker.services.board_video_records.get_video_count_for_board.return_value = 0
     mock_invoker.services.video_records = MagicMock()
+    mock_invoker.services.video_records.get_most_recent_video_for_board.return_value = None
     mock_invoker.services.board_images = MagicMock()
     mock_invoker.services.board_images.get_all_board_image_names_for_board.return_value = []
 

tests/app/routers/test_workflows_multiuser.py

@@ -59,12 +59,14 @@ def client():
 def mock_services() -> InvocationServices:
     from invokeai.app.services.board_image_records.board_image_records_sqlite import SqliteBoardImageRecordStorage
     from invokeai.app.services.board_records.board_records_sqlite import SqliteBoardRecordStorage
+    from invokeai.app.services.board_video_records.board_video_records_sqlite import SqliteBoardVideoRecordStorage
     from invokeai.app.services.boards.boards_default import BoardService
     from invokeai.app.services.bulk_download.bulk_download_default import BulkDownloadService
     from invokeai.app.services.client_state_persistence.client_state_persistence_sqlite import (
         ClientStatePersistenceSqlite,
     )
     from invokeai.app.services.image_records.image_records_sqlite import SqliteImageRecordStorage
+    from invokeai.app.services.video_records.video_records_sqlite import SqliteVideoRecordStorage
     from invokeai.app.services.images.images_default import ImageService
     from invokeai.app.services.invocation_cache.invocation_cache_memory import MemoryInvocationCache
     from invokeai.app.services.invocation_stats.invocation_stats_default import InvocationStatsService
@@ -109,8 +111,8 @@ def mock_services() -> InvocationServices:
         external_generation=None,  # type: ignore
         videos=None,  # type: ignore
         video_files=None,  # type: ignore
-        video_records=None,  # type: ignore
-        board_video_records=None,  # type: ignore
+        video_records=SqliteVideoRecordStorage(db=db),
+        board_video_records=SqliteBoardVideoRecordStorage(db=db),
         gallery=None,  # type: ignore
     )
 

tests/conftest.py

@@ -11,13 +11,15 @@
 import pytest
 
 from invokeai.app.services.board_image_records.board_image_records_sqlite import SqliteBoardImageRecordStorage
+from invokeai.app.services.board_video_records.board_video_records_sqlite import SqliteBoardVideoRecordStorage
 from invokeai.app.services.board_records.board_records_sqlite import SqliteBoardRecordStorage
 from invokeai.app.services.boards.boards_default import BoardService
 from invokeai.app.services.bulk_download.bulk_download_default import BulkDownloadService
 from invokeai.app.services.client_state_persistence.client_state_persistence_sqlite import ClientStatePersistenceSqlite
 from invokeai.app.services.config.config_default import InvokeAIAppConfig
 from invokeai.app.services.external_generation.external_generation_default import ExternalGenerationService
 from invokeai.app.services.image_records.image_records_sqlite import SqliteImageRecordStorage
+from invokeai.app.services.video_records.video_records_sqlite import SqliteVideoRecordStorage
 from invokeai.app.services.images.images_default import ImageService
 from invokeai.app.services.invocation_cache.invocation_cache_memory import MemoryInvocationCache
 from invokeai.app.services.invocation_services import InvocationServices
@@ -71,8 +73,8 @@ def mock_services() -> InvocationServices:
         users=UserService(db),
         videos=None,  # type: ignore
         video_files=None,  # type: ignore
-        video_records=None,  # type: ignore
-        board_video_records=None,  # type: ignore
+        video_records=SqliteVideoRecordStorage(db=db),
+        board_video_records=SqliteBoardVideoRecordStorage(db=db),
         gallery=None,  # type: ignore
     )