From bf33fff20d724cab22cc560bd075425ee4f7463f Mon Sep 17 00:00:00 2001
From: Houkime <>
Date: Fri, 23 Feb 2024 18:36:11 +0000
Subject: [PATCH] fix(backups): finish the autobackup job

---
 selfprivacy_api/backup/tasks.py | 2 ++
 tests/test_autobackup.py        | 6 +++++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/selfprivacy_api/backup/tasks.py b/selfprivacy_api/backup/tasks.py
index dc0e6bd..281ef40 100644
--- a/selfprivacy_api/backup/tasks.py
+++ b/selfprivacy_api/backup/tasks.py
@@ -100,6 +100,8 @@ def do_autobackup() -> None:
         progress = progress + progress_per_service
         Jobs.update(job, JobStatus.RUNNING, progress=progress)
 
+    Jobs.update(job, JobStatus.FINISHED)
+
 
 @huey.periodic_task(validate_datetime=validate_datetime)
 def automatic_backup() -> None:
diff --git a/tests/test_autobackup.py b/tests/test_autobackup.py
index 410694b..d704d12 100644
--- a/tests/test_autobackup.py
+++ b/tests/test_autobackup.py
@@ -17,8 +17,9 @@ from selfprivacy_api.backup.tasks import (
     automatic_backup,
     do_autobackup,
 )
+from selfprivacy_api.backup.jobs import autobackup_job_type
 
-from tests.test_backup import backups
+from tests.test_backup import backups, assert_job_finished
 from tests.test_graphql.test_services import only_dummy_service
 
 
@@ -74,6 +75,7 @@ def test_autobackup_taskbody(backups, only_dummy_service):
     backup_period = 13  # minutes
 
     assert Backups.get_all_snapshots() == []
+    assert_job_finished(autobackup_job_type(), count=0)
 
     Backups.set_autobackup_period_minutes(backup_period)
     assert Backups.is_time_to_backup_service(dummy_service, now)
@@ -88,6 +90,8 @@ def test_autobackup_taskbody(backups, only_dummy_service):
     assert snapshots[0].service_name == dummy_service.get_id()
     assert snapshots[0].reason == BackupReason.AUTO
 
+    assert_job_finished(autobackup_job_type(), count=1)
+
 
 def test_autobackup_timer_periods(backups, dummy_service):
     now = datetime.now(timezone.utc)