refactor: http logger option

snyk · Apr 18, 2024 · 9ae2ace · 9ae2ace
1 parent 584bc26
commit 9ae2ace
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/http/http.go b/http/http.go
@@ -66,6 +66,7 @@ func WithErrorReporter(errorReporter observability.ErrorReporter) OptionFunc {
 func WithLogger(logger *zerolog.Logger) OptionFunc {
 	return func(h *httpClient) {
 		h.logger = logger
+		h.errorReporter = observability.NewErrorReporter(logger)
 	}
 }
 

diff --git a/internal/analysis/analysis_test.go b/internal/analysis/analysis_test.go
@@ -209,7 +209,7 @@ func TestAnalysis_CreateWorkspace_KnownErrors(t *testing.T) {
 
 			logger := zerolog.Nop()
 
-			analysisOrchestrator := analysis.NewAnalysisOrchestrator(&logger, mockHTTPClient, mockInstrumentor, mockErrorReporter, mockConfig)
+			analysisOrchestrator := analysis.NewAnalysisOrchestrator(mockConfig, &logger, mockHTTPClient, mockInstrumentor, mockErrorReporter)
 			_, err := analysisOrchestrator.CreateWorkspace(
 				context.Background(),
 				"4a72d1db-b465-4764-99e1-ecedad03b06a",