chore: Remove IgnoreErrors check for test logger (#109)

scsmithr · web-flow · commit 26f2029e8e9a · 2021-06-09T14:37:40.000-05:00
diff --git a/sloggers/slogtest/t.go b/sloggers/slogtest/t.go
@@ -61,9 +61,6 @@ func (ts testSink) LogEntry(ctx context.Context, ent slog.SinkEntry) {
 			ts.tb.Error(s)
 		}
 	case slog.LevelFatal:
-		if ts.opts.IgnoreErrors {
-			panic("slogtest: cannot fatal in tests when IgnoreErrors option is set")
-		}
 		ts.tb.Fatal(s)
 	}
 }
diff --git a/sloggers/slogtest/t_test.go b/sloggers/slogtest/t_test.go
@@ -39,7 +39,7 @@ func TestIgnoreErrors(t *testing.T) {
 
 	defer func() {
 		recover()
-		assert.Equal(t, "fatals", 0, tb.fatals)
+		assert.Equal(t, "fatals", 1, tb.fatals)
 	}()
 
 	l.Fatal(bg, "hello")

Original file line number	Diff line number	Diff line change
`@@ -61,9 +61,6 @@ func (ts testSink) LogEntry(ctx context.Context, ent slog.SinkEntry) {`
`61`	`61`	`ts.tb.Error(s)`
`62`	`62`	`}`
`63`	`63`	`case slog.LevelFatal:`
`64`		`- if ts.opts.IgnoreErrors {`
`65`		`- panic("slogtest: cannot fatal in tests when IgnoreErrors option is set")`
`66`		`- }`
`67`	`64`	`ts.tb.Fatal(s)`
`68`	`65`	`}`
`69`	`66`	`}`