fix: update test case

Author: e7217

github/github-accessors.go

@@ -107,6 +107,7 @@ func TestString(t *testing.T) {
 		{Hook{ID: Ptr(int64(1))}, `github.Hook{ID:1}`},
 		{IssueComment{ID: Ptr(int64(1))}, `github.IssueComment{ID:1}`},
 		{Issue{Number: Ptr(1)}, `github.Issue{Number:1}`},
+		{SubIssue{ID: Ptr(int64(1))}, `github.SubIssue{ID:1}`},
 		{Key{ID: Ptr(int64(1))}, `github.Key{ID:1}`},
 		{Label{ID: Ptr(int64(1)), Name: Ptr("l")}, `github.Label{ID:1, Name:"l"}`},
 		{Organization{ID: Ptr(int64(1))}, `github.Organization{ID:1}`},

github/github-accessors_test.go

@@ -38,18 +38,18 @@ type SubIssueListByIssueOptions struct {
 
 // SubIssueRequest represents a request to add, remove, or reprioritize sub-issues.
 type SubIssueRequest struct {
-	SubIssueID    int64  `json:"sub_issue_id"`   // Required: The ID of the sub-issue
-	AfterID       *int64  `json:"after_id,omitempty"`       // Optional: Position after this sub-issue ID
-	BeforeID      *int64  `json:"before_id,omitempty"`      // Optional: Position before this sub-issue ID
-	ReplaceParent *bool `json:"replace_parent,omitempty"` // Optional: Whether to replace the existing parent
+	SubIssueID    int64  `json:"sub_issue_id"`             // Required: The ID of the sub-issue
+	AfterID       *int64 `json:"after_id,omitempty"`       // Optional: Position after this sub-issue ID
+	BeforeID      *int64 `json:"before_id,omitempty"`      // Optional: Position before this sub-issue ID
+	ReplaceParent *bool  `json:"replace_parent,omitempty"` // Optional: Whether to replace the existing parent
 }
 
 // Remove a sub-issue from the specified repository.
 //
 // GitHub API docs: https://docs.github.com/rest/issues/sub-issues#remove-sub-issue
 //
 //meta:operation DELETE /repos/{owner}/{repo}/issues/{issue_number}/sub_issue
-func (s *SubIssueService) Remove(ctx context.Context, owner, repo string, subIssueNumber int64, subIssue SubIssueRequest) (*SubIssue, *Response, error) {
+func (s *SubIssueService) Remove(ctx context.Context, owner, repo string, subIssueNumber int64, subIssue *SubIssueRequest) (*SubIssue, *Response, error) {
 	u := fmt.Sprintf("repos/%v/%v/issues/%v/sub_issues", owner, repo, subIssueNumber)
 
 	req, err := s.client.NewRequest("DELETE", u, subIssue)
@@ -129,7 +129,7 @@ func (s *SubIssueService) Add(ctx context.Context, owner string, repo string, is
 // GitHub API docs: https://docs.github.com/rest/issues/sub-issues#reprioritize-sub-issue
 //
 //meta:operation PATCH /repos/{owner}/{repo}/issues/{issue_number}/sub_issues/priority
-func (s *SubIssueService) Reprioritize(ctx context.Context, owner, repo string, issueNumber int64, subIssue SubIssueRequest) (*SubIssue, *Response, error) {
+func (s *SubIssueService) Reprioritize(ctx context.Context, owner, repo string, issueNumber int64, subIssue *SubIssueRequest) (*SubIssue, *Response, error) {
 	u := fmt.Sprintf("repos/%v/%v/issues/%d/sub_issues/priority", owner, repo, issueNumber)
 	req, err := s.client.NewRequest("PATCH", u, subIssue)
 	if err != nil {

github/strings_test.go

@@ -43,6 +43,15 @@ func TestSubIssuesService_Add(t *testing.T) {
 	if !cmp.Equal(got, want) {
 		t.Errorf("SubIssues.Add = %+v, want %+v", got, want)
 	}
+
+	const methodName = "Add"
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.SubIssue.Add(ctx, "o", "r", 1, input)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestSubIssuesService_ListByIssue(t *testing.T) {
@@ -67,6 +76,20 @@ func TestSubIssuesService_ListByIssue(t *testing.T) {
 	if !cmp.Equal(issues, want) {
 		t.Errorf("SubIssues.ListByIssue = %+v, want %+v", issues, want)
 	}
+
+	const methodName = "ListByIssue"
+	testBadOptions(t, methodName, func() (err error) {
+		_, _, err = client.SubIssue.ListByIssue(ctx, "\n", "\n", 1, opt)
+		return err
+	})
+
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.SubIssue.ListByIssue(ctx, "o", "r", 1, opt)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestSubIssuesService_Remove(t *testing.T) {
@@ -99,13 +122,22 @@ func TestSubIssuesService_Remove(t *testing.T) {
 	if !cmp.Equal(got, want) {
 		t.Errorf("SubIssues.Remove = %+v, want %+v", got, want)
 	}
+
+	const methodName = "Remove"
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.SubIssue.Remove(ctx, "o", "r", 1, input)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestSubIssuesService_Reprioritize(t *testing.T) {
 	t.Parallel()
 	client, mux, _ := setup(t)
 
-	input := &SubIssueRequest{SubIssueID: 42, AfterID: 5}
+	input := &SubIssueRequest{SubIssueID: 42, AfterID: Int64(5)}
 
 	mux.HandleFunc("/repos/o/r/issues/1/sub_issues/priority", func(w http.ResponseWriter, r *http.Request) {
 		testMethod(t, r, "PATCH")
@@ -132,4 +164,14 @@ func TestSubIssuesService_Reprioritize(t *testing.T) {
 	if !cmp.Equal(got, want) {
 		t.Errorf("SubIssues.Reprioritize = %+v, want %+v", got, want)
 	}
+
+	const methodName = "Reprioritize"
+
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.SubIssue.Reprioritize(ctx, "o", "r", 1, input)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }