From 56be3d9c31b972c1bb6eb03e8fb2a624b84852d9 Mon Sep 17 00:00:00 2001 From: Houkime <> Date: Fri, 8 Sep 2023 16:22:53 +0000 Subject: [PATCH] fix(backup): trim auto-snapshots on setting the quotas --- selfprivacy_api/backup/__init__.py | 3 +++ tests/test_graphql/test_backup.py | 4 +++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/selfprivacy_api/backup/__init__.py b/selfprivacy_api/backup/__init__.py index 336b705..7056071 100644 --- a/selfprivacy_api/backup/__init__.py +++ b/selfprivacy_api/backup/__init__.py @@ -396,6 +396,9 @@ class Backups: ) ) + for service in get_all_services(): + Backups._prune_auto_snaps(service) + # Restoring @staticmethod diff --git a/tests/test_graphql/test_backup.py b/tests/test_graphql/test_backup.py index 550c56b..5daae0c 100644 --- a/tests/test_graphql/test_backup.py +++ b/tests/test_graphql/test_backup.py @@ -520,8 +520,10 @@ def test_too_many_auto(backups, dummy_service): # Retroactivity quota.total = 1 Backups.set_autobackup_quotas(quota) - snap5 = Backups.back_up(dummy_service, BackupReason.AUTO) + snaps = Backups.get_snapshots(dummy_service) + assert len(snaps) == 1 + snap5 = Backups.back_up(dummy_service, BackupReason.AUTO) snaps = Backups.get_snapshots(dummy_service) assert len(snaps) == 1 assert snap5 in snaps