medplum / medplum / 31202477866

92%

push

github

fix(provider): dismiss DoseSpot sync toast when leaving Meds tab (#10123)

The Meds page fires the "Syncing with DoseSpot" loading toast on mount and
runs three sync bots. The effect cleanup only set a `cancelled` flag, so the
success/error handlers skipped `updateNotification`/`hideNotification` after
unmount. Navigating away before the bots resolved left the persistent
(autoClose: false) loading toast on screen forever.

Track an in-flight `syncing` flag and dismiss the toast in the effect cleanup
when sync is still running, so leaving the tab clears the loader while a
completed success toast is left untouched.

Also clean the module-level Mantine notification store between tests so toasts
from one test no longer leak into the next.

Signed-off-by: finnbergquist <finnyb38@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

23310 of 26428 branches covered (88.2%)

Branch coverage included in aggregate %.

41339 of 43762 relevant lines covered (94.46%)

12043.25 hits per line