diff options
Diffstat (limited to 'app/notification_test.go')
-rw-r--r-- | app/notification_test.go | 52 |
1 files changed, 48 insertions, 4 deletions
diff --git a/app/notification_test.go b/app/notification_test.go index 1a4fb2859..fb8eef4d7 100644 --- a/app/notification_test.go +++ b/app/notification_test.go @@ -148,14 +148,58 @@ func TestGetExplicitMentions(t *testing.T) { OtherPotentialMentions: []string{"user"}, }, }, - "OnePersonWithColonAtEnd": { + "OnePersonWithPeriodAfter": { + Message: "this is a message for @user.", + Keywords: map[string][]string{"@user": {id1}}, + Expected: &ExplicitMentions{ + MentionedUserIds: map[string]bool{ + id1: true, + }, + }, + }, + "OnePersonWithPeriodBefore": { + Message: "this is a message for .@user", + Keywords: map[string][]string{"@user": {id1}}, + Expected: &ExplicitMentions{ + MentionedUserIds: map[string]bool{ + id1: true, + }, + }, + }, + "OnePersonWithColonAfter": { Message: "this is a message for @user:", - Keywords: map[string][]string{"this": {id1}}, + Keywords: map[string][]string{"@user": {id1}}, + Expected: &ExplicitMentions{ + MentionedUserIds: map[string]bool{ + id1: true, + }, + }, + }, + "OnePersonWithColonBefore": { + Message: "this is a message for :@user", + Keywords: map[string][]string{"@user": {id1}}, + Expected: &ExplicitMentions{ + MentionedUserIds: map[string]bool{ + id1: true, + }, + }, + }, + "OnePersonWithHyphenAfter": { + Message: "this is a message for @user.", + Keywords: map[string][]string{"@user": {id1}}, + Expected: &ExplicitMentions{ + MentionedUserIds: map[string]bool{ + id1: true, + }, + }, + }, + "OnePersonWithHyphenBefore": { + Message: "this is a message for -@user", + Keywords: map[string][]string{"@user": {id1}}, Expected: &ExplicitMentions{ MentionedUserIds: map[string]bool{ id1: true, }, - OtherPotentialMentions: []string{"user"}, }, }, "MultiplePeopleWithOneWord": { @@ -493,7 +537,7 @@ func TestGetExplicitMentionsAtHere(t *testing.T) { "(@here(": true, ")@here)": true, "-@here-": true, - "_@here_": false, // This case shouldn't mention since it would be mentioning "@here_" + "_@here_": true, "=@here=": true, "+@here+": true, "[@here[": true, |