Change os.signal from syscall.Signal(21) to syscall.SIGINT for Windows

src/internal/events/events.go

@@ -186,7 +186,7 @@ func NewMetrics(ctx context.Context, w io.Writer) (context.Context, func()) {
 		// logging rates; that's handled by dumpMetrics().
 		sink = metrics.NewInmemSink(reportInterval, retentionDuration)
 		cfg  = metrics.DefaultConfig("corso")
-		sig  = metrics.NewInmemSignal(sink, metrics.DefaultSignal, w)
+		sig  = metrics.NewInmemSignal(sink, sentSignal, w)
 	)
 
 	cfg.EnableHostname = false

src/internal/events/events_signal_test.go

@@ -36,7 +36,7 @@ func (suite *EventsSignalUnitSuite) TestSignalDump() {
 
 	select {
 	case sig := <-sigCh:
-		assert.Equal(t, metrics.DefaultSignal, sig, "received wrong signal")
+		assert.Equal(t, sentSignal, sig, "received wrong signal")
 
 	case <-time.After(1 * time.Second):
 		assert.Fail(t, "timeout waiting for signal")

src/internal/events/events_signal_unix.go

@@ -13,6 +13,10 @@ import (
 	"github.com/alcionai/corso/src/pkg/logger"
 )
 
+const (
+	sentSignal = metrics.DefaultSignal
+)
+
 func signalDump(ctx context.Context) {
 	if err := unix.Kill(syscall.Getpid(), metrics.DefaultSignal); err != nil {
 		logger.CtxErr(ctx, err).Error("metrics interval signal")

src/internal/events/events_signal_windows.go

@@ -7,6 +7,12 @@ import (
 	"golang.org/x/sys/windows"
 )
 
+const (
+	// On Windows, ctrl-break event appears as os.Interrupt(syscall.SIGINT)
+	//https://pkg.go.dev/os/signal#hdr-Windows
+	sentSignal = os.Interrupt
+)
+
 func signalDump(ctx context.Context) {
 	err := windows.GenerateConsoleCtrlEvent(syscall.CTRL_BREAK_EVENT, uint32(syscall.Getpid()))
 	if err != nil {