diff --git a/dashboard/app/bisect_test.go b/dashboard/app/bisect_test.go index a88d20956f80..cb4a1c9e90de 100644 --- a/dashboard/app/bisect_test.go +++ b/dashboard/app/bisect_test.go @@ -195,7 +195,7 @@ For information about bisection process see: https://goo.gl/tpsmEJ#bisection "reviewer1@kernel.org", "reviewer2@kernel.org", }) - c.expectEQ(msg.Subject, "[syzbot] "+crash2.Title) + c.expectEQ(msg.Subject, "[syzbot] "+crash2.Title) // nolint: goconst c.expectEQ(msg.Body, fmt.Sprintf(`Hello, syzbot found the following issue on: @@ -319,7 +319,7 @@ If you want to undo deduplication, reply with: c.expectEQ(msg.Subject, subjects[i]) c.expectTrue(strings.Contains(msg.Body, "Sending this report to the next reporting stage.")) } else { - c.expectEQ(msg.Subject, "[syzbot] "+subjects[i]) + c.expectEQ(msg.Subject, "[syzbot] "+subjects[i]) // nolint: goconst c.expectTrue(strings.Contains(msg.Body, "syzbot found the following issue on")) } } diff --git a/dashboard/app/email_test.go b/dashboard/app/email_test.go index ae40d9e8a984..8da9ab9e8d9f 100644 --- a/dashboard/app/email_test.go +++ b/dashboard/app/email_test.go @@ -224,7 +224,7 @@ If you attach or paste a git patch, syzbot will apply it before testing. "foo@bar.com", "maintainers@repo10.org", }) - c.expectEQ(msg.Subject, "[syzbot] "+crash.Title) + c.expectEQ(msg.Subject, "[syzbot] "+crash.Title) // nolint: goconst c.expectEQ(len(msg.Attachments), 0) c.expectEQ(msg.Body, fmt.Sprintf(`Hello, @@ -314,7 +314,7 @@ Content-Type: text/plain "another@another.com", "bar@foo.com", "bugs@repo10.org", "bugs@syzkaller.com", "default@maintainers.com", "foo@bar.com", "maintainers@repo10.org", "new@new.com", "qux@qux.com"}) - c.expectEQ(msg.Subject, "Re: [syzbot] "+crash.Title) + c.expectEQ(msg.Subject, "Re: [syzbot] "+crash.Title) // nolint: goconst c.expectEQ(len(msg.Attachments), 0) c.expectEQ(msg.Body, fmt.Sprintf(`syzbot has found a reproducer for the following issue on: @@ -1323,7 +1323,7 @@ func expectLabels(t *testing.T, client *apiClient, extID string, labels ...strin var forwardEmailConfig = EmailConfig{ Email: "test@syzkaller.com", HandleListEmails: true, - SubjectPrefix: "[syzbot]", + SubjectPrefix: "[syzbot]", // nolint: goconst MailMaintainers: true, DefaultMaintainers: []string{"some@list.com"}, } diff --git a/dashboard/app/notifications_test.go b/dashboard/app/notifications_test.go index 4e680baba659..1ced5f86140d 100644 --- a/dashboard/app/notifications_test.go +++ b/dashboard/app/notifications_test.go @@ -38,7 +38,7 @@ func TestEmailNotifUpstreamEmbargo(t *testing.T) { c.expectEQ(notifUpstream.Subject, crash.Title) c.expectEQ(notifUpstream.Sender, report.Sender) c.expectEQ(notifUpstream.Body, "Sending this report to the next reporting stage.") - c.expectEQ(upstreamReport.Subject, "[syzbot] "+crash.Title) + c.expectEQ(upstreamReport.Subject, "[syzbot] "+crash.Title) // nolint: goconst c.expectNE(upstreamReport.Sender, report.Sender) c.expectEQ(upstreamReport.To, []string{"bugs@syzkaller.com", "default@maintainers.com"}) }