From 5d444d407e56de94f21712c2a117d7c65deab5e4 Mon Sep 17 00:00:00 2001 From: mamutmk5 <3045922+mamutmk5@users.noreply.github.com> Date: Fri, 21 Jun 2024 17:28:06 +0200 Subject: [PATCH] BC-7239 - sonar 1 --- src/main/java/de/svs/doido/mongo/ConfigmapApi.java | 2 +- .../java/de/svs/doido/mongo/kubernetes/configmap/Write.java | 6 +++--- src/test/java/de/svs/doido/mongo/ConfigmapApiTest.java | 1 - 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/src/main/java/de/svs/doido/mongo/ConfigmapApi.java b/src/main/java/de/svs/doido/mongo/ConfigmapApi.java index 65c2771..7348405 100644 --- a/src/main/java/de/svs/doido/mongo/ConfigmapApi.java +++ b/src/main/java/de/svs/doido/mongo/ConfigmapApi.java @@ -35,7 +35,7 @@ public void configmapsWrite(String namespace, String configmapName) { Configmap cfg = new Configmap(); cfg.setName(configmapName); cfg.setUri("http://www.test.org"); - if( !cfgWrite.writeConfigmap(namespace,configmapName,cfg) ) { + if( !cfgWrite.writeConfigmap(namespace,cfg) ) { throw new ForbiddenException(); } diff --git a/src/main/java/de/svs/doido/mongo/kubernetes/configmap/Write.java b/src/main/java/de/svs/doido/mongo/kubernetes/configmap/Write.java index ce1be92..8bc2357 100644 --- a/src/main/java/de/svs/doido/mongo/kubernetes/configmap/Write.java +++ b/src/main/java/de/svs/doido/mongo/kubernetes/configmap/Write.java @@ -17,7 +17,7 @@ public class Write { @Inject KubernetesClient client; - public boolean writeConfigmap(String namespace, String name, Configmap cfg) { + public boolean writeConfigmap(String namespace, Configmap cfg) { ConfigMap configmap = client.configMaps().inNamespace(namespace).withName(cfg.getName()).get(); Map labels; Map data; @@ -40,8 +40,8 @@ public boolean writeConfigmap(String namespace, String name, Configmap cfg) { } } else { - labels = new HashMap(); - data = new HashMap(); + labels = new HashMap<>(); + data = new HashMap<>(); labels.put("app.kubernetes.io/name", "doido-mongo"); data.put("uri", cfg.getUri()); configmap = new ConfigMapBuilder().withNewMetadata().withName(cfg.getName()).withNamespace(namespace).and().build(); diff --git a/src/test/java/de/svs/doido/mongo/ConfigmapApiTest.java b/src/test/java/de/svs/doido/mongo/ConfigmapApiTest.java index 1135f34..fd221de 100644 --- a/src/test/java/de/svs/doido/mongo/ConfigmapApiTest.java +++ b/src/test/java/de/svs/doido/mongo/ConfigmapApiTest.java @@ -13,7 +13,6 @@ import io.fabric8.kubernetes.api.model.Namespace; import io.fabric8.kubernetes.api.model.NamespaceBuilder; import io.fabric8.kubernetes.api.model.ConfigMap; -import io.fabric8.kubernetes.api.model.ConfigMapList; import io.fabric8.kubernetes.api.model.ConfigMapBuilder; import io.fabric8.kubernetes.api.model.ObjectMeta;