kamash
/
dyzur-bot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

style: minor refinings

This commit is contained in:
Kamil 'Kamaś' Bruchal 2026-07-20 17:45:35 +02:00
parent 7f5bc8454a
commit 37a81a2710
2 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
scheduler.go
View File

@ -181,14 +181,14 @@ func reminderText(rota Rota, target time.Time) (string, bool) {
case 0: case 0:
body = "dyżur jest nieobsadzony" body = "dyżur jest nieobsadzony"
case 1: case 1:
body = "dyżur sprząta " + people[0] body = "dyżur ma " + people[0]
default: default:
body = "dyżur sprzątają " + strings.Join(people, " i ") body = "dyżur mają " + strings.Join(people, " i ")
} }
tail := "." tail := "."
if len(people) > 0 { if len(people) > 0 {
tail = ". Z góry dziękujemy!" tail = ". Dziękujemy!"
} }
return "Przypomnienie: w tygodniu od " + return "Przypomnienie: w tygodniu od " +
week.WeekStart.Format("2006-01-02") + " " + body + tail, true week.WeekStart.Format("2006-01-02") + " " + body + tail, true
@ -201,7 +201,7 @@ func assignedText(week DutyWeek, p1, p2 string, handles map[string]string) strin
return "Nikt nie zgłosił się na dyżur w tygodniu od " + return "Nikt nie zgłosił się na dyżur w tygodniu od " +
week.WeekStart.Format("2006-01-02") + ", więc został przydzielony losowo: " + week.WeekStart.Format("2006-01-02") + ", więc został przydzielony losowo: " +
mention(p1, handles) + " i " + mention(p2, handles) + mention(p1, handles) + " i " + mention(p2, handles) +
". Z góry dziękujemy za sprzątanie!" ". Dziękujemy za sprzątanie!"
} }
// weekWarningText asks people to volunteer for an as-yet-unassigned week before // weekWarningText asks people to volunteer for an as-yet-unassigned week before

14
scheduler_test.go
View File

@ -59,7 +59,7 @@ func TestFire(t *testing.T) {
return DutyWeek{WeekStart: day(t, start), Person1: "X", Person2: "Y", Row: row} return DutyWeek{WeekStart: day(t, start), Person1: "X", Person2: "Y", Row: row}
} }
thisWeekFilled := DutyWeek{WeekStart: day(t, "2026-06-15"), Person1: "Jan", Person2: "Anna", Row: 5} thisWeekFilled := DutyWeek{WeekStart: day(t, "2026-06-15"), Person1: "Jan", Person2: "Anna", Row: 5}
wantThisReminder := "Przypomnienie: w tygodniu od 2026-06-15 dyżur sprzątają Jan i Anna. Z góry dziękujemy!" wantThisReminder := "Przypomnienie: w tygodniu od 2026-06-15 dyżur mają Jan i Anna. Z góry dziękujemy!"
t.Run("this week empty -> reminder only, never assigns", func(t *testing.T) { t.Run("this week empty -> reminder only, never assigns", func(t *testing.T) {
rota := Rota{ rota := Rota{
@ -371,12 +371,12 @@ func TestReminderText(t *testing.T) {
{ {
"both people", "both people",
rota("Ala", "Bartek"), target, true, rota("Ala", "Bartek"), target, true,
"Przypomnienie: w tygodniu od 2026-06-29 dyżur sprzątają Ala i Bartek. Z góry dziękujemy!", "Przypomnienie: w tygodniu od 2026-06-29 dyżur mają Ala i Bartek. Z góry dziękujemy!",
}, },
{ {
"one person", "one person",
rota("Ala", ""), target, true, rota("Ala", ""), target, true,
"Przypomnienie: w tygodniu od 2026-06-29 dyżur sprząta Ala. Z góry dziękujemy!", "Przypomnienie: w tygodniu od 2026-06-29 dyżur ma Ala. Z góry dziękujemy!",
}, },
{ {
"nobody assigned", "nobody assigned",
@ -421,22 +421,22 @@ func TestReminderTextMentions(t *testing.T) {
{ {
"both have handles", "both have handles",
rota("Ala", "Bartek"), rota("Ala", "Bartek"),
"Przypomnienie: w tygodniu od 2026-06-29 dyżur sprzątają @ala i @bart. Z góry dziękujemy!", "Przypomnienie: w tygodniu od 2026-06-29 dyżur mają @ala i @bart. Z góry dziękujemy!",
}, },
{ {
"one handle, one missing -> mixed", "one handle, one missing -> mixed",
rota("Ala", "Cela"), rota("Ala", "Cela"),
"Przypomnienie: w tygodniu od 2026-06-29 dyżur sprzątają @ala i Cela. Z góry dziękujemy!", "Przypomnienie: w tygodniu od 2026-06-29 dyżur mają @ala i Cela. Z góry dziękujemy!",
}, },
{ {
"name not in people list -> plain name", "name not in people list -> plain name",
rota("Ala", "Zenon"), rota("Ala", "Zenon"),
"Przypomnienie: w tygodniu od 2026-06-29 dyżur sprzątają @ala i Zenon. Z góry dziękujemy!", "Przypomnienie: w tygodniu od 2026-06-29 dyżur mają @ala i Zenon. Z góry dziękujemy!",
}, },
{ {
"single person with handle", "single person with handle",
rota("Bartek", ""), rota("Bartek", ""),
"Przypomnienie: w tygodniu od 2026-06-29 dyżur sprząta @bart. Z góry dziękujemy!", "Przypomnienie: w tygodniu od 2026-06-29 dyżur ma @bart. Z góry dziękujemy!",
}, },
} }
for _, tc := range tests { for _, tc := range tests {