From e1ff07fd9f6266122325b728255ce895eaa03084 Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Tue, 2 Dec 2025 13:45:49 +0000
Subject: [PATCH] test: Add singleAlertTemplate type coverage tests

Cover io type (formats as "I/O") and custom type (uses titleCase)
branches that were previously untested in the email template.
---
 internal/notifications/email_template_test.go | 34 +++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/internal/notifications/email_template_test.go b/internal/notifications/email_template_test.go
index 1dab690..5311477 100644
--- a/internal/notifications/email_template_test.go
+++ b/internal/notifications/email_template_test.go
@@ -632,6 +632,40 @@ func TestEmailTemplate(t *testing.T) {
 			expectSingleSubject: false,
 			subjectContains:     "2 Critical alerts",
 		},
+		{
+			name: "io type alert formats as I/O",
+			alerts: []*alerts.Alert{
+				{
+					ID:           "alert-io",
+					Level:        "warning",
+					Type:         "io",
+					ResourceName: "storage-pool",
+					Value:        150.0,
+					Threshold:    100.0,
+					StartTime:    time.Now(),
+				},
+			},
+			isSingle:            true,
+			expectSingleSubject: true,
+			subjectContains:     "I/O",
+		},
+		{
+			name: "custom type alert uses title case",
+			alerts: []*alerts.Alert{
+				{
+					ID:           "alert-custom",
+					Level:        "critical",
+					Type:         "network_latency",
+					ResourceName: "router-1",
+					Value:        500.0,
+					Threshold:    100.0,
+					StartTime:    time.Now(),
+				},
+			},
+			isSingle:            true,
+			expectSingleSubject: true,
+			subjectContains:     "router-1",
+		},
 	}
 
 	for _, tt := range tests {