refactor: make copyFile public (internally)

Author: G-Rath

cmd/osv-scanner/internal/testcmd/copy.go

@@ -8,7 +8,7 @@ import (
 	"testing"
 )
 
-func copyFile(from, to string) (string, error) {
+func CopyFile(from, to string) (string, error) {
 	b, err := os.ReadFile(from)
 	if err != nil {
 		return "", fmt.Errorf("could not read test file: %w", err)
@@ -37,7 +37,7 @@ func CopyFileFlagTo(t *testing.T, tc Case, flagName string, dir string) string {
 		return ""
 	}
 
-	newPath, err := copyFile(flagValue, filepath.Join(dir, filepath.Base(flagValue)))
+	newPath, err := CopyFile(flagValue, filepath.Join(dir, filepath.Base(flagValue)))
 
 	if err != nil {
 		t.Fatalf("%v", err)

cmd/osv-scanner/internal/testcmd/git.go

@@ -43,7 +43,7 @@ func SetupGitFixtures() (func(), error) {
 	}
 
 	for _, f := range gitIgnoreFiles {
-		gitignoreFile, err := copyFile(f, filepath.Join(filepath.Dir(f), ".gitignore"))
+		gitignoreFile, err := CopyFile(f, filepath.Join(filepath.Dir(f), ".gitignore"))
 
 		if err != nil {
 			return cleaner, err

cmd/osv-scanner/scan/source/command_test.go

@@ -1,7 +1,6 @@
 package source_test
 
 import (
-	"fmt"
 	"net/http"
 	"os"
 	"path/filepath"
@@ -1718,19 +1717,6 @@ func TestCommand_FlagDeprecatedPackages(t *testing.T) {
 	}
 }
 
-func copyFile(from, to string) (string, error) {
-	b, err := os.ReadFile(from)
-	if err != nil {
-		return "", fmt.Errorf("could not read test file: %w", err)
-	}
-
-	if err := os.WriteFile(to, b, 0600); err != nil {
-		return "", fmt.Errorf("could not copy test file: %w", err)
-	}
-
-	return to, nil
-}
-
 func TestCommand_UpdateConfigIgnores(t *testing.T) {
 	t.Parallel()
 
@@ -1772,7 +1758,7 @@ func TestCommand_UpdateConfigIgnores(t *testing.T) {
 					t.Fatal(err)
 				}
 
-				_, err = copyFile("testdata/locks-with-invalid-and-configs/"+file, testDir+"/"+file)
+				_, err = testcmd.CopyFile("testdata/locks-with-invalid-and-configs/"+file, testDir+"/"+file)
 
 				if err != nil {
 					t.Fatal(err)