Revert "Revert "Print details on connector name mismatch error (#369)""

This reverts commit c228fef.

.gitignore

@@ -6,3 +6,4 @@
 pipelines/
 defaults/
 site/
+scratch*

kpops/components/base_components/kafka_connector.py

@@ -87,7 +87,7 @@ def connector_config_should_have_component_name(
         component_name = values["prefix"] + values["name"]
         connector_name: str | None = app.get("name")
         if connector_name is not None and connector_name != component_name:
-            msg = "Connector name should be the same as component name"
+            msg = f"Connector name '{connector_name}' should be the same as component name '{component_name}'"
             raise ValueError(msg)
         app["name"] = component_name
         return app

tests/components/test_kafka_connector.py

@@ -1,3 +1,4 @@
+import re
 from pathlib import Path
 from unittest.mock import AsyncMock, MagicMock
 
@@ -85,7 +86,10 @@ def test_connector_config_name_override(
         assert connector.app.name == CONNECTOR_FULL_NAME
 
         with pytest.raises(
-            ValueError, match="Connector name should be the same as component name"
+            ValueError,
+            match=re.escape(
+                f"Connector name 'different-name' should be the same as component name '{CONNECTOR_FULL_NAME}'"
+            ),
         ):
             KafkaConnector(
                 name=CONNECTOR_NAME,
@@ -96,7 +100,10 @@ def test_connector_config_name_override(
             )
 
         with pytest.raises(
-            ValueError, match="Connector name should be the same as component name"
+            ValueError,
+            match=re.escape(
+                f"Connector name '' should be the same as component name '{CONNECTOR_FULL_NAME}'"
+            ),
         ):
             KafkaConnector(
                 name=CONNECTOR_NAME,

tests/pipeline/resources/pipeline-with-env-defaults/defaults.yaml

@@ -1,5 +1,5 @@
 kubernetes-app:
-  name: "${component_type}"
+  name: ${component_type}
   namespace: example-namespace
 kafka-app:
   app:
@@ -24,7 +24,7 @@ streams-app: # inherits from kafka-app
           cleanup.policy: compact,delete
 
 kafka-connector:
-  name: "sink-connector"
+  name: sink-connector
   app:
     batch.size: "2000"
     behavior.on.malformed.documents: "warn"

tests/pipeline/resources/pipeline-with-env-defaults/defaults_development.yaml

@@ -1,3 +1,3 @@
 kubernetes-app:
-  name: "${component_type}-development"
+  name: ${component_type}-development
   namespace: development-namespace