feature(backups): feature(backups): return a snapshot from start_backup

selfprivacy_api/backup/__init__.py

@@ -223,6 +223,7 @@ class Backups:
             raise e
 
         Jobs.update(job, status=JobStatus.FINISHED)
+        return snapshot
 
     @staticmethod
     def init_repo(service: Optional[Service] = None):

selfprivacy_api/backup/storage.py

@@ -104,7 +104,7 @@ class Storage:
 
     @staticmethod
     def get_cached_snapshot_by_id(snapshot_id: str) -> Optional[Snapshot]:
-        key = redis.keys(REDIS_SNAPSHOTS_PREFIX + snapshot_id)
+        key = REDIS_SNAPSHOTS_PREFIX + snapshot_id
         if not redis.exists(key):
             return None
         return hash_as_model(redis, key, Snapshot)

tests/test_graphql/test_backup.py

@@ -243,6 +243,18 @@ def assert_job_had_progress(job_type):
     assert len(Jobs.progress_updates(job)) > 0
 
 
+def test_snapshots_by_id(backups, dummy_service):
+    snap1 = Backups.back_up(dummy_service)
+    snap2 = Backups.back_up(dummy_service)
+    snap3 = Backups.back_up(dummy_service)
+
+    assert snap2.id is not None
+    assert snap2.id != ""
+
+    assert len(Backups.get_snapshots(dummy_service)) == 3
+    assert Backups.get_snapshot_by_id(snap2.id).id == snap2.id
+
+
 def test_backup_service_task(backups, dummy_service):
     handle = start_backup(dummy_service)
     handle(blocking=True)