From 01f4a45fdf95698193428997149a86e3900dda9a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=B6ren=20Henning?= <soeren.henning@email.uni-kiel.de>
Date: Sat, 3 Dec 2022 16:34:31 +0100
Subject: [PATCH] Refactor patchers

---
 .../kubernetes/patcher/LabelPatcher.kt        | 35 ++-----------------
 .../kubernetes/patcher/MatchLabelPatcher.kt   |  4 +--
 .../kubernetes/patcher/NamePatcher.kt         | 21 +----------
 3 files changed, 5 insertions(+), 55 deletions(-)

diff --git a/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/LabelPatcher.kt b/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/LabelPatcher.kt
index 8bb5be97e..9e690ea68 100644
--- a/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/LabelPatcher.kt
+++ b/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/LabelPatcher.kt
@@ -12,39 +12,10 @@ class LabelPatcher(
     AbstractPatcher() {
 
     override fun patchSingleResource(resource: HasMetadata, value: String): HasMetadata {
-        when (resource) {
-            is Deployment -> {
-                if (resource.metadata.labels == null) {
-                    resource.metadata.labels = mutableMapOf()
-                }
-                resource.metadata.labels[this.variableName] = value
-            }
-            is StatefulSet -> {
-                if (resource.metadata.labels == null) {
-                    resource.metadata.labels = mutableMapOf()
-                }
-                resource.metadata.labels[this.variableName] = value
-            }
-            is Service -> {
-                if (resource.metadata.labels == null) {
-                    resource.metadata.labels = mutableMapOf()
-                }
-                resource.metadata.labels[this.variableName] = value
-
-            }
-            is ConfigMap -> {
-                if (resource.metadata.labels == null) {
-                    resource.metadata.labels = mutableMapOf()
-                }
-                resource.metadata.labels[this.variableName] = value
-            }
-            is GenericKubernetesResource -> {
-                if (resource.metadata.labels == null) {
-                    resource.metadata.labels = mutableMapOf()
-                }
-                resource.metadata.labels[this.variableName] = value
-            }
+        if (resource.metadata.labels == null) {
+            resource.metadata.labels = mutableMapOf()
         }
+        resource.metadata.labels[this.variableName] = value
         return resource
     }
 }
\ No newline at end of file
diff --git a/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/MatchLabelPatcher.kt b/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/MatchLabelPatcher.kt
index 725c9cf8a..0b60514cf 100644
--- a/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/MatchLabelPatcher.kt
+++ b/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/MatchLabelPatcher.kt
@@ -9,9 +9,7 @@ import io.fabric8.kubernetes.api.model.apps.StatefulSet
  *
  * @property variableName The matchLabel which should be set
  */
-class MatchLabelPatcher(
-    val variableName: String) :
-    AbstractPatcher() {
+class MatchLabelPatcher(val variableName: String) : AbstractPatcher() {
 
     override fun patchSingleResource(resource: HasMetadata, value: String): HasMetadata {
         when (resource) {
diff --git a/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/NamePatcher.kt b/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/NamePatcher.kt
index a6416a7e7..86c6c6b97 100644
--- a/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/NamePatcher.kt
+++ b/theodolite/src/main/kotlin/rocks/theodolite/kubernetes/patcher/NamePatcher.kt
@@ -10,26 +10,7 @@ import io.fabric8.kubernetes.api.model.apps.StatefulSet
 class NamePatcher : AbstractPatcher() {
 
     override fun patchSingleResource(resource: HasMetadata, value: String): HasMetadata {
-        when (resource) {
-            is Deployment -> {
-                resource.metadata.name = value
-            }
-            is StatefulSet -> {
-                resource.metadata.name = value
-            }
-            is Service -> {
-                resource.metadata.name = value
-            }
-            is ConfigMap -> {
-                resource.metadata.name = value
-            }
-            is io.fabric8.kubernetes.api.model.networking.v1.Ingress -> {
-                resource.metadata.name = value
-            }
-            is GenericKubernetesResource -> {
-                resource.metadata.name = value
-            }
-        }
+        resource.metadata.name = value
         return resource
     }
 }
\ No newline at end of file
-- 
GitLab