fix gh_event handler (#74)

cpanato · web-flow · commit 9ee4f62a2a8f · 2023-03-10T15:10:29.000+01:00
Signed-off-by: cpanato &lt;ctadeu@gmail.com&gt;
diff --git a/internal/server/server.go b/internal/server/server.go
@@ -72,9 +72,9 @@ func NewServer(logger log.Logger, opts Opts) *Server {
 	}
 
 	muxMetrics.Handle(opts.MetricsPath, promhttp.Handler())
-	muxMetrics.HandleFunc(opts.WebhookPath, workflowExporter.HandleGHWebHook)
 
 	muxIngress.HandleFunc("/", server.handleRoot)
+	muxIngress.HandleFunc(opts.WebhookPath, workflowExporter.HandleGHWebHook)
 
 	return server
 }
diff --git a/internal/server/server_test.go b/internal/server/server_test.go
@@ -109,7 +109,7 @@ func Test_Server_MetricsRouteAfterWorkflowJob(t *testing.T) {
 			RunnerGroupName: &runnerGroupName,
 		},
 	}
-	req := testWebhookRequest(t, "http://localhost:8000/webhook", "workflow_job", event)
+	req := testWebhookRequest(t, "http://localhost:8001/webhook", "workflow_job", event)
 	res, err := http.DefaultClient.Do(req)
 	require.NoError(t, err)
 	defer res.Body.Close()

Original file line number	Diff line number	Diff line change
`@@ -72,9 +72,9 @@ func NewServer(logger log.Logger, opts Opts) *Server {`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`muxMetrics.Handle(opts.MetricsPath, promhttp.Handler())`
`75`		`- muxMetrics.HandleFunc(opts.WebhookPath, workflowExporter.HandleGHWebHook)`
`76`	`75`
`77`	`76`	`muxIngress.HandleFunc("/", server.handleRoot)`
	`77`	`+ muxIngress.HandleFunc(opts.WebhookPath, workflowExporter.HandleGHWebHook)`
`78`	`78`
`79`	`79`	`return server`
`80`	`80`	`}`
Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,7 @@ func Test_Server_MetricsRouteAfterWorkflowJob(t *testing.T) {`
`109`	`109`	`RunnerGroupName: &runnerGroupName,`
`110`	`110`	`},`
`111`	`111`	`}`
`112`		`- req := testWebhookRequest(t, "http://localhost:8000/webhook", "workflow_job", event)`
	`112`	`+ req := testWebhookRequest(t, "http://localhost:8001/webhook", "workflow_job", event)`
`113`	`113`	`res, err := http.DefaultClient.Do(req)`
`114`	`114`	`require.NoError(t, err)`
`115`	`115`	`defer res.Body.Close()`