Fix webhook tests and reformat payload info

Co-authored-by: Till-André Diegeler<[email protected]> Pushed-by: Till-André Diegeler<[email protected]> Signed-off-by: Till-André Diegeler<[email protected]>
scm-manager · Nov 28, 2024 · b828dab · b828dab
1 parent 18acb89
commit b828dab
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 9 deletions.
diff --git a/testdata/webhook/scm-webhook-data.json b/testdata/webhook/scm-webhook-data.json
@@ -1,7 +1,7 @@
 {
-  "HTMLURL": "github.com/scm-manager/argocd-test",
-  "Branch": {
-    "Name": "testrepo",
-    "DefaultBranch": true
+  "sourceUrl": "https://scm-manager.org/scm/scm-manager/argocd-test",
+  "branch": {
+    "name": "develop",
+    "defaultBranch": true
   }
 }
diff --git a/webhook_payloads.go b/webhook_payloads.go
@@ -1,6 +1,6 @@
 package goscm
 
 type PushEventPayload struct {
-	HTMLURL string `json:"HTMLURL"`
-	Branch  Branch `json:"branch"`
+	SourceUrl string `json:"sourceUrl"`
+	Branch    Branch `json:"branch"`
 }
diff --git a/webhook_test.go b/webhook_test.go
@@ -2,6 +2,7 @@ package goscm
 
 import (
 	"bytes"
+	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 	"io"
 	"net/http"
@@ -27,7 +28,7 @@ func TestWebhook_Parse(t *testing.T) {
 	plPush, err := hook.Parse(req, PushEvent)
 	require.NoError(t, err, "Error while parsing hook information in Goscm.")
 
-	UNUSED(plPush)
+	assert.Equal(t, "https://scm-manager.org/scm/scm-manager/argocd-test", plPush.(PushEventPayload).SourceUrl)
+	assert.Equal(t, "develop", plPush.(PushEventPayload).Branch.Name)
+	assert.True(t, plPush.(PushEventPayload).Branch.DefaultBranch)
 }
-
-func UNUSED(x ...interface{}) {}