From e2304396d82ad2019878921c8657217e3c5c27a8 Mon Sep 17 00:00:00 2001 From: Javier Segarra Date: Sun, 1 Dec 2024 19:18:55 +0100 Subject: [PATCH] fix: parsePhone fn when no phone --- src/filters/parsePhone.js | 3 +++ .../__tests__/components/common/VnLinkPhone.spec.js | 8 ++++++++ 2 files changed, 11 insertions(+) diff --git a/src/filters/parsePhone.js b/src/filters/parsePhone.js index 696f55007..cded224b9 100644 --- a/src/filters/parsePhone.js +++ b/src/filters/parsePhone.js @@ -1,4 +1,7 @@ export default function (phone, prefix = 34) { + if (!phone) { + return; + } if (phone.startsWith('+')) { return `${phone.slice(1)}`; } diff --git a/test/vitest/__tests__/components/common/VnLinkPhone.spec.js b/test/vitest/__tests__/components/common/VnLinkPhone.spec.js index e460ab2fc..e31bff4a8 100644 --- a/test/vitest/__tests__/components/common/VnLinkPhone.spec.js +++ b/test/vitest/__tests__/components/common/VnLinkPhone.spec.js @@ -2,6 +2,14 @@ import { describe, it, expect } from 'vitest'; import parsePhone from 'src/filters/parsePhone'; describe('parsePhone filter', () => { + it('no phone', () => { + const resultado = parsePhone(null, '34'); + expect(resultado).toBe(undefined); + }); + it('no phone and no prefix', () => { + const resultado = parsePhone(null, null); + expect(resultado).toBe(undefined); + }); it("adds prefix +34 if it doesn't have one", () => { const resultado = parsePhone('123456789', '34'); expect(resultado).toBe('34123456789');