Fix Test_GetIssue mock data - add missing User field

martinnormark · martinnormark · commit a0e757af01b1 · 2025-06-19T19:35:31.000+02:00
The assertion was already checking User.Login but the mock was incomplete
diff --git a/pkg/github/issues_test.go b/pkg/github/issues_test.go
@@ -38,6 +38,9 @@ func Test_GetIssue(t *testing.T) {
 		Body:    github.Ptr("This is a test issue"),
 		State:   github.Ptr("open"),
 		HTMLURL: github.Ptr("https://github.com/owner/repo/issues/42"),
+		User: &github.User{
+			Login: github.Ptr("testuser"),
+		},
 	}
 
 	tests := []struct {

Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,9 @@ func Test_GetIssue(t *testing.T) {`
`38`	`38`	`Body: github.Ptr("This is a test issue"),`
`39`	`39`	`State: github.Ptr("open"),`
`40`	`40`	`HTMLURL: github.Ptr("https://github.com/owner/repo/issues/42"),`
	`41`	`+ User: &github.User{`
	`42`	`+ Login: github.Ptr("testuser"),`
	`43`	`+ },`
`41`	`44`	`}`
`42`	`45`
`43`	`46`	`tests := []struct {`