From 0b0672d42a29add0feb323407813ba68d85b87e2 Mon Sep 17 00:00:00 2001 From: immanuwell Date: Wed, 27 May 2026 22:45:08 +0400 Subject: [PATCH] fix: handle 'none' severity in getSeverityValue and remove duplicate assignment in handleError --- src/cli/main.ts | 1 - src/lib/formatters/get-severity-value.ts | 2 +- test/jest/unit/lib/formatters/get-severity-value.spec.ts | 3 +++ 3 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/cli/main.ts b/src/cli/main.ts index dbec95ca3e..f0739d9055 100755 --- a/src/cli/main.ts +++ b/src/cli/main.ts @@ -99,7 +99,6 @@ async function handleError(args, error) { error.userMessage = error.nestedUserMessage || error.userMessage; error.code = error.nestedCode || error.code; error.strCode = error.nestedStrCode || error.strCode; - error.userMessage = error.nestedUserMessage || error.userMessage; } let command = 'bad-command'; diff --git a/src/lib/formatters/get-severity-value.ts b/src/lib/formatters/get-severity-value.ts index 8dcc9afc4f..dfd6822dcb 100644 --- a/src/lib/formatters/get-severity-value.ts +++ b/src/lib/formatters/get-severity-value.ts @@ -1,5 +1,5 @@ import { SEVERITIES, SEVERITY } from '../snyk-test/common'; export function getSeverityValue(severity: SEVERITY | 'none'): number { - return SEVERITIES.find((s) => s.verboseName === severity)!.value; + return SEVERITIES.find((s) => s.verboseName === severity)?.value ?? 0; } diff --git a/test/jest/unit/lib/formatters/get-severity-value.spec.ts b/test/jest/unit/lib/formatters/get-severity-value.spec.ts index b641039647..44bb9bf766 100644 --- a/test/jest/unit/lib/formatters/get-severity-value.spec.ts +++ b/test/jest/unit/lib/formatters/get-severity-value.spec.ts @@ -14,4 +14,7 @@ describe('getSeverityValue', () => { it('Low returns 4', () => { expect(getSeverityValue(SEVERITY.LOW)).toEqual(1); }); + it('none returns 0', () => { + expect(getSeverityValue('none')).toEqual(0); + }); });