From 77d2fc04a74f7ff4ae069412c535fb51a80fde22 Mon Sep 17 00:00:00 2001
From: Nathan L Smith <nathan.smith@elastic.co>
Date: Tue, 12 May 2020 09:05:45 -0500
Subject: [PATCH] Fix major severity service map ring colors (#66124) (#66143)

Using `||` in the switch statement made it so "major" did not work, and we would get the border width but not the color. Change to splitting case statements.

Fixes #66081
---
 .../apm/public/components/app/ServiceMap/cytoscapeOptions.ts   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/x-pack/plugins/apm/public/components/app/ServiceMap/cytoscapeOptions.ts b/x-pack/plugins/apm/public/components/app/ServiceMap/cytoscapeOptions.ts
index 0cdc7c4eb124d..d3c4654de8164 100644
--- a/x-pack/plugins/apm/public/components/app/ServiceMap/cytoscapeOptions.ts
+++ b/x-pack/plugins/apm/public/components/app/ServiceMap/cytoscapeOptions.ts
@@ -17,7 +17,8 @@ export const getSeverityColor = (nodeSeverity: string) => {
   switch (nodeSeverity) {
     case severity.warning:
       return theme.euiColorVis0;
-    case severity.minor || severity.major:
+    case severity.minor:
+    case severity.major:
       return theme.euiColorVis5;
     case severity.critical:
       return theme.euiColorVis9;