fix(unit): use global logrus logging instance

Jeff Peeler · Jeff Peeler · commit 1902e0b34329 · 2018-12-10T15:46:30.000-05:00
This allows log level changes in the unit tests to be respected. Also,
change operator groups default level to debug.
diff --git a/pkg/controller/operators/olm/operator_test.go b/pkg/controller/operators/olm/operator_test.go
@@ -131,7 +131,7 @@ func NewFakeOperator(clientObjs []runtime.Object, k8sObjs []runtime.Object, extO
 	}
 
 	// Create the new operator
-	queueOperator, err := queueinformer.NewOperatorFromClient(opClientFake, logrus.New())
+	queueOperator, err := queueinformer.NewOperatorFromClient(opClientFake, logrus.StandardLogger())
 	op := &Operator{
 		Operator:  queueOperator,
 		client:    clientFake,
@@ -2063,6 +2063,7 @@ func TestTransitionCSV(t *testing.T) {
 }
 
 func TestSyncOperatorGroups(t *testing.T) {
+	logrus.SetLevel(logrus.DebugLevel)
 	nowTime := metav1.Date(2006, time.January, 2, 15, 4, 5, 0, time.FixedZone("MST", -7*3600))
 	timeNow = func() metav1.Time { return nowTime }
 

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ func NewFakeOperator(clientObjs []runtime.Object, k8sObjs []runtime.Object, extO`
`131`	`131`	`}`
`132`	`132`
`133`	`133`	`// Create the new operator`
`134`		`- queueOperator, err := queueinformer.NewOperatorFromClient(opClientFake, logrus.New())`
	`134`	`+ queueOperator, err := queueinformer.NewOperatorFromClient(opClientFake, logrus.StandardLogger())`
`135`	`135`	`op := &Operator{`
`136`	`136`	`Operator: queueOperator,`
`137`	`137`	`client: clientFake,`
`@@ -2063,6 +2063,7 @@ func TestTransitionCSV(t *testing.T) {`
`2063`	`2063`	`}`
`2064`	`2064`
`2065`	`2065`	`func TestSyncOperatorGroups(t *testing.T) {`
	`2066`	`+ logrus.SetLevel(logrus.DebugLevel)`
`2066`	`2067`	`nowTime := metav1.Date(2006, time.January, 2, 15, 4, 5, 0, time.FixedZone("MST", -7*3600))`
`2067`	`2068`	`timeNow = func() metav1.Time { return nowTime }`
`2068`	`2069`