9996197633

Committed 18 Jul 2024 05:44PM UTC coverage: 97.423% (-0.3%) from 97.682%

Build # 9996197633

Build Type

Pull #423

github

Committed by

GermanSaracca

Commit Message

feat: refactor alerts management

Pull Request Pull Request #423: 356 publish dataset

Run Details

983 of 1025 branches covered (95.9%)

Branch coverage included in aggregate %.

115 of 124 new or added lines in 18 files covered. (92.74%)

3 existing lines in 2 files now uncovered.

2646 of 2700 relevant lines covered (98.0%)

8390.31 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

50.0

/src/sections/alerts/AlertProvider.tsx

import { PropsWithChildren, useState } from 'react'
import { AlertContext } from './AlertContext'

import { Alert, AlertMessageKey } from '../../alert/domain/models/Alert'

export const AlertProvider = ({ children }: PropsWithChildren) => {
  const [alerts, setAlerts] = useState<Alert[]>([])

  const addAlert = (newAlert: Alert) => {
    setAlerts((prevAlerts) => {
      const alertExists = prevAlerts.some((alert) => alert.messageKey === newAlert.messageKey)
      if (!alertExists) return [...prevAlerts, newAlert]
      return prevAlerts
    })
  }

  const removeAlert = (alertId: AlertMessageKey) => {
    setAlerts((prevAlerts) => prevAlerts.filter((alert) => alert.messageKey !== alertId))
  }

  return (
    <AlertContext.Provider
      value={{
        alerts,
        addAlert,
        removeAlert,
        setAlerts
      }}>
      {children}
    </AlertContext.Provider>
  )
}

1	import { PropsWithChildren, useState } from 'react'
2	import { AlertContext } from './AlertContext'
3
4	import { Alert, AlertMessageKey } from '../../alert/domain/models/Alert'
5
6	export const AlertProvider = ({ children }: PropsWithChildren) => {	289✔
7	const [alerts, setAlerts] = useState<Alert[]>([])	289✔
8
9	const addAlert = (newAlert: Alert) => {	289✔
NEW 10	setAlerts((prevAlerts) => {	×
NEW 11	const alertExists = prevAlerts.some((alert) => alert.messageKey === newAlert.messageKey)	×
NEW 12	if (!alertExists) return [...prevAlerts, newAlert]	×
NEW 13	return prevAlerts	×
14	})
15	}
16
17	const removeAlert = (alertId: AlertMessageKey) => {	289✔
NEW 18	setAlerts((prevAlerts) => prevAlerts.filter((alert) => alert.messageKey !== alertId))	×
19	}
20
21	return (	289✔
22	<AlertContext.Provider
23	value={{
24	alerts,
25	addAlert,
26	removeAlert,
27	setAlerts
28	}}>
29	{children}
30	</AlertContext.Provider>
31	)
32	}	21✔

IQSS / dataverse-frontend / 9996197633

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous