From 9195adfc06551f312c04b2f21f82c8c3658b9f3c Mon Sep 17 00:00:00 2001 From: James Liu Date: Wed, 10 Jan 2024 13:18:57 +1100 Subject: backup: Remove test exceptions for WAL Now that backups no longer invoke the RemoveAll RPC, we can remove the exemption for these tests when the WAL is enabled. --- internal/cli/gitalybackup/restore_test.go | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/internal/cli/gitalybackup/restore_test.go b/internal/cli/gitalybackup/restore_test.go index 033420d8f..4b1eee9aa 100644 --- a/internal/cli/gitalybackup/restore_test.go +++ b/internal/cli/gitalybackup/restore_test.go @@ -26,13 +26,6 @@ import ( func TestRestoreSubcommand(t *testing.T) { gittest.SkipWithSHA256(t) - testhelper.SkipWithWAL(t, ` -RemoveAll is removing the entire content of the storage. This would also remove the database's and -the transaction manager's disk state. The RPC needs to be updated to shut down all partitions and -the database and only then perform the removal. - -Issue: https://gitlab.com/gitlab-org/gitaly/-/issues/5269`) - ctx := testhelper.Context(t) cfg := testcfg.Build(t) @@ -122,13 +115,6 @@ Issue: https://gitlab.com/gitlab-org/gitaly/-/issues/5269`) func TestRestoreSubcommand_serverSide(t *testing.T) { gittest.SkipWithSHA256(t) - testhelper.SkipWithWAL(t, ` -RemoveAll is removing the entire content of the storage. This would also remove the database's and -the transaction manager's disk state. The RPC needs to be updated to shut down all partitions and -the database and only then perform the removal. - -Issue: https://gitlab.com/gitlab-org/gitaly/-/issues/5269`) - ctx := testhelper.Context(t) backupDir := testhelper.TempDir(t) -- cgit v1.2.3