refactor: use builtin copy method instead of cp command

.gitignore

@@ -20,4 +20,6 @@
 # Test Files
 /internal/constants/assets/RUNTIPI_VERSION
 /internal/constants/assets/CLI_VERSION
-/.env.local
+/.env.local
+/cpDest.txt
+/cpSrc.txt

internal/backups/backups.go

@@ -7,6 +7,8 @@ import (
 	"path"
 	"path/filepath"
 	"time"
+
+	"github.com/steveiliop56/runtipi-cli-go/internal/system"
 )
 
 func CreateBackup() (string, error) {
@@ -36,7 +38,7 @@ func CreateBackup() (string, error) {
 	}
 
 	// Copy archive to backups folder
-	_, cpErr := exec.Command("cp", tmpArchivePath, archivePath).Output()
+	cpErr := system.Copy(tmpArchivePath, archivePath)
 
 	if cpErr != nil {
 		return "", cpErr

internal/release/release.go

@@ -141,7 +141,7 @@ func BackupCurrentCLI() (error) {
 	cliPath := path.Join(rootFolder, "runtipi-cli-go")
 	cliBackupPath := path.Join(rootFolder, "runtipi-cli-go.bak")
 
-	_, copyErr := exec.Command("cp", cliPath, cliBackupPath).Output()
+	copyErr := system.Copy(cliPath, cliBackupPath)
 
 	if copyErr != nil {
 		return copyErr

internal/system/system.go

@@ -4,6 +4,7 @@ import (
 	"crypto/sha256"
 	"encoding/hex"
 	"errors"
+	"io"
 	"net"
 	"os"
 	"os/exec"
@@ -152,5 +153,43 @@ func EnsureTar() (error) {
 		return err
 	}
 
+	return nil
+}
+
+func Copy(src string, dest string) (error) {
+	source, sourceErr := os.Open(src)
+
+	if sourceErr != nil {
+		return sourceErr
+	}
+
+	defer source.Close()
+
+	sourceStat, sourceStatErr := source.Stat()
+
+	if sourceStatErr != nil {
+		return sourceStatErr
+	}
+
+	destination, destinationErr := os.Create(dest)
+
+	if destinationErr != nil {
+		return destinationErr
+	}
+
+	defer destination.Close()
+
+	_, copyErr := io.Copy(destination, source)
+
+	if copyErr != nil {
+		return copyErr
+	}
+
+	chmodErr := os.Chmod(dest, sourceStat.Mode())
+
+	if chmodErr != nil {
+		return chmodErr
+	}
+
 	return nil
 }

internal/system/system_test.go

@@ -265,4 +265,61 @@ func TestEnsureFilePermissions(t *testing.T) {
 			}
 		}
 	}
+}
+
+// Test copy
+func TestCopy(t *testing.T) {
+	// Get root folder
+	rootFolder, osErr := os.Getwd()
+
+	if osErr != nil {
+		t.Fatalf("Failed to get root folder, error: %s\n", osErr)
+	}
+
+	// Define paths
+	srcFilePath := path.Join(rootFolder, "cpSrc.txt")
+	destFilePath := path.Join(rootFolder, "cpDest.txt")
+
+	// Delete old files
+	os.Remove(srcFilePath)
+	os.Remove(destFilePath)
+
+	// Create src file
+	writeErr := os.WriteFile(srcFilePath, []byte("test"), 0644)
+
+	// Check for errors
+	if writeErr != nil {
+		t.Fatalf("Failed to write test file, error: %s\n", writeErr)
+	}
+
+	// Copy file
+	cpErr := system.Copy(srcFilePath, destFilePath)
+
+	// Check for errors
+	if cpErr != nil {
+		t.Fatalf("Failed to copy test file, error: %s\n", cpErr)
+	}
+
+	// Check if file got copied
+	destStatus, statErr := os.Stat(destFilePath)
+
+	if statErr != nil {
+		t.Fatalf("Test file doesn't exist, error: %s\n", statErr)
+	}
+
+	// Check if permissions are the same
+	if destStatus.Mode() != os.FileMode(0644) {
+		t.Fatalf("Permissions are not the same! Expected -rw-r--r--, got %s\n", destStatus.Mode().String())
+	}
+
+	// Check contents
+	contents, readErr := os.ReadFile(destFilePath)
+
+	if readErr != nil {
+		t.Fatalf("Failed to read destination file, error: %s\n", readErr)
+	}
+
+	if string(contents) != "test" {
+		t.Fatalf("Contents not the same! Expected test got %s\n", string(contents))
+	}
 }