From 083099a9575f8b2fac22c1d4a51a9dd0e2201243 Mon Sep 17 00:00:00 2001 From: "R. Aidan Campbell" Date: Mon, 11 Oct 2021 05:37:33 -0700 Subject: reference global logrus (#274) * reference logrus' global logger instead of passing and storing a logger reference everywhere * always directly use global logrus logger instead of referencing an instance * test suites should also directly use the global logrus logger * rename gin logging function to clarify that it's middleware * correct comments which erroneously referenced removed logger parameter * setting log level for tests now uses logrus' exported type instead of the string value, to guarantee error isn't possible --- testrig/mediahandler.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'testrig/mediahandler.go') diff --git a/testrig/mediahandler.go b/testrig/mediahandler.go index 3cf6f539a..9ff8179c0 100644 --- a/testrig/mediahandler.go +++ b/testrig/mediahandler.go @@ -24,8 +24,7 @@ import ( "github.com/superseriousbusiness/gotosocial/internal/media" ) -// NewTestMediaHandler returns a media handler with the default test config, the default test logger, -// and the given db and storage. +// NewTestMediaHandler returns a media handler with the default test config, and the given db and storage. func NewTestMediaHandler(db db.DB, storage *kv.KVStore) media.Handler { - return media.New(NewTestConfig(), db, storage, NewTestLog()) + return media.New(NewTestConfig(), db, storage) } -- cgit v1.2.3