fix: Logging and Cleanup test names

backup/cmd/list.go

@@ -14,8 +14,7 @@ func buildListCommand() *cobra.Command {
 			configPath, _ := cmd.Flags().GetString("config")
 			rsyncPath, _ := cmd.Flags().GetString("rsync-path")
 			cfg := internal.LoadResolvedConfig(configPath)
-			command := internal.NewRSyncCommand(rsyncPath)
-			command.ListOnly = true
+			command := internal.NewListCommand(rsyncPath)
 
 			cfg.Apply(command)
 		},

backup/cmd/simulate.go

@@ -15,8 +15,7 @@ func buildSimulateCommand() *cobra.Command {
 			rsyncPath, _ := cmd.Flags().GetString("rsync-path")
 
 			cfg := internal.LoadResolvedConfig(configPath)
-			command := internal.NewRSyncCommand(rsyncPath)
-			command.Simulate = true
+			command := internal.NewRSyncSimulateCommand(rsyncPath)
 
 			cfg.Apply(command)
 		},

backup/internal/helper.go

@@ -38,13 +38,6 @@ func createFileLogger() (*log.Logger, string) {
 		log.Fatalf("Failed to open overall log file: %v", err)
 	}
 
-	defer func() {
-		err := overallLogFile.Close()
-		if err != nil {
-			log.Fatalf("Failed to close overall log file: %v", err)
-		}
-	}()
-
 	logger := log.New(overallLogFile, "", log.LstdFlags)
 
 	return logger, logPath

backup/internal/job.go

@@ -1,29 +1,9 @@
 package internal
 
-import (
-	"fmt"
-	"strings"
-)
-
 func (job Job) Apply(rsync RSyncCommand, logPath string) string {
 	if !job.Enabled {
 		return "SKIPPED"
 	}
 
-	args := rsync.ArgumentsForJob(job, logPath)
-	fmt.Printf("Job: %s\n", job.Name)
-	fmt.Printf("Command: rsync %s %s\n", rsync.BinPath, strings.Join(args, " "))
-
-	if rsync.ListOnly {
-		return "SUCCESS"
-	}
-
-	out, err := rsync.Executor.Execute(rsync.BinPath, args...)
-	fmt.Printf("Output:\n%s\n", string(out))
-
-	if err != nil {
-		return "FAILURE"
-	}
-
-	return "SUCCESS"
+	return rsync.Run(job, logPath)
 }

backup/internal/rsync.go

@@ -24,14 +24,30 @@ func NewRSyncCommand(binPath string) RSyncCommand {
 	}
 }
 
