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

1 files changed, 8 insertions, 6 deletions

diff --git a/internal/media/util_test.go b/internal/media/util_test.go
index 9ba0f76ef..c54fb6f2c 100644
--- a/internal/media/util_test.go
+++ b/internal/media/util_test.go
@@ -19,16 +19,16 @@
 package media
 
 import (
+	"github.com/sirupsen/logrus"
+	"github.com/superseriousbusiness/gotosocial/internal/log"
 	"io/ioutil"
 	"testing"
 
-	"github.com/sirupsen/logrus"
 	"github.com/stretchr/testify/suite"
 )
 
 type MediaUtilTestSuite struct {
 	suite.Suite
-	log *logrus.Logger
 }
 
 /*
@@ -37,10 +37,12 @@ type MediaUtilTestSuite struct {
 
 // SetupSuite sets some variables on the suite that we can use as consts (more or less) throughout
 func (suite *MediaUtilTestSuite) SetupSuite() {
-	// some of our subsequent entities need a log so create this here
-	log := logrus.New()
-	log.SetLevel(logrus.TraceLevel)
-	suite.log = log
+	// doesn't use testrig.InitTestLog() helper to prevent import cycle
+	err := log.Initialize(logrus.TraceLevel.String())
+	if err != nil {
+		panic(err)
+	}
+
 }
 
 func (suite *MediaUtilTestSuite) TearDownSuite() {