drop the unchecked suppressions that the warning fixes left behind

`HelmTest` uses the static, non-deprecated `Serialization.unmarshal` with
a Jackson `TypeReference`, so the `KubernetesSerialization` constant and
two `@SuppressWarnings("unchecked")` go away.

`ClusterConnectionReconcilerErrorTest` uses the reified `mock()` of
Mockito 5, which needs no suppression and no temporary variable.

Author: ThoSap

operator/src/test/java/it/aboutbits/postgresql/crd/clusterconnection/ClusterConnectionReconcilerErrorTest.java

@@ -52,9 +52,7 @@ void setUp() {
         resource.setSpec(spec);
         resource.setMetadata(metadata);
 
-        @SuppressWarnings("unchecked")
-        Context<ClusterConnection> mockedContext = mock(Context.class);
-        context = mockedContext;
+        context = mock();
     }
 
     @Test

operator/src/test/java/it/aboutbits/postgresql/helm/HelmTest.java

@@ -1,13 +1,13 @@
 package it.aboutbits.postgresql.helm;
 
+import com.fasterxml.jackson.core.type.TypeReference;
 import com.fasterxml.jackson.databind.JsonNode;
 import io.fabric8.kubernetes.api.model.ConfigBuilder;
 import io.fabric8.kubernetes.api.model.LocalObjectReference;
 import io.fabric8.kubernetes.api.model.Volume;
 import io.fabric8.kubernetes.api.model.VolumeMount;
 import io.fabric8.kubernetes.api.model.apps.Deployment;
 import io.fabric8.kubernetes.client.KubernetesClient;
-import io.fabric8.kubernetes.client.utils.KubernetesSerialization;
 import io.fabric8.kubernetes.client.utils.Serialization;
 import io.quarkus.test.junit.QuarkusTest;
 import io.smallrye.common.process.ProcessBuilder;
@@ -37,7 +37,6 @@
 @NullMarked
 class HelmTest {
     private static final String ENV_VAR_KUBECONFIG = "KUBECONFIG";
-    private static final KubernetesSerialization KUBERNETES_SERIALIZATION = new KubernetesSerialization();
 
     /// The Pod and Container list fields that the chart exposes as free-form Helm values.
     private static final List<String> LIST_VALUES = List.of(
@@ -86,19 +85,19 @@ void helmInstall_createsDeployment() throws IOException {
 
         // 1. Verify files exist and contain expected data
         // ./Chart.yaml
-        @SuppressWarnings("unchecked")
-        Map<String, Object> chartMetadata = KUBERNETES_SERIALIZATION.unmarshal(
+        Map<String, Object> chartMetadata = Serialization.unmarshal(
                 Files.newInputStream(chartPath.resolve("Chart.yaml")),
-                Map.class
+                new TypeReference<Map<String, Object>>() {
+                }
         );
 
         assertThat(chartMetadata.get("name")).isEqualTo(chartName);
 
         // ./values.yaml
-        @SuppressWarnings("unchecked")
-        Map<String, Object> values = KUBERNETES_SERIALIZATION.unmarshal(
+        Map<String, Object> values = Serialization.unmarshal(
                 Files.newInputStream(chartPath.resolve("values.yaml")),
-                Map.class
+                new TypeReference<Map<String, Object>>() {
+                }
         );
 
         assertThat(values).containsKey(rootValuesAlias);
@@ -124,7 +123,7 @@ void helmInstall_createsDeployment() throws IOException {
         // ./values.schema.json
         // The type must be declared for every list value, otherwise the generated schema
         // falls back to `string` and `helm install` rejects a list.
-        var valuesSchema = KUBERNETES_SERIALIZATION.unmarshal(
+        var valuesSchema = Serialization.unmarshal(
                 Files.newInputStream(chartPath.resolve("values.schema.json")),
                 JsonNode.class
         );