From e9c4cd9f3f15227968b0a48f4d087ca443ac68bf Mon Sep 17 00:00:00 2001
From: Tim Ramlot <42113979+inteon@users.noreply.github.com>
Date: Sat, 6 May 2023 21:36:53 +0200
Subject: [PATCH] check that issuer is not nil before reading its field values
 on cleanup

Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>
---
 test/e2e/suite/issuers/venafi/tpp/certificate.go | 4 +++-
 test/e2e/suite/issuers/venafi/tpp/setup.go       | 4 +++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/test/e2e/suite/issuers/venafi/tpp/certificate.go b/test/e2e/suite/issuers/venafi/tpp/certificate.go
index c9b0ce8fa..341719316 100644
--- a/test/e2e/suite/issuers/venafi/tpp/certificate.go
+++ b/test/e2e/suite/issuers/venafi/tpp/certificate.go
@@ -69,7 +69,9 @@ var _ = TPPDescribe("Certificate with a properly configured Issuer", func() {
 
 	AfterEach(func() {
 		By("Cleaning up")
-		f.CertManagerClientSet.CertmanagerV1().Issuers(f.Namespace.Name).Delete(context.TODO(), issuer.Name, metav1.DeleteOptions{})
+		if issuer != nil {
+			f.CertManagerClientSet.CertmanagerV1().Issuers(f.Namespace.Name).Delete(context.TODO(), issuer.Name, metav1.DeleteOptions{})
+		}
 	})
 
 	It("should obtain a signed certificate for a single domain", func() {
diff --git a/test/e2e/suite/issuers/venafi/tpp/setup.go b/test/e2e/suite/issuers/venafi/tpp/setup.go
index 75bfb53fe..fad976a2a 100644
--- a/test/e2e/suite/issuers/venafi/tpp/setup.go
+++ b/test/e2e/suite/issuers/venafi/tpp/setup.go
@@ -46,7 +46,9 @@ var _ = TPPDescribe("properly configured Venafi TPP Issuer", func() {
 
 	AfterEach(func() {
 		By("Cleaning up")
-		f.CertManagerClientSet.CertmanagerV1().Issuers(f.Namespace.Name).Delete(context.TODO(), issuer.Name, metav1.DeleteOptions{})
+		if issuer != nil {
+			f.CertManagerClientSet.CertmanagerV1().Issuers(f.Namespace.Name).Delete(context.TODO(), issuer.Name, metav1.DeleteOptions{})
+		}
 	})
 
 	It("should set Ready=True accordingly", func() {