fixed panic in describe output while displaying Hooksattempted or failed count

Author: stillalearner

cmd/non-admin/backup/describe.go

@@ -372,8 +372,13 @@ func printNonAdminBackupDetails(cmd *cobra.Command, nab *nacv1alpha1.NonAdminBac
 		fmt.Fprintf(out, "\n")
 
 		// Hooks
-		fmt.Fprintf(out, "HooksAttempted:  %d\n", status.HookStatus.HooksAttempted)
-		fmt.Fprintf(out, "HooksFailed:     %d\n", status.HookStatus.HooksFailed)
+		if status.HookStatus != nil {
+			fmt.Fprintf(out, "HooksAttempted:  %d\n", status.HookStatus.HooksAttempted)
+			fmt.Fprintf(out, "HooksFailed:     %d\n", status.HookStatus.HooksFailed)
+		} else {
+			fmt.Fprintf(out, "HooksAttempted:  <none>\n")
+			fmt.Fprintf(out, "HooksFailed:     <none>\n")
+		}
 	} else {
 		// Velero backup not available yet
 		fmt.Fprintf(out, "Velero backup information not yet available.\n")

cmd/non-admin/restore/describe.go

@@ -317,8 +317,13 @@ func printNonAdminRestoreDetails(cmd *cobra.Command, nar *nacv1alpha1.NonAdminRe
 		fmt.Fprintf(out, "\n")
 
 		// Hooks
-		fmt.Fprintf(out, "HooksAttempted:  %d\n", status.HookStatus.HooksAttempted)
-		fmt.Fprintf(out, "HooksFailed:     %d\n", status.HookStatus.HooksFailed)
+		if status.HookStatus != nil {
+			fmt.Fprintf(out, "HooksAttempted:  %d\n", status.HookStatus.HooksAttempted)
+			fmt.Fprintf(out, "HooksFailed:     %d\n", status.HookStatus.HooksFailed)
+		} else {
+			fmt.Fprintf(out, "HooksAttempted:  <none>\n")
+			fmt.Fprintf(out, "HooksFailed:     <none>\n")
+		}
 	} else {
 		// Velero restore not available yet
 		fmt.Fprintf(out, "Velero restore information not yet available.\n")

cmd/root_test.go

@@ -777,8 +777,8 @@ func TestApplyTimeoutToConfig_DialerTimeout(t *testing.T) {
 	}
 
 	// Should complete within a reasonable time of the timeout
-	// Allow some margin for test execution overhead
-	maxExpected := timeout + 500*time.Millisecond
+	// Allow some margin for test execution overhead and system load variations
+	maxExpected := timeout + 1*time.Second
 	if elapsed > maxExpected {
 		t.Errorf("Dial took too long: %v (expected ~%v)", elapsed, timeout)
 	}