+func NewRSyncSimulateCommand(binPath string) RSyncCommand {
+	return RSyncCommand{
+		BinPath:  binPath,
+		Simulate: true,
+		Executor: &RealSync{},
+	}
+}
+
+func NewListCommand(binPath string) RSyncCommand {
+	return RSyncCommand{
+		BinPath:  binPath,
+		ListOnly: true,
+		Executor: &RealSync{},
+	}
+}
+
 func (command RSyncCommand) GetVersionInfo() (string, error) {
 	rsyncPath := command.BinPath
 
 	if !filepath.IsAbs(rsyncPath) {
 		return "", fmt.Errorf("%w: \"%s\"", ErrInvalidRsyncPath, rsyncPath)
 	}
 
-	output, err := command.Executor.Execute(rsyncPath, "--version")
+	output, err := command.Executor.Execute("--version")
 	if err != nil {
 		return "", fmt.Errorf("error fetching rsync version: %w", err)
 	}
@@ -44,7 +60,7 @@ func (command RSyncCommand) GetVersionInfo() (string, error) {
 	return string(output), nil
 }
 
-func (command RSyncCommand) ArgumentsForJob(job Job, logPath string) []string {
+func ArgumentsForJob(job Job, logPath string, simulate bool) []string {
 	args := []string{"-aiv", "--stats"}
 	if job.Delete {
 		args = append(args, "--delete")
@@ -59,9 +75,28 @@ func (command RSyncCommand) ArgumentsForJob(job Job, logPath string) []string {
 	}
 
 	args = append(args, job.Source, job.Target)
-	if command.Simulate {
+	if simulate {
 		args = append([]string{"--dry-run"}, args...)
 	}
 
 	return args
 }
+
+func (rsync RSyncCommand) Run(job Job, logPath string) string {
+	args := ArgumentsForJob(job, logPath, rsync.Simulate)
+	fmt.Printf("Job: %s\n", job.Name)
+	fmt.Printf("Command: rsync %s %s\n", rsync.BinPath, strings.Join(args, " "))
+
+	if rsync.ListOnly {
+		return "SUCCESS"
+	}
+
+	out, err := rsync.Executor.Execute(rsync.BinPath, args...)
+	fmt.Printf("Output:\n%s\n", string(out))
+
+	if err != nil {
+		return "FAILURE"
+	}
+
+	return "SUCCESS"
+}

backup/internal/test/rsync_test.go

@@ -14,18 +14,13 @@ var errCommandNotFound = errors.New("command not found")
 
 const rsyncPath = "/usr/bin/rsync"
 
-func TestBuildRsyncCmd(t *testing.T) {
+func TestArgumentsForJob(t *testing.T) {
 	job := *NewJob(
 		WithSource("/home/user/Music/"),
 		WithTarget("/target/user/music/home"),
 		WithExclusions([]string{"*.tmp", "node_modules/"}),
 	)
-	command := internal.RSyncCommand{
-		BinPath:  rsyncPath,
-		Simulate: true,
-		Executor: nil,
-	}
-	args := command.ArgumentsForJob(job, "")
+	args := internal.ArgumentsForJob(job, "", true)
 
 	expectedArgs := []string{
 		"--dry-run", "-aiv", "--stats", "--delete",
@@ -36,7 +31,7 @@ func TestBuildRsyncCmd(t *testing.T) {
 	assert.Equal(t, strings.Join(expectedArgs, " "), strings.Join(args, " "))
 }
 
-func TestFetchRsyncVersion_Success(t *testing.T) {
+func TestGetVersionInfo_Success(t *testing.T) {
 	rsync := internal.RSyncCommand{
 		BinPath: rsyncPath,
 		Executor: &MockCommandExecutor{
@@ -51,7 +46,7 @@ func TestFetchRsyncVersion_Success(t *testing.T) {
 	assert.Equal(t, "rsync  version 3.2.3  protocol version 31\n", versionInfo)
 }
 
-func TestFetchRsyncVersion_CommandError(t *testing.T) {
+func TestGetVersionInfo_CommandError(t *testing.T) {
 	rsync := internal.RSyncCommand{
 		BinPath: rsyncPath,
 		Executor: &MockCommandExecutor{
@@ -66,7 +61,7 @@ func TestFetchRsyncVersion_CommandError(t *testing.T) {
 	assert.Empty(t, versionInfo)
 }
 
-func TestFetchRsyncVersion_InvalidOutput(t *testing.T) {
+func TestGetVersionInfo_InvalidOutput(t *testing.T) {
 	rsync := internal.RSyncCommand{
 		BinPath: rsyncPath,
 		Executor: &MockCommandExecutor{
@@ -81,7 +76,7 @@ func TestFetchRsyncVersion_InvalidOutput(t *testing.T) {
 	assert.Empty(t, versionInfo)
 }
 
-func TestFetchRsyncVersion_EmptyPath(t *testing.T) {
+func TestGetVersionInfo_EmptyPath(t *testing.T) {
 	rsync := internal.RSyncCommand{
 		BinPath: "",
 		Executor: &MockCommandExecutor{
@@ -97,7 +92,7 @@ func TestFetchRsyncVersion_EmptyPath(t *testing.T) {
 	assert.Empty(t, versionInfo)
 }
 
-func TestFetchRsyncVersion_IncompletePath(t *testing.T) {
+func TestGetVersionInfo_IncompletePath(t *testing.T) {
 	rsync := internal.RSyncCommand{
 		BinPath: "bin/rsync",
 		Executor: &MockCommandExecutor{