chore: disallow inbox as default method (#17093)

Disallow setting `inbox` as the default notifications method.
This commit is contained in:
Danielle Maywood
2025-03-25 20:10:15 +00:00
committed by GitHub
parent cf10d98aab
commit c131d01cfd
2 changed files with 28 additions and 2 deletions

View File

@ -1856,6 +1856,18 @@ func TestNotificationDuplicates(t *testing.T) {
require.NoError(t, err)
}
func TestNotificationMethodCannotDefaultToInbox(t *testing.T) {
t.Parallel()
store, _ := dbtestutil.NewDB(t)
logger := testutil.Logger(t)
cfg := defaultNotificationsConfig(database.NotificationMethodInbox)
_, err := notifications.NewStoreEnqueuer(cfg, store, defaultHelpers(), logger.Named("enqueuer"), quartz.NewMock(t))
require.ErrorIs(t, err, notifications.InvalidDefaultNotificationMethodError{Method: string(database.NotificationMethodInbox)})
}
func TestNotificationTargetMatrix(t *testing.T) {
t.Parallel()