Fixup

Remove default values to prevent parameters from being optional

Signed-off-by: Toni Finger <[email protected]>

Author: tonifinger

Tests/kaas/plugin/interface.py

@@ -24,20 +24,20 @@ class KubernetesClusterPlugin():
 
           class PluginX(KubernetesClusterPlugin):
 
-              def __init__(self, config_file=None):
+              def __init__(self, config_file):
                   self.config = config_file
 
-              def create_cluster(self, cluster_name="scs-cluster", version=None, kubeconfig_filepath=None):
+              def create_cluster(self, cluster_name, version, kubeconfig_filepath):
                   self.cluster = ClusterAPI(name=cluster_name, image=cluster_image, kubeconfig_filepath)
                   self.cluster.create(self.config)
 
-              def delete_cluster(self, cluster_name=None, version=None):
+              def delete_cluster(self, cluster_name, version):
                   self.cluster = ClusterAPI(cluster_name)
                   self.cluster.delete()
         ..
     """
 
-    def create_cluster(self, cluster_name="scs-cluster", version=None, kubeconfig_filepath=None):
+    def create_cluster(self, cluster_name, version, kubeconfig_filepath):
         """
         This method is to be called to create a k8s cluster
         :param: cluster_name:
@@ -46,7 +46,7 @@ def create_cluster(self, cluster_name="scs-cluster", version=None, kubeconfig_fi
         """
         raise NotImplementedError
 
-    def delete_cluster(self, cluster_name=None, version=None):
+    def delete_cluster(self, cluster_name, version):
         """
         This method is to be called in order to unprovision a cluster
         :param: cluster_name:

Tests/kaas/plugin/plugin_kind.py

@@ -14,14 +14,14 @@ class PluginKind(KubernetesClusterPlugin):
     Plugin to handle the provisioning of kubernetes cluster for
     conformance testing purpose with the use of Kind
     """
-    def __init__(self, config_file=None):
+    def __init__(self, config_file):
         logger.info("Init PluginKind")
         self.config = config_file
         logger.debug(self.config)
         self.working_directory = os.getcwd()
         logger.debug(f"Working from {self.working_directory}")
 
-    def create_cluster(self, cluster_name="scs-cluster", version=None, kubeconfig=None):
+    def create_cluster(self, cluster_name, version, kubeconfig):
         """
         This method is to be called to create a k8s cluster
         :param: kubernetes_version:
@@ -45,6 +45,6 @@ def create_cluster(self, cluster_name="scs-cluster", version=None, kubeconfig=No
         else:
             self.cluster.create(self.config)
 
-    def delete_cluster(self, cluster_name=None, version=None):
+    def delete_cluster(self, cluster_name, version):
         self.cluster = KindCluster(cluster_name)
         self.cluster.delete()

Tests/kaas/plugin/plugin_static.py

@@ -9,11 +9,11 @@ class PluginStatic(KubernetesClusterPlugin):
     using a kubeconfig file
     """
 
-    def __init__(self, config_file=None):
+    def __init__(self, config_file):
         self.kubeconfig_path = config_file
 
-    def create_cluster(self, cluster_name="scs-cluster", version=None, kubeconfig=None):
+    def create_cluster(self, cluster_name, version, kubeconfig):
         shutil.copyfile(self.kubeconfig_path, kubeconfig)
 
-    def delete_cluster(self, cluster_name=None, version=None):
+    def delete_cluster(self, cluster_name, version):
         pass