chore: fix junit test

flanksource · Oct 5, 2023 · 89291f6 · 89291f6
1 parent 95e381e
commit 89291f6
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 41 deletions.
diff --git a/checks/metrics.go b/checks/metrics.go
@@ -41,24 +41,21 @@ func getOrAddPrometheusMetric(name, metricType string, labelNames []string) (pro
 }
 
 func getWithEnvironment(ctx *context.Context, r *pkg.CheckResult) *context.Context {
-	templateInput := map[string]any{
-		"result": r.Data,
-		"canary": map[string]any{
-			"name":      r.Canary.GetName(),
-			"namespace": r.Canary.GetNamespace(),
-			"labels":    r.Canary.GetLabels(),
-			"id":        r.Canary.GetPersistedID(),
-		},
-		"check": map[string]any{
-			"name":        r.Check.GetName(),
-			"id":          r.Canary.GetCheckID(r.Check.GetName()),
-			"description": r.Check.GetDescription(),
-			"labels":      r.Check.GetLabels(),
-			"endpoint":    r.Check.GetEndpoint(),
-			"duration":    time.Millisecond * time.Duration(r.GetDuration()),
-		},
+	r.Data["canary"] = map[string]any{
+		"name":      r.Canary.GetName(),
+		"namespace": r.Canary.GetNamespace(),
+		"labels":    r.Canary.GetLabels(),
+		"id":        r.Canary.GetPersistedID(),
 	}
-	return ctx.New(templateInput)
+	r.Data["check"] = map[string]any{
+		"name":        r.Check.GetName(),
+		"id":          r.Canary.GetCheckID(r.Check.GetName()),
+		"description": r.Check.GetDescription(),
+		"labels":      r.Check.GetLabels(),
+		"endpoint":    r.Check.GetEndpoint(),
+		"duration":    time.Millisecond * time.Duration(r.GetDuration()),
+	}
+	return ctx.New(r.Data)
 }
 
 func getLabels(ctx *context.Context, metric external.Metrics) (map[string]string, []string, error) {

diff --git a/fixtures/k8s/junit_pass.yaml b/fixtures/k8s/junit_pass.yaml
@@ -2,21 +2,18 @@ apiVersion: canaries.flanksource.com/v1
 kind: Canary
 metadata:
   name: junit-pass
+  labels:
+    part-of: canary-tools
 spec:
   interval: 120
-  owner: DBAdmin
   severity: high
   junit:
     - testResults: "/tmp/junit-results/"
       name: junit-pass
+      test:
+        expr: results.failed == 0 && results.passed > 0
       display:
-        template: |
-          ✅ {{.results.passed}} ❌ {{.results.failed}} in 🕑 {{.results.duration}}
-          {{  range $r := .results.suites}}
-          {{- if gt (conv.ToInt $r.failed)  0 }}
-            {{$r.name}} ✅ {{$r.passed}} ❌ {{$r.failed}} in 🕑 {{$r.duration}}
-          {{- end }}
-          {{- end }}
+        expr: "string(results.failed) + ' of ' +  string(results.passed)"
       spec:
         containers:
           - name: jes
@@ -25,26 +22,19 @@ spec:
       metrics:
         - name: junit_check_pass_count
           type: gauge
-          value: "result.results.passed"
+          value: results.passed
           labels:
-            - name: name
-              value: junit_check_pass_count
-            - name: check_name
-              valueExpr: check.name
+            - name: suite_name
+              valueExpr: results.suites[0].name
         - name: junit_check_failed_count
           type: gauge
-          value: "result.results.failed"
+          value: results.failed
           labels:
-            - name: name
-              value: junit_check_fail_count
-            - name: check_name
-              valueExpr: check.name
+            - name: part_of
+              valueExpr: canary.labels['part-of']
         - name: junit_check_duration_ms
           type: histogram
-          value: "check.duration"
+          value: results.duration * 1000.0
           labels:
-            - name: name
-              value: junit_check_duration_ms
-            - name: check_name
-              valueExpr: check.name
-
+            - name: suite_name
+              valueExpr: results.suites[0].name