terminal error fix

kubernetes-sigs · Aug 4, 2023 · a993454 · a993454
1 parent dbcf380
commit a993454
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/pkg/reconcile/reconcile.go b/pkg/reconcile/reconcile.go
@@ -117,6 +117,9 @@ func (te *terminalError) Unwrap() error {
 }
 
 func (te *terminalError) Error() string {
+	if te.err == nil {
+		return "nil terminal error"
+	}
 	return "terminal error: " + te.err.Error()
 }
 

diff --git a/pkg/reconcile/reconcile_test.go b/pkg/reconcile/reconcile_test.go
@@ -96,5 +96,10 @@ var _ = Describe("reconcile", func() {
 
 			Expect(apierrors.IsGone(terminalError)).To(BeTrue())
 		})
+
+		It("should handle nil terminal errors properly", func() {
+			err := reconcile.TerminalError(nil)
+			Expect(err.Error()).To(Equal("nil terminal error"))
+		})
 	})
 })