[v7] Remove Severity.Log and Severity.Critical to make it inline with spec

kamilogorek · kamilogorek · commit 6bbb39b68027 · 2021-03-25T11:46:34.000+01:00
diff --git a/packages/integration-node-breadcrumbs/src/console.ts b/packages/integration-node-breadcrumbs/src/console.ts
@@ -27,24 +27,7 @@ export class ConsoleBreadcrumbs implements Integration {
  */
 function createConsoleWrapper(level: string, client: ClientLike): (originalConsoleMethod: () => void) => void {
   return function consoleWrapper(originalConsoleMethod: () => void): () => void {
-    let sentryLevel: Severity;
-
-    switch (level) {
-      case 'debug':
-        sentryLevel = Severity.Debug;
-        break;
-      case 'error':
-        sentryLevel = Severity.Error;
-        break;
-      case 'info':
-        sentryLevel = Severity.Info;
-        break;
-      case 'warn':
-        sentryLevel = Severity.Warning;
-        break;
-      default:
-        sentryLevel = Severity.Log;
-    }
+    const sentryLevel = Severity.fromString(level);
 
     return function(this: typeof console, ...args: unknown[]): void {
       client.getScope().addBreadcrumb(
diff --git a/packages/types/src/severity.ts b/packages/types/src/severity.ts
@@ -3,10 +3,8 @@ export enum Severity {
   Fatal = 'fatal',
   Error = 'error',
   Warning = 'warning',
-  Log = 'log',
   Info = 'info',
   Debug = 'debug',
-  Critical = 'critical',
 }
 
 // eslint-disable-next-line @typescript-eslint/no-namespace, import/export
@@ -19,22 +17,18 @@ export namespace Severity {
    */
   export function fromString(level: string): Severity {
     switch (level) {
-      case 'debug':
-        return Severity.Debug;
-      case 'info':
-        return Severity.Info;
+      case 'fatal':
+        return Severity.Fatal;
       case 'warn':
       case 'warning':
         return Severity.Warning;
-      case 'error':
-        return Severity.Error;
-      case 'fatal':
-        return Severity.Fatal;
-      case 'critical':
-        return Severity.Critical;
       case 'log':
+      case 'info':
+        return Severity.Info;
+      case 'debug':
+        return Severity.Debug;
       default:
-        return Severity.Log;
+        return Severity.Error;
     }
   }
 }
