From 4c45a42170b419cf42428353bc622802b32a685c Mon Sep 17 00:00:00 2001 From: Rishi Date: Sat, 11 Mar 2023 14:19:21 +0530 Subject: [PATCH] Set reportFormat to lower before generating report Fixes issue #449 When reportFormat in `kuttl-test.yaml` is specified in uppercase, it is passed as it is to report.Report func in harness.go. It needs to be passed as lowercase so that the `switch` compares against valid report format types (`ftype`) --- pkg/test/harness.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pkg/test/harness.go b/pkg/test/harness.go index b221a5bf..4edcf957 100644 --- a/pkg/test/harness.go +++ b/pkg/test/harness.go @@ -605,7 +605,9 @@ func (h *Harness) Report() { if len(h.TestSuite.ReportFormat) == 0 { return } - if err := h.report.Report(h.TestSuite.ArtifactsDir, h.reportName(), report.Type(h.TestSuite.ReportFormat)); err != nil { + + reportType := report.Type(strings.ToLower(h.TestSuite.ReportFormat)) + if err := h.report.Report(h.TestSuite.ArtifactsDir, h.reportName(), reportType); err != nil { h.fatal(fmt.Errorf("fatal error writing report: %v", err)) } }