Pascal-cased Helm values do not work

[rassie]

Description of the bug:

When a chart is using Pascal-cased values keys, cdk8s generates a camel-cased property which does not work when applied to the chart.

Reproduction Steps:

• Import helm:https://SonarSource.github.io/helm-chart-sonarqube/[email protected]+3598
• Compare YAML output of two following chart declarations. You'll notice that OpenShift options described in the Helm chart do not activate when cdk8s-generated type declaration is used.

Non-working:

        new Sonarqube(this, "sonarqube", {
            values: {
                replicaCount: SonarqubeReplicaCount.VALUE_1,
                openShift: {
                    additionalValues: {
                        enabled: true,
                    },
                },
            },
        });

Working:

        new Sonarqube(this, "sonarqube", {
            values: {
                replicaCount: SonarqubeReplicaCount.VALUE_1,
                additionalValues: {
                    OpenShift: {
                        enabled: true,
                    },
                },
            },
        });

NB: I've not tested this extensively, so that root cause can be different, including my own stupidity.

---

This is 🐛 Bug Report

[bdoyle0182]

seems valid issue to me after taking a quick glance at the chart generated import in typescript. If the generated schema value variable names don't map identically you're going to have to specify them in additionalValues manually. If doing a helm import, it probably doesn't make sense to do any variable name mutation to a case type; or store a mapping of the variable name to the actual helm key string to keep things pretty and use that mapping on generation?

[iliapolo]

Legit. We should be able to store original casing and encode it within the toJson_XXX functions of the generated code. (i'm kind of surprised it doesn't already do that...need to check why)