test(metrics): explicitly check for non-nil writer instead of crash testing

Signed-off-by: abhaygoudannavar <abhaysgoudnvr@gmail.com>

Author: abhaygoudannavar

internal/metrics/metrics_test.go

@@ -56,11 +56,11 @@ func TestZerologMetricsMetadata(t *testing.T) {
 func TestZerologMetricsInvalidFileDoesNotPanic(t *testing.T) {
 	// Provide a path to a directory that definitely does not exist
 	invalidPath := "/does/not/exist/timestamps.log"
-	
+
 	// Create the metrics writer with timestamps enabled but an invalid path
 	writer := NewZerologMetrics(true, invalidPath, "container-test")
-	
-	// This call would panic with a nil pointer dereference without the fix.
-	// With the fix, it will safely execute the mockWriter's no-op Capture().
-	writer.Capture(TS00)
+
+	if writer == nil {
+		t.Fatal("expected non-nil writer (fallback to mockWriter), got nil")
+	}
 }