Skip to content

Commit 061b2bf

Browse files
committed
make lint/fmt
Signed-off-by: Danny Kopping <danny@coder.com>
1 parent 4ba160d commit 061b2bf

File tree

2 files changed

+16
-15
lines changed

2 files changed

+16
-15
lines changed

coderd/notifications/notifications_test.go

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -748,12 +748,12 @@ func TestCustomNotificationMethod(t *testing.T) {
748748

749749
// GIVEN: a notification template which has a method explicitly set
750750
var (
751-
template = notifications.TemplateWorkspaceDormant
751+
template = notifications.TemplateWorkspaceDormant
752752
defaultMethod = database.NotificationMethodSmtp
753-
customMethod = database.NotificationMethodWebhook
753+
customMethod = database.NotificationMethodWebhook
754754
)
755755
out, err := db.UpdateNotificationTemplateMethodByID(ctx, database.UpdateNotificationTemplateMethodByIDParams{
756-
ID: template,
756+
ID: template,
757757
Method: database.NullNotificationMethod{NotificationMethod: customMethod, Valid: true},
758758
})
759759
require.NoError(t, err)
@@ -762,8 +762,8 @@ func TestCustomNotificationMethod(t *testing.T) {
762762
// GIVEN: a manager configured with multiple dispatch methods
763763
cfg := defaultNotificationsConfig(defaultMethod)
764764
cfg.SMTP = codersdk.NotificationsEmailConfig{
765-
From: "danny@coder.com",
766-
Hello: "localhost",
765+
From: "danny@coder.com",
766+
Hello: "localhost",
767767
Smarthost: serpent.HostPort{Host: "localhost", Port: fmt.Sprintf("%d", mockSMTPSrv.PortNumber())},
768768
}
769769
cfg.Webhook = codersdk.NotificationsWebhookConfig{
@@ -782,6 +782,7 @@ func TestCustomNotificationMethod(t *testing.T) {
782782
// WHEN: a notification of that template is enqueued, it should be delivered with the configured method - not the default.
783783
user := createSampleUser(t, db)
784784
msgID, err := enq.Enqueue(ctx, user.ID, template, map[string]string{}, "test")
785+
require.NoError(t, err)
785786

786787
// THEN: the notification should be received by the custom dispatch method
787788
mgr.Run(ctx)

coderd/notifications/notifier.go

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -33,21 +33,21 @@ type notifier struct {
3333
quit chan any
3434
done chan any
3535

36-
handlers map[database.NotificationMethod]Handler
36+
handlers map[database.NotificationMethod]Handler
3737
metrics *Metrics
3838
}
3939

4040
func newNotifier(cfg codersdk.NotificationsConfig, id uuid.UUID, log slog.Logger, db Store, hr map[database.NotificationMethod]Handler, metrics *Metrics) *notifier {
4141
return &notifier{
42-
id: id,
43-
cfg: cfg,
44-
log: log.Named("notifier").With(slog.F("notifier_id", id)),
45-
quit: make(chan any),
46-
done: make(chan any),
47-
tick: time.NewTicker(cfg.FetchInterval.Value()),
48-
store: db,
49-
handlers: hr,
50-
metrics: metrics,
42+
id: id,
43+
cfg: cfg,
44+
log: log.Named("notifier").With(slog.F("notifier_id", id)),
45+
quit: make(chan any),
46+
done: make(chan any),
47+
tick: time.NewTicker(cfg.FetchInterval.Value()),
48+
store: db,
49+
handlers: hr,
50+
metrics: metrics,
5151
}
5252
}
5353

0 commit comments

Comments
 (0)