From 3a9523c060a4aa3c08aa1a39a654c7886576ce18 Mon Sep 17 00:00:00 2001 From: HendrikThePendric Date: Wed, 14 Feb 2024 11:45:59 +0100 Subject: [PATCH] fix: remove >0 check so last array item can be removed too --- adapter/src/components/Alerts.js | 11 +++-------- adapter/src/components/__tests__/Alerts.test.js | 3 ++- 2 files changed, 5 insertions(+), 9 deletions(-) diff --git a/adapter/src/components/Alerts.js b/adapter/src/components/Alerts.js index 5679c4bc5..3b339576f 100644 --- a/adapter/src/components/Alerts.js +++ b/adapter/src/components/Alerts.js @@ -25,14 +25,9 @@ const Alerts = () => { ) useEffect(() => { - if (alertManagerAlerts.length > 0) { - setAlertStackAlerts((currentAlertStackAlerts) => - mergeAlertStackAlerts( - currentAlertStackAlerts, - alertManagerAlerts - ) - ) - } + setAlertStackAlerts((currentAlertStackAlerts) => + mergeAlertStackAlerts(currentAlertStackAlerts, alertManagerAlerts) + ) }, [alertManagerAlerts]) return ( diff --git a/adapter/src/components/__tests__/Alerts.test.js b/adapter/src/components/__tests__/Alerts.test.js index 845dfc0b2..72f36c22f 100644 --- a/adapter/src/components/__tests__/Alerts.test.js +++ b/adapter/src/components/__tests__/Alerts.test.js @@ -49,9 +49,10 @@ describe('Alerts', () => { it('can hide an alert', async () => { const msg = 'TEST ALERT' + const options = { permanent: true } render( - + )