Fix NullPointerException, only close client in ClientInjectionHandler when client is set (#118) (#135)

BramMeerten · web-flow · commit b00b9ae9f70e · 2023-11-23T09:01:41.000+01:00
Fix NullPointerException, only close client in ClientInjectionHandler when client is set (#118)
diff --git a/fabric8/src/main/java/io/javaoperatorsdk/jenvtest/junit/Fabric8ClientInjectionHandler.java b/fabric8/src/main/java/io/javaoperatorsdk/jenvtest/junit/Fabric8ClientInjectionHandler.java
@@ -45,7 +45,9 @@ private void setKubernetesClientToField(ExtensionContext extensionContext,
 
   @Override
   public void cleanup(ExtensionContext extensionContext) {
-    client.close();
+    if (client != null) {
+      client.close();
+    }
   }
 
   public static Optional<Field> getFieldForKubeClientInjection(ExtensionContext extensionContext,

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,9 @@ private void setKubernetesClientToField(ExtensionContext extensionContext,`
`45`	`45`
`46`	`46`	`@Override`
`47`	`47`	`public void cleanup(ExtensionContext extensionContext) {`
`48`		`- client.close();`
	`48`	`+ if (client != null) {`
	`49`	`+ client.close();`
	`50`	`+ }`
`49`	`51`	`}`
`50`	`52`
`51`	`53`	`public static Optional<Field> getFieldForKubeClientInjection(ExtensionContext extensionContext,`