KAB-46 add missing validation

Author: tomasfejfar

kbcstorage/configurations_metadata.py

@@ -58,7 +58,9 @@ def delete(self, component_id, configuration_id, metadata_id):
         if not isinstance(component_id, str) or component_id == '':
             raise ValueError("Invalid component_id '{}'.".format(component_id))
         if not isinstance(configuration_id, str) or configuration_id == '':
-            raise ValueError("Invalid component_id '{}'.".format(configuration_id))
+            raise ValueError("Invalid configuration_id '{}'.".format(configuration_id))
+        if not isinstance(metadata_id, str) or metadata_id == '':
+            raise ValueError("Invalid metadata_id '{}'.".format(metadata_id))
         url = '{}/{}/configs/{}/metadata/{}'.format(self.base_url, component_id, configuration_id, metadata_id)
         self._delete(url)
 

kbcstorage/tables_metadata.py

@@ -80,6 +80,12 @@ def create(self, table_id, provider, metadata, columns_metadata):
         """
         if not isinstance(table_id, str) or table_id == '':
             raise ValueError("Invalid table_id '{}'.".format(table_id))
+        if not isinstance(provider, str) or provider == '':
+            raise ValueError("Invalid provider '{}'.".format(provider))
+        if not isinstance(metadata, list):
+            raise ValueError("Invalid metadata '{}'.".format(metadata))
+        if not isinstance(columns_metadata, dict):
+            raise ValueError("Invalid columns_metadata '{}'.".format(columns_metadata))
 
         url = '{}/{}/metadata'.format(self.base_url, table_id)
         headers = {