From a65b546c615748018a9b76907e19ccb36f94d2f1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Miloslav=20Trma=C4=8D?= Date: Thu, 7 Mar 2024 14:08:54 +0100 Subject: [PATCH] Remove copySparseFile MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It is unused, and it clearly doesn't work (it closes dest before writing anything to it). Just drop it, it can always be re-added. Should not change behavior. Signed-off-by: Miloslav Trmač --- pkg/machine/e2e/machine_test.go | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/pkg/machine/e2e/machine_test.go b/pkg/machine/e2e/machine_test.go index 4ef85a2eec..f067ab451a 100644 --- a/pkg/machine/e2e/machine_test.go +++ b/pkg/machine/e2e/machine_test.go @@ -161,22 +161,6 @@ func teardown(origHomeDir string, testDir string, mb *machineTestBuilder) { } } -// copySparseFile is a helper method for tests only. copies a file sparsely -// between two string inputs -func copySparseFile(src, dst string) error { //nolint:unused - dstWriter, err := os.OpenFile(dst, os.O_CREATE|os.O_RDWR, 0600) - if err != nil { - return err - } - dstWriter.Close() - fSrc, err := os.Open(src) - if err != nil { - return err - } - defer fSrc.Close() - return copySparse(dstWriter, fSrc) -} - // copySparse is a helper method for tests only; caller is responsible for closures func copySparse(dst compression.WriteSeekCloser, src io.Reader) error { spWriter := compression.NewSparseWriter(dst)