Fix Test_GetIssue mock data - add missing User field

martinnormark · martinnormark · commit cd9c1e490848 · 2025-06-04T00:02:48.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
@@ -36,6 +36,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
`@@ -36,6 +36,9 @@ func Test_GetIssue(t *testing.T) {`
`36`	`36`	`Body: github.Ptr("This is a test issue"),`
`37`	`37`	`State: github.Ptr("open"),`
`38`	`38`	`HTMLURL: github.Ptr("https://github.com/owner/repo/issues/42"),`
	`39`	`+ User: &github.User{`
	`40`	`+ Login: github.Ptr("testuser"),`
	`41`	`+ },`
`39`	`42`	`}`
`40`	`43`
`41`	`44`	`tests := []struct {`