fix: Move function to bottom and add test

Author: Matovidlo

internal/pkg/encoding/json/schema/schema.go

@@ -20,18 +20,6 @@ import (
 // pseudoSchemaFile - the validated schema is registered as this resource.
 const pseudoSchemaFile = "file:///schema.json"
 
-// skipSchemaValidation returns true for components where schema validation should be skipped.
-func skipSchemaValidation(componentID keboola.ComponentID) bool {
-	switch componentID {
-	case "keboola.python-transformation-v2",
-		"keboola.snowflake-transformation",
-		"keboola.google-bigquery-transformation":
-		return true
-	default:
-		return false
-	}
-}
-
 func ValidateObjects(ctx context.Context, logger log.Logger, objects model.ObjectStates) error {
 	errs := errors.NewMultiError()
 	for _, config := range objects.Configs() {
@@ -269,3 +257,15 @@ func compileSchema(s []byte, savePropertyOrder bool) (*jsonschema.Schema, error)
 
 	return schema, nil
 }
+
+// skipSchemaValidation returns true for components where schema validation should be skipped.
+func skipSchemaValidation(componentID keboola.ComponentID) bool {
+	switch componentID {
+	case "keboola.python-transformation-v2",
+		"keboola.snowflake-transformation",
+		"keboola.google-bigquery-transformation":
+		return true
+	default:
+		return false
+	}
+}

internal/pkg/encoding/json/schema/schema_test.go

@@ -384,6 +384,55 @@ func testInvalidComponentSchema(t *testing.T, invalidSchema []byte, expectedLogs
 	assert.Equal(t, strings.TrimLeft(expectedLogs, "\n"), logger.AllMessagesTxt())
 }
 
+func TestValidateConfig_SkipTransformationComponents(t *testing.T) {
+	t.Parallel()
+
+	// Schema that requires "firstName" field
+	schema := getTestSchema()
+
+	// Content that violates the schema (missing required "firstName")
+	invalidContent := orderedmap.FromPairs([]orderedmap.Pair{
+		{
+			Key: "parameters",
+			Value: orderedmap.FromPairs([]orderedmap.Pair{
+				{Key: "lastName", Value: "Brown"},
+			}),
+		},
+	})
+
+	// Test that validation is skipped for transformation components
+	transformationComponents := []keboola.ComponentID{
+		"keboola.python-transformation-v2",
+		"keboola.snowflake-transformation",
+		"keboola.google-bigquery-transformation",
+	}
+
+	for _, componentID := range transformationComponents {
+		component := &keboola.Component{
+			ComponentKey: keboola.ComponentKey{ID: componentID},
+			Type:         "transformation",
+			Name:         "Test Transformation",
+			Schema:       schema,
+		}
+		config := &model.Config{Content: invalidContent}
+
+		// Should return nil because validation is skipped for transformation components
+		err := ValidateConfig(component, config)
+		require.NoError(t, err, "validation should be skipped for component %s", componentID)
+	}
+
+	// Verify that a regular component still gets validated
+	regularComponent := &keboola.Component{
+		ComponentKey: keboola.ComponentKey{ID: "keboola.ex-generic"},
+		Type:         "extractor",
+		Name:         "Generic Extractor",
+		Schema:       schema,
+	}
+	config := &model.Config{Content: invalidContent}
+	err := ValidateConfig(regularComponent, config)
+	require.Error(t, err, "validation should NOT be skipped for regular component")
+}
+
 func TestNormalizeSchema_RequiredTrue(t *testing.T) {
 	t.Parallel()