Skip to content

Commit 867aded

Browse files
eamonnmoloneyclaude
andcommitted
test(8.9,8.10): update configmap test to assert no explicit camundaexporter registration
CamundaExporter is now auto-registered via autoconfigure-camunda-exporter: true; the legacy zeebe.broker.exporters.camundaexporter block was removed in this PR, so the test assertion is updated to verify the entry is absent. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent b10035c commit 867aded

2 files changed

Lines changed: 8 additions & 6 deletions

File tree

charts/camunda-platform-8.10/test/unit/orchestration/configmap_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -103,16 +103,17 @@ func TestGoldenConfigmapWithHistoryRetentionEnabled(t *testing.T) {
103103
func (s *ConfigmapLegacyTemplateTest) TestDifferentValuesInputs() {
104104
testCases := []testhelpers.TestCase{
105105
{
106-
Name: "TestContainerShouldContainExporterClassPerDefault",
106+
Name: "TestExportersShouldBeEmptyByDefault",
107107
Values: map[string]string{},
108108
Verifier: func(t *testing.T, output string, err error) {
109109
var configmap corev1.ConfigMap
110110
var configmapApplication camunda.OrchestrationApplicationYAML
111111
helm.UnmarshalK8SYaml(s.T(), output, &configmap)
112112
helm.UnmarshalK8SYaml(s.T(), configmap.Data["application.yaml"], &configmapApplication)
113113

114-
// then
115-
s.Require().Equal("io.camunda.exporter.CamundaExporter", configmapApplication.Zeebe.Broker.Exporters.CamundaExporter.ClassName)
114+
// CamundaExporter is auto-registered via autoconfigure-camunda-exporter: true;
115+
// the legacy zeebe.broker.exporters.camundaexporter entry must not be present.
116+
s.Require().Empty(configmapApplication.Zeebe.Broker.Exporters.CamundaExporter.ClassName)
116117
},
117118
},
118119
}

charts/camunda-platform-8.9/test/unit/orchestration/configmap_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -103,16 +103,17 @@ func TestGoldenConfigmapWithHistoryRetentionEnabled(t *testing.T) {
103103
func (s *ConfigmapLegacyTemplateTest) TestDifferentValuesInputs() {
104104
testCases := []testhelpers.TestCase{
105105
{
106-
Name: "TestContainerShouldContainExporterClassPerDefault",
106+
Name: "TestExportersShouldBeEmptyByDefault",
107107
Values: map[string]string{},
108108
Verifier: func(t *testing.T, output string, err error) {
109109
var configmap corev1.ConfigMap
110110
var configmapApplication camunda.OrchestrationApplicationYAML
111111
helm.UnmarshalK8SYaml(s.T(), output, &configmap)
112112
helm.UnmarshalK8SYaml(s.T(), configmap.Data["application.yaml"], &configmapApplication)
113113

114-
// then
115-
s.Require().Equal("io.camunda.exporter.CamundaExporter", configmapApplication.Zeebe.Broker.Exporters.CamundaExporter.ClassName)
114+
// CamundaExporter is auto-registered via autoconfigure-camunda-exporter: true;
115+
// the legacy zeebe.broker.exporters.camundaexporter entry must not be present.
116+
s.Require().Empty(configmapApplication.Zeebe.Broker.Exporters.CamundaExporter.ClassName)
116117
},
117118
},
118119
}

0 commit comments

Comments
 (0)