Log failed summary builds

[onechat] / schedule_test.go

diff --git a/schedule_test.go b/schedule_test.go
index 85e11d09c12d0ca7786ab0d55b39b3b45485290d..96063aacae7611b4d8655473c8cff7b6ef3b2bea 100644 (file)
--- a/schedule_test.go
+++ b/schedule_test.go
@@ -2,7 +2,6 @@ package main
 
 import (
        "context"
 
 import (
        "context"

-       "io"

        "log"
        "os"
        "path/filepath"
        "log"
        "os"
        "path/filepath"


@@ -130,7 +129,7 @@ func TestSchedulesSurviveARestart(t *testing.T) {
        require.NoError(t, err)
        require.Equal(t, os.FileMode(0o600), info.Mode().Perm())
 
        require.NoError(t, err)
        require.Equal(t, os.FileMode(0o600), info.Mode().Perm())
 

-       reopened, err := openSchedules(path, schedules.clock, log.New(io.Discard, "", 0))
+       reopened, err := openSchedules(path, schedules.clock, discardLog())

        require.NoError(t, err)
        require.Equal(t, "2 | Monday 2026-11-09 08:00 -03:00 | 0 8 * * 1-5 | Tell the user the news.", reopened.List())
 
        require.NoError(t, err)
        require.Equal(t, "2 | Monday 2026-11-09 08:00 -03:00 | 0 8 * * 1-5 | Tell the user the news.", reopened.List())
 


@@ -232,7 +231,7 @@ func TestOpenSchedulesRefusesABadCron(t *testing.T) {
        path := filepath.Join(t.TempDir(), scheduleFileName)
        require.NoError(t, os.WriteFile(path, []byte(`{"next":1,"schedules":[{"id":1,"text":"x","cron":"* * * * *","due":"2026-11-09T09:00:00-03:00"}]}`), 0o600))
 
        path := filepath.Join(t.TempDir(), scheduleFileName)
        require.NoError(t, os.WriteFile(path, []byte(`{"next":1,"schedules":[{"id":1,"text":"x","cron":"* * * * *","due":"2026-11-09T09:00:00-03:00"}]}`), 0o600))
 

-       _, err := openSchedules(path, time.Now, log.New(io.Discard, "", 0))
+       _, err := openSchedules(path, time.Now, discardLog())

        require.ErrorContains(t, err, "schedule 1: the runs of")
 }
 
        require.ErrorContains(t, err, "schedule 1: the runs of")
 }
 


@@ -293,7 +292,7 @@ func TestDaemonRunsADueSchedule(t *testing.T) {
                now: time.Date(2026, time.November, 8, 20, 0, 0, 0, time.FixedZone("", -3*60*60)),
        }
 
                now: time.Date(2026, time.November, 8, 20, 0, 0, 0, time.FixedZone("", -3*60*60)),
        }
 

-       schedules, err := openSchedules(filepath.Join(t.TempDir(), scheduleFileName), clock.Now, log.New(io.Discard, "", 0))
+       schedules, err := openSchedules(filepath.Join(t.TempDir(), scheduleFileName), clock.Now, discardLog())

        require.NoError(t, err)
 
        schedules.check = time.Millisecond
        require.NoError(t, err)
 
        schedules.check = time.Millisecond


@@ -360,7 +359,7 @@ func scheduleTestOpen(t *testing.T) (*Schedules, string) {
                now: time.Date(2026, time.November, 8, 20, 0, 0, 0, time.FixedZone("", -3*60*60)),
        }
 
                now: time.Date(2026, time.November, 8, 20, 0, 0, 0, time.FixedZone("", -3*60*60)),
        }
 

-       schedules, err := openSchedules(path, clock.Now, log.New(io.Discard, "", 0))
+       schedules, err := openSchedules(path, clock.Now, discardLog())

        require.NoError(t, err)
 
        return schedules, path
        require.NoError(t, err)
 
        return schedules, path