Fix async/await bug in folder_rename_service
- _update_series_folder: make async, await AnimeSeriesService.update() - test: fix mock method name get_by_key -> get_by_folder Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
@@ -363,7 +363,7 @@ def _cleanup_orphaned_folder(old_path: Path, new_path: Path, dry_run: bool = Fal
|
|||||||
return False
|
return False
|
||||||
|
|
||||||
|
|
||||||
def _update_series_folder(db, series, new_folder: str) -> None:
|
async def _update_series_folder(db, series, new_folder: str) -> None:
|
||||||
"""Update AnimeSeries.folder in the database.
|
"""Update AnimeSeries.folder in the database.
|
||||||
|
|
||||||
Args:
|
Args:
|
||||||
@@ -374,7 +374,7 @@ def _update_series_folder(db, series, new_folder: str) -> None:
|
|||||||
if series is None:
|
if series is None:
|
||||||
return
|
return
|
||||||
|
|
||||||
AnimeSeriesService.update(db, series.id, folder=new_folder)
|
await AnimeSeriesService.update(db, series.id, folder=new_folder)
|
||||||
logger.info("Updated AnimeSeries.folder: %s (id=%s)", new_folder, series.id)
|
logger.info("Updated AnimeSeries.folder: %s (id=%s)", new_folder, series.id)
|
||||||
|
|
||||||
|
|
||||||
@@ -455,7 +455,7 @@ async def _update_database_paths(
|
|||||||
series = s
|
series = s
|
||||||
break
|
break
|
||||||
|
|
||||||
_update_series_folder(db, series, new_folder)
|
await _update_series_folder(db, series, new_folder)
|
||||||
|
|
||||||
if series is None:
|
if series is None:
|
||||||
return
|
return
|
||||||
|
|||||||
@@ -226,7 +226,7 @@ class TestUpdateDatabasePaths:
|
|||||||
mock_get_db.return_value.__aenter__ = AsyncMock(return_value=mock_db)
|
mock_get_db.return_value.__aenter__ = AsyncMock(return_value=mock_db)
|
||||||
mock_get_db.return_value.__aexit__ = AsyncMock(return_value=False)
|
mock_get_db.return_value.__aexit__ = AsyncMock(return_value=False)
|
||||||
|
|
||||||
mock_series_svc.get_by_key = AsyncMock(return_value=mock_series)
|
mock_series_svc.get_by_folder = AsyncMock(return_value=mock_series)
|
||||||
mock_series_svc.get_all = AsyncMock(return_value=[])
|
mock_series_svc.get_all = AsyncMock(return_value=[])
|
||||||
mock_series_svc.update = AsyncMock(return_value=mock_series)
|
mock_series_svc.update = AsyncMock(return_value=mock_series)
|
||||||
|
|
||||||
@@ -267,7 +267,7 @@ class TestUpdateDatabasePaths:
|
|||||||
mock_get_db.return_value.__aenter__ = AsyncMock(return_value=mock_db)
|
mock_get_db.return_value.__aenter__ = AsyncMock(return_value=mock_db)
|
||||||
mock_get_db.return_value.__aexit__ = AsyncMock(return_value=False)
|
mock_get_db.return_value.__aexit__ = AsyncMock(return_value=False)
|
||||||
|
|
||||||
mock_series_svc.get_by_key = AsyncMock(return_value=mock_series)
|
mock_series_svc.get_by_folder = AsyncMock(return_value=mock_series)
|
||||||
mock_series_svc.get_all = AsyncMock(return_value=[])
|
mock_series_svc.get_all = AsyncMock(return_value=[])
|
||||||
mock_series_svc.update = AsyncMock(return_value=mock_series)
|
mock_series_svc.update = AsyncMock(return_value=mock_series)
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user