test: cover notification verification renewal and storage read recovery #24729

notificationStatus determines whether Settings and the notification-verification indicator treat browser notifications as confirmed. Its helper tests cover initial confirmation, a seeded expired record, invalid JSON and write failures. Two transitions would benefit from direct coverage:

• After an expired confirmation is renewed, verification stays valid for the new period and then expires relative to the renewed timestamp.
• When localStorage.getItem throws SecurityError, verification returns false; once reads recover, the saved confirmation is still available.

Proposed scope: add these two cases to web/src/lib/__tests__/notificationStatus.test.ts, with no production-source changes. Existing indicator tests mock the helper, so they do not cover these transitions.

Source inspection on October 31 at 30721fcd699ea103e2daeea1d4988823e3c60197 confirmed that the helper and its existing test file remain unchanged from the historical tested baseline.

Historical validation on October 27: a private patch was checked against 88bf46ed03e1bcad05ee4e96f79e2febe8929659 using a fresh npm ci installation from that commit's unchanged lockfile, with lifecycle scripts disabled. All 715 installed package entries matched that lockfile.

cd web
npm test -- --run src/lib/__tests__/notificationStatus.test.ts \
  --maxWorkers=1 --pool=forks --config vitest.task.config.ts

Historical result: 11 passed, 0 failed or skipped, including both proposed cases.

Validation scope: one focused helper test file in a trimmed source snapshot. The private task wrapper loaded the repository Vite config and test setup, redirected caches/reports, and disabled application-page warmup, filesystem test caching and retries. The wrapper is an execution aid outside the proposed patch. These results apply to the historical commit above. Project build/lint checks are handled by PR CI.

AI assistance: Codex identified the coverage gap, authored the two private tests, and ran the reported historical pinned-dependency checks. This proposal has not received maintainer scope approval.