fix: handle nil logger in SetLogger method and add corresponding test (#27)

Author: sgaunet

pkg/app/app.go

@@ -26,8 +26,12 @@ func NewApp() *App {
 }
 
 // SetLogger sets the logger
-func (a *App) SetLogger(logger logger.Logger) {
-	a.logger = logger
+func (a *App) SetLogger(log logger.Logger) {
+	if log == nil {
+		a.logger = logger.NoLogger()
+		return
+	}
+	a.logger = log
 }
 
 // SetBackupCmd sets the backup command

pkg/app/app_test.go

@@ -29,3 +29,16 @@ func TestRunCommandReportsError(t *testing.T) {
 		t.Fatalf("Expected error, got nil")
 	}
 }
+
+func TestSetLoggerNil(t *testing.T) {
+	a := app.NewApp()
+	if a == nil {
+		t.Fatalf("Expected non-nil app, got nil")
+	}
+	a.SetLogger(nil)
+	a.SetBackupCmd([]string{"echo", "hello"})
+	err := a.Run()
+	if err != nil {
+		t.Fatalf("Expected nil error, got %v", err)
+	}
+}