From 90d4bbc1f81d082003b33b64a7cee5fed1fdba19 Mon Sep 17 00:00:00 2001 From: Oleksandr Redko Date: Fri, 5 Jun 2026 16:32:05 +0300 Subject: [PATCH 1/2] test: Fix test names and error messages --- github/actions_hosted_runners_test.go | 4 +- github/actions_oidc_test.go | 4 +- github/actions_permissions_orgs_test.go | 6 +- github/actions_runner_groups_test.go | 2 +- github/actions_runners_test.go | 18 +++--- github/actions_variables_test.go | 10 ++-- github/actions_workflow_runs_test.go | 26 ++++----- github/actions_workflows_test.go | 6 +- github/activity_events_test.go | 28 +++++----- github/admin_orgs_test.go | 14 ++--- github/apps_manifest_test.go | 6 +- github/apps_test.go | 2 +- github/codespaces_test.go | 2 +- .../enterprise_actions_hosted_runners_test.go | 4 +- .../enterprise_actions_runner_groups_test.go | 56 +++++++++---------- github/gists_comments_test.go | 4 +- github/gists_test.go | 12 ++-- github/git_blobs_test.go | 2 +- github/git_trees_test.go | 2 +- github/gitignore_test.go | 2 +- github/migrations_test.go | 4 +- github/orgs_members_test.go | 2 +- github/orgs_properties_test.go | 4 +- github/orgs_rules_test.go | 4 +- github/private_registries_test.go | 2 +- github/rate_limit_test.go | 6 +- github/reactions_test.go | 28 +++++----- github/repos_actions_allowed_test.go | 8 +-- github/repos_commits_test.go | 8 +-- github/repos_keys_test.go | 4 +- github/repos_prereceive_hooks_test.go | 2 +- github/repos_releases_test.go | 4 +- github/repos_stats_test.go | 8 +-- github/repos_test.go | 4 +- github/search_test.go | 6 +- github/teams_members_test.go | 12 ++-- github/teams_test.go | 6 +- github/users_followers_test.go | 2 +- github/users_gpg_keys_test.go | 2 +- github/users_test.go | 2 +- 40 files changed, 164 insertions(+), 164 deletions(-) diff --git a/github/actions_hosted_runners_test.go b/github/actions_hosted_runners_test.go index 10d97162b6b..4a98f800c71 100644 --- a/github/actions_hosted_runners_test.go +++ b/github/actions_hosted_runners_test.go @@ -454,7 +454,7 @@ func TestActionsService_GetHostedRunnerLimits(t *testing.T) { ctx := t.Context() publicIPLimits, _, err := client.Actions.GetHostedRunnerLimits(ctx, "o") if err != nil { - t.Errorf("Actions.GetPartnerImages returned error: %v", err) + t.Errorf("Actions.GetHostedRunnerLimits returned error: %v", err) } want := &HostedRunnerPublicIPLimits{ @@ -808,7 +808,7 @@ func TestActionsService_DeleteHostedRunner(t *testing.T) { ctx := t.Context() hostedRunner, _, err := client.Actions.DeleteHostedRunner(ctx, "o", 23) if err != nil { - t.Errorf("Actions.GetHostedRunner returned error: %v", err) + t.Errorf("Actions.DeleteHostedRunner returned error: %v", err) } lastActiveOn := Timestamp{time.Date(2023, 4, 26, 15, 23, 37, 0, time.UTC)} diff --git a/github/actions_oidc_test.go b/github/actions_oidc_test.go index 56684e02d48..9bad8dbbee2 100644 --- a/github/actions_oidc_test.go +++ b/github/actions_oidc_test.go @@ -65,7 +65,7 @@ func TestActionsService_GetRepoOIDCSubjectClaimCustomTemplate(t *testing.T) { want := &OIDCSubjectClaimCustomTemplate{UseDefault: Ptr(false), IncludeClaimKeys: []string{"repo", "context"}} if !cmp.Equal(template, want) { - t.Errorf("Actions.GetOrgOIDCSubjectClaimCustomTemplate returned %+v, want %+v", template, want) + t.Errorf("Actions.GetRepoOIDCSubjectClaimCustomTemplate returned %+v, want %+v", template, want) } const methodName = "GetRepoOIDCSubjectClaimCustomTemplate" @@ -150,7 +150,7 @@ func TestActionsService_SetRepoOIDCSubjectClaimCustomTemplate(t *testing.T) { }) } -func TestActionService_SetRepoOIDCSubjectClaimCustomTemplateToDefault(t *testing.T) { +func TestActionsService_SetRepoOIDCSubjectClaimCustomTemplateToDefault(t *testing.T) { t.Parallel() client, mux, _ := setup(t) diff --git a/github/actions_permissions_orgs_test.go b/github/actions_permissions_orgs_test.go index db27e11e02a..dc949ae04c0 100644 --- a/github/actions_permissions_orgs_test.go +++ b/github/actions_permissions_orgs_test.go @@ -103,7 +103,7 @@ func TestActionsService_ListEnabledReposInOrg(t *testing.T) { } got, _, err := client.Actions.ListEnabledReposInOrg(ctx, "o", opt) if err != nil { - t.Errorf("Actions.ListEnabledRepos returned error: %v", err) + t.Errorf("Actions.ListEnabledReposInOrg returned error: %v", err) } want := &ActionsEnabledOnOrgRepos{TotalCount: int(2), Repositories: []*Repository{ @@ -111,7 +111,7 @@ func TestActionsService_ListEnabledReposInOrg(t *testing.T) { {ID: Ptr(int64(3))}, }} if !cmp.Equal(got, want) { - t.Errorf("Actions.ListEnabledRepos returned %+v, want %+v", got, want) + t.Errorf("Actions.ListEnabledReposInOrg returned %+v, want %+v", got, want) } const methodName = "ListEnabledRepos" @@ -149,7 +149,7 @@ func TestActionsService_SetEnabledReposInOrg(t *testing.T) { ctx := t.Context() _, err := client.Actions.SetEnabledReposInOrg(ctx, "o", input) if err != nil { - t.Errorf("Actions.SetEnabledRepos returned error: %v", err) + t.Errorf("Actions.SetEnabledReposInOrg returned error: %v", err) } const methodName = "SetEnabledRepos" diff --git a/github/actions_runner_groups_test.go b/github/actions_runner_groups_test.go index 5bdc978175c..21cedd6d759 100644 --- a/github/actions_runner_groups_test.go +++ b/github/actions_runner_groups_test.go @@ -114,7 +114,7 @@ func TestActionsService_GetOrganizationRunnerGroup(t *testing.T) { ctx := t.Context() group, _, err := client.Actions.GetOrganizationRunnerGroup(ctx, "o", 2) if err != nil { - t.Errorf("Actions.ListOrganizationRunnerGroups returned error: %v", err) + t.Errorf("Actions.GetOrganizationRunnerGroup returned error: %v", err) } want := &RunnerGroup{ diff --git a/github/actions_runners_test.go b/github/actions_runners_test.go index 0940a21f2f9..ec487410038 100644 --- a/github/actions_runners_test.go +++ b/github/actions_runners_test.go @@ -327,7 +327,7 @@ func TestActionsService_ListOrganizationRunnerApplicationDownloads(t *testing.T) ctx := t.Context() downloads, _, err := client.Actions.ListOrganizationRunnerApplicationDownloads(ctx, "o") if err != nil { - t.Errorf("Actions.ListRunnerApplicationDownloads returned error: %v", err) + t.Errorf("Actions.ListOrganizationRunnerApplicationDownloads returned error: %v", err) } want := []*RunnerApplicationDownload{ @@ -368,7 +368,7 @@ func TestActionsService_CreateOrganizationRegistrationToken(t *testing.T) { ctx := t.Context() token, _, err := client.Actions.CreateOrganizationRegistrationToken(ctx, "o") if err != nil { - t.Errorf("Actions.CreateRegistrationToken returned error: %v", err) + t.Errorf("Actions.CreateOrganizationRegistrationToken returned error: %v", err) } want := &RegistrationToken{ @@ -377,7 +377,7 @@ func TestActionsService_CreateOrganizationRegistrationToken(t *testing.T) { 123000000, time.UTC)}, } if !cmp.Equal(token, want) { - t.Errorf("Actions.CreateRegistrationToken returned %+v, want %+v", token, want) + t.Errorf("Actions.CreateOrganizationRegistrationToken returned %+v, want %+v", token, want) } const methodName = "CreateOrganizationRegistrationToken" @@ -411,7 +411,7 @@ func TestActionsService_ListOrganizationRunners(t *testing.T) { ctx := t.Context() runners, _, err := client.Actions.ListOrganizationRunners(ctx, "o", opts) if err != nil { - t.Errorf("Actions.ListRunners returned error: %v", err) + t.Errorf("Actions.ListOrganizationRunners returned error: %v", err) } want := &Runners{ @@ -422,7 +422,7 @@ func TestActionsService_ListOrganizationRunners(t *testing.T) { }, } if !cmp.Equal(runners, want) { - t.Errorf("Actions.ListRunners returned %+v, want %+v", runners, want) + t.Errorf("Actions.ListOrganizationRunners returned %+v, want %+v", runners, want) } const methodName = "ListOrganizationRunners" @@ -452,7 +452,7 @@ func TestActionsService_GetOrganizationRunner(t *testing.T) { ctx := t.Context() runner, _, err := client.Actions.GetOrganizationRunner(ctx, "o", 23) if err != nil { - t.Errorf("Actions.GetRunner returned error: %v", err) + t.Errorf("Actions.GetOrganizationRunner returned error: %v", err) } want := &Runner{ @@ -462,7 +462,7 @@ func TestActionsService_GetOrganizationRunner(t *testing.T) { Status: Ptr("online"), } if !cmp.Equal(runner, want) { - t.Errorf("Actions.GetRunner returned %+v, want %+v", runner, want) + t.Errorf("Actions.GetOrganizationRunner returned %+v, want %+v", runner, want) } const methodName = "GetOrganizationRunner" @@ -492,12 +492,12 @@ func TestActionsService_CreateOrganizationRemoveToken(t *testing.T) { ctx := t.Context() token, _, err := client.Actions.CreateOrganizationRemoveToken(ctx, "o") if err != nil { - t.Errorf("Actions.CreateRemoveToken returned error: %v", err) + t.Errorf("Actions.CreateOrganizationRemoveToken returned error: %v", err) } want := &RemoveToken{Token: Ptr("AABF3JGZDX3P5PMEXLND6TS6FCWO6"), ExpiresAt: &Timestamp{time.Date(2020, time.January, 29, 12, 13, 35, 123000000, time.UTC)}} if !cmp.Equal(token, want) { - t.Errorf("Actions.CreateRemoveToken returned %+v, want %+v", token, want) + t.Errorf("Actions.CreateOrganizationRemoveToken returned %+v, want %+v", token, want) } const methodName = "CreateOrganizationRemoveToken" diff --git a/github/actions_variables_test.go b/github/actions_variables_test.go index 1e07e046dcb..07a51bccfef 100644 --- a/github/actions_variables_test.go +++ b/github/actions_variables_test.go @@ -221,7 +221,7 @@ func TestActionsService_DeleteRepoVariable(t *testing.T) { ctx := t.Context() _, err := client.Actions.DeleteRepoVariable(ctx, "o", "r", "NAME") if err != nil { - t.Errorf("Actions.( returned error: %v", err) + t.Errorf("Actions.DeleteRepoVariable returned error: %v", err) } const methodName = "DeleteRepoVariable" @@ -408,7 +408,7 @@ func TestActionsService_ListSelectedReposForOrgVariable(t *testing.T) { ctx := t.Context() repos, _, err := client.Actions.ListSelectedReposForOrgVariable(ctx, "o", "NAME", opts) if err != nil { - t.Errorf("Actions.( returned error: %v", err) + t.Errorf("Actions.ListSelectedReposForOrgVariable returned error: %v", err) } want := &SelectedReposList{ @@ -418,7 +418,7 @@ func TestActionsService_ListSelectedReposForOrgVariable(t *testing.T) { }, } if !cmp.Equal(repos, want) { - t.Errorf("Actions.( returned %+v, want %+v", repos, want) + t.Errorf("Actions.ListSelectedReposForOrgVariable returned %+v, want %+v", repos, want) } const methodName = "ListSelectedReposForOrgVariable" @@ -436,7 +436,7 @@ func TestActionsService_ListSelectedReposForOrgVariable(t *testing.T) { }) } -func TestActionsService_SetSelectedReposForOrgSVariable(t *testing.T) { +func TestActionsService_SetSelectedReposForOrgVariable(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -455,7 +455,7 @@ func TestActionsService_SetSelectedReposForOrgSVariable(t *testing.T) { ctx := t.Context() _, err := client.Actions.SetSelectedReposForOrgVariable(ctx, "o", "NAME", input) if err != nil { - t.Errorf("Actions.( returned error: %v", err) + t.Errorf("Actions.SetSelectedReposForOrgVariable returned error: %v", err) } const methodName = "SetSelectedReposForOrgVariable" diff --git a/github/actions_workflow_runs_test.go b/github/actions_workflow_runs_test.go index 06ab98788f6..d60bfe32a24 100644 --- a/github/actions_workflow_runs_test.go +++ b/github/actions_workflow_runs_test.go @@ -31,7 +31,7 @@ func TestActionsService_ListWorkflowRunsByID(t *testing.T) { ctx := t.Context() runs, _, err := client.Actions.ListWorkflowRunsByID(ctx, "o", "r", 29679449, opts) if err != nil { - t.Errorf("Actions.ListWorkFlowRunsByID returned error: %v", err) + t.Errorf("Actions.ListWorkflowRunsByID returned error: %v", err) } want := &WorkflowRuns{ @@ -74,7 +74,7 @@ func TestActionsService_ListWorkflowRunsFileName(t *testing.T) { ctx := t.Context() runs, _, err := client.Actions.ListWorkflowRunsByFileName(ctx, "o", "r", "29679449", opts) if err != nil { - t.Errorf("Actions.ListWorkFlowRunsByFileName returned error: %v", err) + t.Errorf("Actions.ListWorkflowRunsByFileName returned error: %v", err) } want := &WorkflowRuns{ @@ -383,7 +383,7 @@ func TestActionsService_GetWorkflowRunAttemptLogs_unexpectedCode(t *testing.T) { } } -func TestActionsService_RerunWorkflowRunByID(t *testing.T) { +func TestActionsService_RerunWorkflowByID(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -398,7 +398,7 @@ func TestActionsService_RerunWorkflowRunByID(t *testing.T) { t.Errorf("Actions.RerunWorkflowByID returned error: %v", err) } if resp.StatusCode != http.StatusCreated { - t.Errorf("Actions.RerunWorkflowRunByID returned status: %v, want %v", resp.StatusCode, http.StatusCreated) + t.Errorf("Actions.RerunWorkflowByID returned status: %v, want %v", resp.StatusCode, http.StatusCreated) } const methodName = "RerunWorkflowByID" @@ -578,7 +578,7 @@ func TestActionsService_GetWorkflowRunLogs_StatusMovedPermanently_dontFollowRedi ctx := t.Context() _, resp, _ := client.Actions.GetWorkflowRunLogs(ctx, "o", "r", 399444496, 0) if resp.StatusCode != http.StatusMovedPermanently { - t.Errorf("Actions.GetWorkflowJobLogs returned status: %v, want %v", resp.StatusCode, http.StatusMovedPermanently) + t.Errorf("Actions.GetWorkflowRunLogs returned status: %v, want %v", resp.StatusCode, http.StatusMovedPermanently) } }) } @@ -621,16 +621,16 @@ func TestActionsService_GetWorkflowRunLogs_StatusMovedPermanently_followRedirect ctx := t.Context() url, resp, err := client.Actions.GetWorkflowRunLogs(ctx, "o", "r", 399444496, 1) if err != nil { - t.Errorf("Actions.GetWorkflowJobLogs returned error: %v", err) + t.Errorf("Actions.GetWorkflowRunLogs returned error: %v", err) } if resp.StatusCode != http.StatusFound { - t.Errorf("Actions.GetWorkflowJobLogs returned status: %v, want %v", resp.StatusCode, http.StatusFound) + t.Errorf("Actions.GetWorkflowRunLogs returned status: %v, want %v", resp.StatusCode, http.StatusFound) } want := "https://github.com/a" if url.String() != want { - t.Errorf("Actions.GetWorkflowJobLogs returned %+v, want %+v", url, want) + t.Errorf("Actions.GetWorkflowRunLogs returned %+v, want %+v", url, want) } const methodName = "GetWorkflowRunLogs" @@ -694,7 +694,7 @@ func TestActionsService_GetWorkflowRunLogs_unexpectedCode(t *testing.T) { } } -func TestActionService_ListRepositoryWorkflowRuns(t *testing.T) { +func TestActionsService_ListRepositoryWorkflowRuns(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -743,7 +743,7 @@ func TestActionService_ListRepositoryWorkflowRuns(t *testing.T) { }) } -func TestActionService_DeleteWorkflowRun(t *testing.T) { +func TestActionsService_DeleteWorkflowRun(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -769,7 +769,7 @@ func TestActionService_DeleteWorkflowRun(t *testing.T) { }) } -func TestActionService_DeleteWorkflowRunLogs(t *testing.T) { +func TestActionsService_DeleteWorkflowRunLogs(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -895,7 +895,7 @@ func TestActionsService_GetWorkflowRunUsageByID(t *testing.T) { }) } -func TestActionService_PendingDeployments(t *testing.T) { +func TestActionsService_PendingDeployments(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -933,7 +933,7 @@ func TestActionService_PendingDeployments(t *testing.T) { }) } -func TestActionService_GetPendingDeployments(t *testing.T) { +func TestActionsService_GetPendingDeployments(t *testing.T) { t.Parallel() client, mux, _ := setup(t) diff --git a/github/actions_workflows_test.go b/github/actions_workflows_test.go index 3b27d54ff1b..c69c78f7274 100644 --- a/github/actions_workflows_test.go +++ b/github/actions_workflows_test.go @@ -428,7 +428,7 @@ func TestActionsService_EnableWorkflowByID(t *testing.T) { }) } -func TestActionsService_EnableWorkflowByFilename(t *testing.T) { +func TestActionsService_EnableWorkflowByFileName(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -442,14 +442,14 @@ func TestActionsService_EnableWorkflowByFilename(t *testing.T) { ctx := t.Context() _, err := client.Actions.EnableWorkflowByFileName(ctx, "o", "r", "main.yml") if err != nil { - t.Errorf("Actions.EnableWorkflowByFilename returned error: %v", err) + t.Errorf("Actions.EnableWorkflowByFileName returned error: %v", err) } // Test s.client.NewRequest failure client.baseURL.Path = "" _, err = client.Actions.EnableWorkflowByFileName(ctx, "o", "r", "main.yml") if err == nil { - t.Error("client.BaseURL.Path='' EnableWorkflowByFilename err = nil, want error") + t.Error("client.BaseURL.Path='' EnableWorkflowByFileName err = nil, want error") } const methodName = "EnableWorkflowByFileName" diff --git a/github/activity_events_test.go b/github/activity_events_test.go index b84611c4546..459c8ab4f46 100644 --- a/github/activity_events_test.go +++ b/github/activity_events_test.go @@ -160,12 +160,12 @@ func TestActivityService_ListEventsForRepoNetwork(t *testing.T) { ctx := t.Context() events, _, err := client.Activity.ListEventsForRepoNetwork(ctx, "o", "r", opt) if err != nil { - t.Errorf("Activities.ListEventsForRepoNetwork returned error: %v", err) + t.Errorf("Activity.ListEventsForRepoNetwork returned error: %v", err) } want := []*Event{{ID: Ptr("1")}, {ID: Ptr("2")}} if !cmp.Equal(events, want) { - t.Errorf("Activities.ListEventsForRepoNetwork returned %+v, want %+v", events, want) + t.Errorf("Activity.ListEventsForRepoNetwork returned %+v, want %+v", events, want) } const methodName = "ListEventsForRepoNetwork" @@ -208,12 +208,12 @@ func TestActivityService_ListEventsForOrganization(t *testing.T) { ctx := t.Context() events, _, err := client.Activity.ListEventsForOrganization(ctx, "o", opt) if err != nil { - t.Errorf("Activities.ListEventsForOrganization returned error: %v", err) + t.Errorf("Activity.ListEventsForOrganization returned error: %v", err) } want := []*Event{{ID: Ptr("1")}, {ID: Ptr("2")}} if !cmp.Equal(events, want) { - t.Errorf("Activities.ListEventsForOrganization returned %+v, want %+v", events, want) + t.Errorf("Activity.ListEventsForOrganization returned %+v, want %+v", events, want) } const methodName = "ListEventsForOrganization" @@ -256,12 +256,12 @@ func TestActivityService_ListEventsPerformedByUser_all(t *testing.T) { ctx := t.Context() events, _, err := client.Activity.ListEventsPerformedByUser(ctx, "u", false, opt) if err != nil { - t.Errorf("Events.ListPerformedByUser returned error: %v", err) + t.Errorf("Activity.ListEventsPerformedByUser returned error: %v", err) } want := []*Event{{ID: Ptr("1")}, {ID: Ptr("2")}} if !cmp.Equal(events, want) { - t.Errorf("Events.ListPerformedByUser returned %+v, want %+v", events, want) + t.Errorf("Activity.ListEventsPerformedByUser returned %+v, want %+v", events, want) } const methodName = "ListEventsPerformedByUser" @@ -291,12 +291,12 @@ func TestActivityService_ListEventsPerformedByUser_publicOnly(t *testing.T) { ctx := t.Context() events, _, err := client.Activity.ListEventsPerformedByUser(ctx, "u", true, nil) if err != nil { - t.Errorf("Events.ListPerformedByUser returned error: %v", err) + t.Errorf("Activity.ListEventsPerformedByUser returned error: %v", err) } want := []*Event{{ID: Ptr("1")}, {ID: Ptr("2")}} if !cmp.Equal(events, want) { - t.Errorf("Events.ListPerformedByUser returned %+v, want %+v", events, want) + t.Errorf("Activity.ListEventsPerformedByUser returned %+v, want %+v", events, want) } } @@ -325,12 +325,12 @@ func TestActivityService_ListEventsReceivedByUser_all(t *testing.T) { ctx := t.Context() events, _, err := client.Activity.ListEventsReceivedByUser(ctx, "u", false, opt) if err != nil { - t.Errorf("Events.ListReceivedByUser returned error: %v", err) + t.Errorf("Activity.ListEventsReceivedByUser returned error: %v", err) } want := []*Event{{ID: Ptr("1")}, {ID: Ptr("2")}} if !cmp.Equal(events, want) { - t.Errorf("Events.ListReceivedUser returned %+v, want %+v", events, want) + t.Errorf("Activity.ListEventsReceivedByUser returned %+v, want %+v", events, want) } const methodName = "ListEventsReceivedByUser" @@ -360,12 +360,12 @@ func TestActivityService_ListEventsReceivedByUser_publicOnly(t *testing.T) { ctx := t.Context() events, _, err := client.Activity.ListEventsReceivedByUser(ctx, "u", true, nil) if err != nil { - t.Errorf("Events.ListReceivedByUser returned error: %v", err) + t.Errorf("Activity.ListEventsReceivedByUser returned error: %v", err) } want := []*Event{{ID: Ptr("1")}, {ID: Ptr("2")}} if !cmp.Equal(events, want) { - t.Errorf("Events.ListReceivedByUser returned %+v, want %+v", events, want) + t.Errorf("Activity.ListEventsReceivedByUser returned %+v, want %+v", events, want) } } @@ -394,12 +394,12 @@ func TestActivityService_ListUserEventsForOrganization(t *testing.T) { ctx := t.Context() events, _, err := client.Activity.ListUserEventsForOrganization(ctx, "o", "u", opt) if err != nil { - t.Errorf("Activities.ListUserEventsForOrganization returned error: %v", err) + t.Errorf("Activity.ListUserEventsForOrganization returned error: %v", err) } want := []*Event{{ID: Ptr("1")}, {ID: Ptr("2")}} if !cmp.Equal(events, want) { - t.Errorf("Activities.ListUserEventsForOrganization returned %+v, want %+v", events, want) + t.Errorf("Activity.ListUserEventsForOrganization returned %+v, want %+v", events, want) } const methodName = "ListUserEventsForOrganization" diff --git a/github/admin_orgs_test.go b/github/admin_orgs_test.go index 46fa5f77632..83ef0ba963f 100644 --- a/github/admin_orgs_test.go +++ b/github/admin_orgs_test.go @@ -13,7 +13,7 @@ import ( "github.com/google/go-cmp/cmp" ) -func TestAdminOrgs_Create(t *testing.T) { +func TestAdminService_CreateOrg(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -48,7 +48,7 @@ func TestAdminOrgs_Create(t *testing.T) { }) } -func TestAdminOrgs_Rename(t *testing.T) { +func TestAdminService_RenameOrg(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -66,12 +66,12 @@ func TestAdminOrgs_Rename(t *testing.T) { ctx := t.Context() resp, _, err := client.Admin.RenameOrg(ctx, input, "the-new-octocats") if err != nil { - t.Errorf("Admin.RenameOrg returned error: %v", err) + t.Errorf("Admin.RenameOrgByName returned error: %v", err) } want := &RenameOrgResponse{Message: Ptr("Job queued to rename organization. It may take a few minutes to complete."), URL: Ptr("https:///api/v3/organizations/1")} if !cmp.Equal(resp, want) { - t.Errorf("Admin.RenameOrg returned %+v, want %+v", resp, want) + t.Errorf("Admin.RenameOrgByName returned %+v, want %+v", resp, want) } const methodName = "RenameOrg" @@ -93,7 +93,7 @@ func TestAdminOrgs_Rename(t *testing.T) { }) } -func TestAdminOrgs_RenameByName(t *testing.T) { +func TestAdminService_RenameOrgByName(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -107,12 +107,12 @@ func TestAdminOrgs_RenameByName(t *testing.T) { ctx := t.Context() resp, _, err := client.Admin.RenameOrgByName(ctx, "o", "the-new-octocats") if err != nil { - t.Errorf("Admin.RenameOrg returned error: %v", err) + t.Errorf("Admin.RenameOrgByName returned error: %v", err) } want := &RenameOrgResponse{Message: Ptr("Job queued to rename organization. It may take a few minutes to complete."), URL: Ptr("https:///api/v3/organizations/1")} if !cmp.Equal(resp, want) { - t.Errorf("Admin.RenameOrg returned %+v, want %+v", resp, want) + t.Errorf("Admin.RenameOrgByName returned %+v, want %+v", resp, want) } const methodName = "RenameOrgByName" diff --git a/github/apps_manifest_test.go b/github/apps_manifest_test.go index 2478c3d096c..799bb2eb494 100644 --- a/github/apps_manifest_test.go +++ b/github/apps_manifest_test.go @@ -24,7 +24,7 @@ const ( ` ) -func TestGetConfig(t *testing.T) { +func TestAppsService_CompleteAppManifest(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -36,7 +36,7 @@ func TestGetConfig(t *testing.T) { ctx := t.Context() cfg, _, err := client.Apps.CompleteAppManifest(ctx, "code") if err != nil { - t.Errorf("AppManifest.GetConfig returned error: %v", err) + t.Errorf("Apps.CompleteAppManifest returned error: %v", err) } want := &AppConfig{ @@ -48,7 +48,7 @@ func TestGetConfig(t *testing.T) { } if !cmp.Equal(cfg, want) { - t.Errorf("GetConfig returned %+v, want %+v", cfg, want) + t.Errorf("Apps.CompleteAppManifest returned %+v, want %+v", cfg, want) } const methodName = "CompleteAppManifest" diff --git a/github/apps_test.go b/github/apps_test.go index dec340c5a89..55c9b2bd00c 100644 --- a/github/apps_test.go +++ b/github/apps_test.go @@ -93,7 +93,7 @@ func TestAppsService_ListInstallationRequests(t *testing.T) { ctx := t.Context() installationRequests, _, err := client.Apps.ListInstallationRequests(ctx, opt) if err != nil { - t.Errorf("Apps.ListInstallations returned error: %v", err) + t.Errorf("Apps.ListInstallationRequests returned error: %v", err) } date := Timestamp{Time: time.Date(2018, time.January, 1, 0, 0, 0, 0, time.UTC)} diff --git a/github/codespaces_test.go b/github/codespaces_test.go index c20253d916e..4c6494d0190 100644 --- a/github/codespaces_test.go +++ b/github/codespaces_test.go @@ -129,7 +129,7 @@ func TestCodespacesService_List(t *testing.T) { }, }} if !cmp.Equal(codespaces, want) { - t.Errorf("Codespaces.ListInRepo returned %+v, want %+v", codespaces, want) + t.Errorf("Codespaces.List returned %+v, want %+v", codespaces, want) } const methodName = "List" diff --git a/github/enterprise_actions_hosted_runners_test.go b/github/enterprise_actions_hosted_runners_test.go index 1bff9ccf8c7..1575ef22086 100644 --- a/github/enterprise_actions_hosted_runners_test.go +++ b/github/enterprise_actions_hosted_runners_test.go @@ -452,7 +452,7 @@ func TestEnterpriseService_GetHostedRunnerLimits(t *testing.T) { ctx := t.Context() publicIPLimits, _, err := client.Enterprise.GetHostedRunnerLimits(ctx, "o") if err != nil { - t.Errorf("Enterprise.GetPartnerImages returned error: %v", err) + t.Errorf("Enterprise.GetHostedRunnerLimits returned error: %v", err) } want := &HostedRunnerPublicIPLimits{ @@ -807,7 +807,7 @@ func TestEnterpriseService_DeleteHostedRunner(t *testing.T) { ctx := t.Context() hostedRunner, _, err := client.Enterprise.DeleteHostedRunner(ctx, "o", 23) if err != nil { - t.Errorf("Enterprise.GetHostedRunner returned error: %v", err) + t.Errorf("Enterprise.DeleteHostedRunner returned error: %v", err) } lastActiveOn := Timestamp{time.Date(2023, 4, 26, 15, 23, 37, 0, time.UTC)} diff --git a/github/enterprise_actions_runner_groups_test.go b/github/enterprise_actions_runner_groups_test.go index bb7ea22f73e..f77a4ff0450 100644 --- a/github/enterprise_actions_runner_groups_test.go +++ b/github/enterprise_actions_runner_groups_test.go @@ -101,7 +101,7 @@ func TestEnterpriseService_ListRunnerGroupsVisibleToOrganization(t *testing.T) { }) } -func TestEnterpriseService_GetRunnerGroup(t *testing.T) { +func TestEnterpriseService_GetEnterpriseRunnerGroup(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -113,7 +113,7 @@ func TestEnterpriseService_GetRunnerGroup(t *testing.T) { ctx := t.Context() group, _, err := client.Enterprise.GetEnterpriseRunnerGroup(ctx, "o", 2) if err != nil { - t.Errorf("Enterprise.GetRunnerGroup returned error: %v", err) + t.Errorf("Enterprise.GetEnterpriseRunnerGroup returned error: %v", err) } want := &EnterpriseRunnerGroup{ @@ -130,10 +130,10 @@ func TestEnterpriseService_GetRunnerGroup(t *testing.T) { } if !cmp.Equal(group, want) { - t.Errorf("Enterprise.GetRunnerGroup returned %+v, want %+v", group, want) + t.Errorf("Enterprise.GetEnterpriseRunnerGroup returned %+v, want %+v", group, want) } - const methodName = "GetRunnerGroup" + const methodName = "GetEnterpriseRunnerGroup" testBadOptions(t, methodName, func() (err error) { _, _, err = client.Enterprise.GetEnterpriseRunnerGroup(ctx, "\n", 2) return err @@ -148,7 +148,7 @@ func TestEnterpriseService_GetRunnerGroup(t *testing.T) { }) } -func TestEnterpriseService_DeleteRunnerGroup(t *testing.T) { +func TestEnterpriseService_DeleteEnterpriseRunnerGroup(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -159,10 +159,10 @@ func TestEnterpriseService_DeleteRunnerGroup(t *testing.T) { ctx := t.Context() _, err := client.Enterprise.DeleteEnterpriseRunnerGroup(ctx, "o", 2) if err != nil { - t.Errorf("Enterprise.DeleteRunnerGroup returned error: %v", err) + t.Errorf("Enterprise.DeleteEnterpriseRunnerGroup returned error: %v", err) } - const methodName = "DeleteRunnerGroup" + const methodName = "DeleteEnterpriseRunnerGroup" testBadOptions(t, methodName, func() (err error) { _, err = client.Enterprise.DeleteEnterpriseRunnerGroup(ctx, "\n", 2) return err @@ -173,7 +173,7 @@ func TestEnterpriseService_DeleteRunnerGroup(t *testing.T) { }) } -func TestEnterpriseService_CreateRunnerGroup(t *testing.T) { +func TestEnterpriseService_CreateEnterpriseRunnerGroup(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -192,7 +192,7 @@ func TestEnterpriseService_CreateRunnerGroup(t *testing.T) { } group, _, err := client.Enterprise.CreateEnterpriseRunnerGroup(ctx, "o", req) if err != nil { - t.Errorf("Enterprise.CreateRunnerGroup returned error: %v", err) + t.Errorf("Enterprise.CreateEnterpriseRunnerGroup returned error: %v", err) } want := &EnterpriseRunnerGroup{ @@ -209,10 +209,10 @@ func TestEnterpriseService_CreateRunnerGroup(t *testing.T) { } if !cmp.Equal(group, want) { - t.Errorf("Enterprise.CreateRunnerGroup returned %+v, want %+v", group, want) + t.Errorf("Enterprise.CreateEnterpriseRunnerGroup returned %+v, want %+v", group, want) } - const methodName = "CreateRunnerGroup" + const methodName = "CreateEnterpriseRunnerGroup" testBadOptions(t, methodName, func() (err error) { _, _, err = client.Enterprise.CreateEnterpriseRunnerGroup(ctx, "\n", req) return err @@ -227,7 +227,7 @@ func TestEnterpriseService_CreateRunnerGroup(t *testing.T) { }) } -func TestEnterpriseService_UpdateRunnerGroup(t *testing.T) { +func TestEnterpriseService_UpdateEnterpriseRunnerGroup(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -246,7 +246,7 @@ func TestEnterpriseService_UpdateRunnerGroup(t *testing.T) { } group, _, err := client.Enterprise.UpdateEnterpriseRunnerGroup(ctx, "o", 2, req) if err != nil { - t.Errorf("Enterprise.UpdateRunnerGroup returned error: %v", err) + t.Errorf("Enterprise.UpdateEnterpriseRunnerGroup returned error: %v", err) } want := &EnterpriseRunnerGroup{ @@ -263,10 +263,10 @@ func TestEnterpriseService_UpdateRunnerGroup(t *testing.T) { } if !cmp.Equal(group, want) { - t.Errorf("Enterprise.UpdateRunnerGroup returned %+v, want %+v", group, want) + t.Errorf("Enterprise.UpdateEnterpriseRunnerGroup returned %+v, want %+v", group, want) } - const methodName = "UpdateRunnerGroup" + const methodName = "UpdateEnterpriseRunnerGroup" testBadOptions(t, methodName, func() (err error) { _, _, err = client.Enterprise.UpdateEnterpriseRunnerGroup(ctx, "\n", 2, req) return err @@ -405,7 +405,7 @@ func TestEnterpriseService_RemoveOrganizationAccessRunnerGroup(t *testing.T) { }) } -func TestEnterpriseService_ListEnterpriseRunnerGroupRunners(t *testing.T) { +func TestEnterpriseService_ListRunnerGroupRunners(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -419,7 +419,7 @@ func TestEnterpriseService_ListEnterpriseRunnerGroupRunners(t *testing.T) { ctx := t.Context() runners, _, err := client.Enterprise.ListRunnerGroupRunners(ctx, "o", 2, opts) if err != nil { - t.Errorf("Enterprise.ListEnterpriseRunnerGroupRunners returned error: %v", err) + t.Errorf("Enterprise.ListRunnerGroupRunners returned error: %v", err) } want := &Runners{ @@ -430,10 +430,10 @@ func TestEnterpriseService_ListEnterpriseRunnerGroupRunners(t *testing.T) { }, } if !cmp.Equal(runners, want) { - t.Errorf("Enterprise.ListEnterpriseRunnerGroupRunners returned %+v, want %+v", runners, want) + t.Errorf("Enterprise.ListRunnerGroupRunners returned %+v, want %+v", runners, want) } - const methodName = "ListEnterpriseRunnerGroupRunners" + const methodName = "ListRunnerGroupRunners" testBadOptions(t, methodName, func() (err error) { _, _, err = client.Enterprise.ListRunnerGroupRunners(ctx, "\n", 2, opts) return err @@ -448,7 +448,7 @@ func TestEnterpriseService_ListEnterpriseRunnerGroupRunners(t *testing.T) { }) } -func TestEnterpriseService_SetEnterpriseRunnerGroupRunners(t *testing.T) { +func TestEnterpriseService_SetRunnerGroupRunners(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -466,10 +466,10 @@ func TestEnterpriseService_SetEnterpriseRunnerGroupRunners(t *testing.T) { ctx := t.Context() _, err := client.Enterprise.SetRunnerGroupRunners(ctx, "o", 2, req) if err != nil { - t.Errorf("Enterprise.SetEnterpriseRunnerGroupRunners returned error: %v", err) + t.Errorf("Enterprise.SetRunnerGroupRunners returned error: %v", err) } - const methodName = "SetEnterpriseRunnerGroupRunners" + const methodName = "SetRunnerGroupRunners" testBadOptions(t, methodName, func() (err error) { _, err = client.Enterprise.SetRunnerGroupRunners(ctx, "\n", 2, req) return err @@ -480,7 +480,7 @@ func TestEnterpriseService_SetEnterpriseRunnerGroupRunners(t *testing.T) { }) } -func TestEnterpriseService_AddEnterpriseRunnerGroupRunners(t *testing.T) { +func TestEnterpriseService_AddRunnerGroupRunners(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -491,10 +491,10 @@ func TestEnterpriseService_AddEnterpriseRunnerGroupRunners(t *testing.T) { ctx := t.Context() _, err := client.Enterprise.AddRunnerGroupRunners(ctx, "o", 2, 42) if err != nil { - t.Errorf("Enterprise.AddEnterpriseRunnerGroupRunners returned error: %v", err) + t.Errorf("Enterprise.AddRunnerGroupRunners returned error: %v", err) } - const methodName = "AddEnterpriseRunnerGroupRunners" + const methodName = "AddRunnerGroupRunners" testBadOptions(t, methodName, func() (err error) { _, err = client.Enterprise.AddRunnerGroupRunners(ctx, "\n", 2, 42) return err @@ -505,7 +505,7 @@ func TestEnterpriseService_AddEnterpriseRunnerGroupRunners(t *testing.T) { }) } -func TestEnterpriseService_RemoveEnterpriseRunnerGroupRunners(t *testing.T) { +func TestEnterpriseService_RemoveRunnerGroupRunners(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -516,10 +516,10 @@ func TestEnterpriseService_RemoveEnterpriseRunnerGroupRunners(t *testing.T) { ctx := t.Context() _, err := client.Enterprise.RemoveRunnerGroupRunners(ctx, "o", 2, 42) if err != nil { - t.Errorf("Enterprise.RemoveEnterpriseRunnerGroupRunners returned error: %v", err) + t.Errorf("Enterprise.RemoveRunnerGroupRunners returned error: %v", err) } - const methodName = "RemoveEnterpriseRunnerGroupRunners" + const methodName = "RemoveRunnerGroupRunners" testBadOptions(t, methodName, func() (err error) { _, err = client.Enterprise.RemoveRunnerGroupRunners(ctx, "\n", 2, 42) return err diff --git a/github/gists_comments_test.go b/github/gists_comments_test.go index 05c5458b8e4..9db219e64fb 100644 --- a/github/gists_comments_test.go +++ b/github/gists_comments_test.go @@ -27,7 +27,7 @@ func TestGistsService_ListComments(t *testing.T) { ctx := t.Context() comments, _, err := client.Gists.ListComments(ctx, "1", opt) if err != nil { - t.Errorf("Gists.Comments returned error: %v", err) + t.Errorf("Gists.ListComments returned error: %v", err) } want := []*GistComment{{ID: Ptr(int64(1))}} @@ -208,7 +208,7 @@ func TestGistsService_DeleteComment(t *testing.T) { ctx := t.Context() _, err := client.Gists.DeleteComment(ctx, "1", 2) if err != nil { - t.Errorf("Gists.Delete returned error: %v", err) + t.Errorf("Gists.DeleteComment returned error: %v", err) } const methodName = "DeleteComment" diff --git a/github/gists_test.go b/github/gists_test.go index 3e6c85ce059..39928c0d2bc 100644 --- a/github/gists_test.go +++ b/github/gists_test.go @@ -227,12 +227,12 @@ func TestGistsService_GetRevision(t *testing.T) { ctx := t.Context() gist, _, err := client.Gists.GetRevision(ctx, "1", "s") if err != nil { - t.Errorf("Gists.Get returned error: %v", err) + t.Errorf("Gists.GetRevision returned error: %v", err) } want := &Gist{ID: Ptr("1")} if !cmp.Equal(gist, want) { - t.Errorf("Gists.Get returned %+v, want %+v", gist, want) + t.Errorf("Gists.GetRevision returned %+v, want %+v", gist, want) } const methodName = "GetRevision" @@ -611,10 +611,10 @@ func TestGistsService_IsStarred_hasStar(t *testing.T) { ctx := t.Context() star, _, err := client.Gists.IsStarred(ctx, "1") if err != nil { - t.Errorf("Gists.Starred returned error: %v", err) + t.Errorf("Gists.IsStarred returned error: %v", err) } if want := true; star != want { - t.Errorf("Gists.Starred returned %+v, want %+v", star, want) + t.Errorf("Gists.IsStarred returned %+v, want %+v", star, want) } const methodName = "IsStarred" @@ -644,10 +644,10 @@ func TestGistsService_IsStarred_noStar(t *testing.T) { ctx := t.Context() star, _, err := client.Gists.IsStarred(ctx, "1") if err != nil { - t.Errorf("Gists.Starred returned error: %v", err) + t.Errorf("Gists.IsStarred returned error: %v", err) } if want := false; star != want { - t.Errorf("Gists.Starred returned %+v, want %+v", star, want) + t.Errorf("Gists.IsStarred returned %+v, want %+v", star, want) } const methodName = "IsStarred" diff --git a/github/git_blobs_test.go b/github/git_blobs_test.go index c36ff4c3d70..cfc02da02de 100644 --- a/github/git_blobs_test.go +++ b/github/git_blobs_test.go @@ -39,7 +39,7 @@ func TestGitService_GetBlob(t *testing.T) { } if !cmp.Equal(*blob, want) { - t.Errorf("Blob.Get returned %+v, want %+v", *blob, want) + t.Errorf("Git.GetBlob returned %+v, want %+v", *blob, want) } const methodName = "GetBlob" diff --git a/github/git_trees_test.go b/github/git_trees_test.go index 5c269ef0526..035c648e30c 100644 --- a/github/git_trees_test.go +++ b/github/git_trees_test.go @@ -42,7 +42,7 @@ func TestGitService_GetTree(t *testing.T) { Truncated: Ptr(true), } if !cmp.Equal(*tree, want) { - t.Errorf("Tree.Get returned %+v, want %+v", *tree, want) + t.Errorf("Git.GetTree returned %+v, want %+v", *tree, want) } const methodName = "GetTree" diff --git a/github/gitignore_test.go b/github/gitignore_test.go index 16dbac4b216..1d21836ea4b 100644 --- a/github/gitignore_test.go +++ b/github/gitignore_test.go @@ -55,7 +55,7 @@ func TestGitignoresService_Get(t *testing.T) { ctx := t.Context() gitignore, _, err := client.Gitignores.Get(ctx, "name") if err != nil { - t.Errorf("Gitignores.List returned error: %v", err) + t.Errorf("Gitignores.Get returned error: %v", err) } want := &Gitignore{Name: Ptr("Name"), Source: Ptr("template source")} diff --git a/github/migrations_test.go b/github/migrations_test.go index eb6568fe570..73d34ec46f2 100644 --- a/github/migrations_test.go +++ b/github/migrations_test.go @@ -107,7 +107,7 @@ func TestMigrationService_MigrationStatus(t *testing.T) { ctx := t.Context() got, _, err := client.Migrations.MigrationStatus(ctx, "o", 1) if err != nil { - t.Errorf("MigrationStatus returned error: %v", err) + t.Errorf("Migrations.MigrationStatus returned error: %v", err) } if want := wantMigration; !cmp.Equal(got, want) { t.Errorf("MigrationStatus = %+v, want %+v", got, want) @@ -148,7 +148,7 @@ func TestMigrationService_MigrationArchiveURL_Redirect(t *testing.T) { ctx := t.Context() got, err := client.Migrations.MigrationArchiveURL(ctx, "o", 1) if err != nil { - t.Errorf("MigrationStatus returned error: %v", err) + t.Errorf("Migrations.MigrationArchiveURL returned error: %v", err) } if want := "/yo"; !strings.HasSuffix(got, want) { t.Errorf("MigrationArchiveURL = %+v, want %+v", got, want) diff --git a/github/orgs_members_test.go b/github/orgs_members_test.go index 8f25c7ac491..7e77f1de3bb 100644 --- a/github/orgs_members_test.go +++ b/github/orgs_members_test.go @@ -673,7 +673,7 @@ func TestOrganizationsService_CreateOrgInvitation(t *testing.T) { want := &Invitation{Email: Ptr("octocat@github.com")} if !cmp.Equal(invitations, want) { - t.Errorf("Organizations.ListPendingOrgInvitations returned %+v, want %+v", invitations, want) + t.Errorf("Organizations.CreateOrgInvitation returned %+v, want %+v", invitations, want) } const methodName = "CreateOrgInvitation" diff --git a/github/orgs_properties_test.go b/github/orgs_properties_test.go index 036b23d7783..c2e5f8a5201 100644 --- a/github/orgs_properties_test.go +++ b/github/orgs_properties_test.go @@ -516,10 +516,10 @@ func TestOrganizationsService_CreateOrUpdateRepoCustomPropertyValues(t *testing. ctx := t.Context() _, err := client.Organizations.CreateOrUpdateRepoCustomPropertyValues(ctx, "o", input.RepositoryNames, input.Properties) if err != nil { - t.Errorf("Organizations.CreateOrUpdateCustomPropertyValuesForRepos returned error: %v", err) + t.Errorf("Organizations.CreateOrUpdateRepoCustomPropertyValues returned error: %v", err) } - const methodName = "CreateOrUpdateCustomPropertyValuesForRepos" + const methodName = "CreateOrUpdateRepoCustomPropertyValues" testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) { return client.Organizations.CreateOrUpdateRepoCustomPropertyValues(ctx, "o", nil, nil) diff --git a/github/orgs_rules_test.go b/github/orgs_rules_test.go index 3d583f321f0..8d46b456c33 100644 --- a/github/orgs_rules_test.go +++ b/github/orgs_rules_test.go @@ -1242,7 +1242,7 @@ func TestOrganizationsService_GetRepositoryRuleset(t *testing.T) { ctx := t.Context() rulesets, _, err := client.Organizations.GetRepositoryRuleset(ctx, "o", 21) if err != nil { - t.Errorf("Organizations.GetOrganizationRepositoryRuleset returned error: %v", err) + t.Errorf("Organizations.GetRepositoryRuleset returned error: %v", err) } want := &RepositoryRuleset{ @@ -1329,7 +1329,7 @@ func TestOrganizationsService_GetRepositoryRulesetWithRepoPropCondition(t *testi ctx := t.Context() rulesets, _, err := client.Organizations.GetRepositoryRuleset(ctx, "o", 21) if err != nil { - t.Errorf("Organizations.GetOrganizationRepositoryRuleset returned error: %v", err) + t.Errorf("Organizations.GetRepositoryRuleset returned error: %v", err) } want := &RepositoryRuleset{ diff --git a/github/private_registries_test.go b/github/private_registries_test.go index 58cea11371b..fccb6af0b25 100644 --- a/github/private_registries_test.go +++ b/github/private_registries_test.go @@ -118,7 +118,7 @@ func TestPrivateRegistriesService_CreateOrganizationPrivateRegistry(t *testing.T ctx := t.Context() privateRegistry, _, err := client.PrivateRegistries.CreateOrganizationPrivateRegistry(ctx, "o", *input) if err != nil { - t.Fatalf("PrivateRegistries.CreateOrganizationPrivateRegistries returned error: %v", err) + t.Fatalf("PrivateRegistries.CreateOrganizationPrivateRegistry returned error: %v", err) } want := &PrivateRegistry{ diff --git a/github/rate_limit_test.go b/github/rate_limit_test.go index 36430a7ed98..5c90948ecae 100644 --- a/github/rate_limit_test.go +++ b/github/rate_limit_test.go @@ -61,7 +61,7 @@ func TestRateLimits(t *testing.T) { ctx := t.Context() rate, _, err := client.RateLimit.Get(ctx) if err != nil { - t.Errorf("RateLimits returned error: %v", err) + t.Errorf("RateLimit.Get returned error: %v", err) } want := &RateLimits{ @@ -245,7 +245,7 @@ func TestRateLimits_overQuota(t *testing.T) { ctx := t.Context() rate, _, err := client.RateLimit.Get(ctx) if err != nil { - t.Errorf("RateLimits returned error: %v", err) + t.Errorf("RateLimit.Get returned error: %v", err) } want := &RateLimits{ @@ -418,7 +418,7 @@ func TestRateLimits_bypassRateLimitCheckContext(t *testing.T) { _, resp, err := client.RateLimit.Get(t.Context()) if err != nil { - t.Errorf("RateLimits returned error: %v", err) + t.Errorf("RateLimit.Get returned error: %v", err) } if got, want := resp.Request.Context().Value(BypassRateLimitCheck), tt.wantBypassValue; got != want { diff --git a/github/reactions_test.go b/github/reactions_test.go index 559657aa25e..cd3a32b8d35 100644 --- a/github/reactions_test.go +++ b/github/reactions_test.go @@ -503,7 +503,7 @@ func TestReactionService_CreateTeamDiscussionCommentReaction(t *testing.T) { }) } -func TestReactionsService_DeleteCommitCommentReaction(t *testing.T) { +func TestReactionsService_DeleteCommentReaction(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -530,7 +530,7 @@ func TestReactionsService_DeleteCommitCommentReaction(t *testing.T) { }) } -func TestReactionsService_DeleteCommitCommentReactionByRepoID(t *testing.T) { +func TestReactionsService_DeleteCommentReactionByID(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -543,7 +543,7 @@ func TestReactionsService_DeleteCommitCommentReactionByRepoID(t *testing.T) { ctx := t.Context() if _, err := client.Reactions.DeleteCommentReactionByID(ctx, 1, 2, 3); err != nil { - t.Errorf("DeleteCommentReactionByRepoID returned error: %v", err) + t.Errorf("Reactions.DeleteCommentReactionByID returned error: %v", err) } const methodName = "DeleteCommentReactionByID" @@ -597,7 +597,7 @@ func TestReactionsService_DeleteIssueReactionByRepoID(t *testing.T) { ctx := t.Context() if _, err := client.Reactions.DeleteIssueReactionByID(ctx, 1, 2, 3); err != nil { - t.Errorf("DeleteIssueReactionByRepoID returned error: %v", err) + t.Errorf("Reactions.DeleteIssueReactionByID returned error: %v", err) } const methodName = "DeleteIssueReactionByID" @@ -638,7 +638,7 @@ func TestReactionsService_DeleteIssueCommentReaction(t *testing.T) { }) } -func TestReactionsService_DeleteIssueCommentReactionByRepoID(t *testing.T) { +func TestReactionsService_DeleteIssueCommentReactionByID(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -651,7 +651,7 @@ func TestReactionsService_DeleteIssueCommentReactionByRepoID(t *testing.T) { ctx := t.Context() if _, err := client.Reactions.DeleteIssueCommentReactionByID(ctx, 1, 2, 3); err != nil { - t.Errorf("DeleteIssueCommentReactionByRepoID returned error: %v", err) + t.Errorf("Reactions.DeleteIssueCommentReactionByID returned error: %v", err) } const methodName = "DeleteIssueCommentReactionByID" @@ -692,7 +692,7 @@ func TestReactionsService_DeletePullRequestCommentReaction(t *testing.T) { }) } -func TestReactionsService_DeletePullRequestCommentReactionByRepoID(t *testing.T) { +func TestReactionsService_DeletePullRequestCommentReactionByID(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -705,7 +705,7 @@ func TestReactionsService_DeletePullRequestCommentReactionByRepoID(t *testing.T) ctx := t.Context() if _, err := client.Reactions.DeletePullRequestCommentReactionByID(ctx, 1, 2, 3); err != nil { - t.Errorf("DeletePullRequestCommentReactionByRepoID returned error: %v", err) + t.Errorf("Reactions.DeletePullRequestCommentReactionByID returned error: %v", err) } const methodName = "DeletePullRequestCommentReactionByID" @@ -732,7 +732,7 @@ func TestReactionsService_DeleteTeamDiscussionReaction(t *testing.T) { ctx := t.Context() if _, err := client.Reactions.DeleteTeamDiscussionReaction(ctx, "o", "s", 1, 2); err != nil { - t.Errorf("DeleteTeamDiscussionReaction returned error: %v", err) + t.Errorf("Reactions.DeleteTeamDiscussionReaction returned error: %v", err) } const methodName = "DeleteTeamDiscussionReaction" @@ -759,7 +759,7 @@ func TestReactionsService_DeleteTeamDiscussionReactionByTeamIDAndOrgID(t *testin ctx := t.Context() if _, err := client.Reactions.DeleteTeamDiscussionReactionByOrgIDAndTeamID(ctx, 1, 2, 3, 4); err != nil { - t.Errorf("DeleteTeamDiscussionReactionByTeamIDAndOrgID returned error: %v", err) + t.Errorf("Reactions.DeleteTeamDiscussionReactionByOrgIDAndTeamID returned error: %v", err) } const methodName = "DeleteTeamDiscussionReactionByOrgIDAndTeamID" @@ -786,7 +786,7 @@ func TestReactionsService_DeleteTeamDiscussionCommentReaction(t *testing.T) { ctx := t.Context() if _, err := client.Reactions.DeleteTeamDiscussionCommentReaction(ctx, "o", "s", 1, 2, 3); err != nil { - t.Errorf("DeleteTeamDiscussionCommentReaction returned error: %v", err) + t.Errorf("Reactions.DeleteTeamDiscussionCommentReaction returned error: %v", err) } const methodName = "DeleteTeamDiscussionCommentReaction" @@ -800,7 +800,7 @@ func TestReactionsService_DeleteTeamDiscussionCommentReaction(t *testing.T) { }) } -func TestReactionsService_DeleteTeamDiscussionCommentReactionByTeamIDAndOrgID(t *testing.T) { +func TestReactionsService_DeleteTeamDiscussionCommentReactionByOrgIDAndTeamID(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -813,7 +813,7 @@ func TestReactionsService_DeleteTeamDiscussionCommentReactionByTeamIDAndOrgID(t ctx := t.Context() if _, err := client.Reactions.DeleteTeamDiscussionCommentReactionByOrgIDAndTeamID(ctx, 1, 2, 3, 4, 5); err != nil { - t.Errorf("DeleteTeamDiscussionCommentReactionByTeamIDAndOrgID returned error: %v", err) + t.Errorf("Reactions.DeleteTeamDiscussionCommentReactionByOrgIDAndTeamID returned error: %v", err) } const methodName = "DeleteTeamDiscussionCommentReactionByOrgIDAndTeamID" @@ -951,7 +951,7 @@ func TestReactionsService_DeleteReleaseReactionByRepoID(t *testing.T) { ctx := t.Context() if _, err := client.Reactions.DeleteReleaseReactionByID(ctx, 1, 2, 3); err != nil { - t.Errorf("DeleteReleaseReactionByRepoID returned error: %v", err) + t.Errorf("Reactions.DeleteReleaseReactionByID returned error: %v", err) } const methodName = "DeleteReleaseReactionByID" diff --git a/github/repos_actions_allowed_test.go b/github/repos_actions_allowed_test.go index 317760777e5..0882ea95335 100644 --- a/github/repos_actions_allowed_test.go +++ b/github/repos_actions_allowed_test.go @@ -47,7 +47,7 @@ func TestRepositoryService_GetActionsAllowed(t *testing.T) { }) } -func TestRepositoriesService_UpdateActionsAllowed(t *testing.T) { +func TestRepositoriesService_EditActionsAllowed(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -62,15 +62,15 @@ func TestRepositoriesService_UpdateActionsAllowed(t *testing.T) { ctx := t.Context() org, _, err := client.Repositories.EditActionsAllowed(ctx, "o", "r", *input) if err != nil { - t.Errorf("Repositories.UpdateActionsAllowed returned error: %v", err) + t.Errorf("Repositories.EditActionsAllowed returned error: %v", err) } want := &ActionsAllowed{GithubOwnedAllowed: Ptr(true), VerifiedAllowed: Ptr(false), PatternsAllowed: []string{"a/b"}} if !cmp.Equal(org, want) { - t.Errorf("Repositories.UpdateActionsAllowed returned %+v, want %+v", org, want) + t.Errorf("Repositories.EditActionsAllowed returned %+v, want %+v", org, want) } - const methodName = "UpdateActionsAllowed" + const methodName = "EditActionsAllowed" testBadOptions(t, methodName, func() (err error) { _, _, err = client.Repositories.EditActionsAllowed(ctx, "\n", "\n", *input) return err diff --git a/github/repos_commits_test.go b/github/repos_commits_test.go index a7370f39403..a6b64b3a7f3 100644 --- a/github/repos_commits_test.go +++ b/github/repos_commits_test.go @@ -546,11 +546,11 @@ func TestRepositoriesService_CompareCommitsRaw_diff(t *testing.T) { ctx := t.Context() got, _, err := client.Repositories.CompareCommitsRaw(ctx, "o", "r", base, head, RawOptions{Type: Diff}) if err != nil { - t.Fatalf("Repositories.GetCommitRaw returned error: %v", err) + t.Fatalf("Repositories.CompareCommitsRaw returned error: %v", err) } want := rawStr if got != want { - t.Errorf("Repositories.GetCommitRaw returned %v want %v", got, want) + t.Errorf("Repositories.CompareCommitsRaw returned %v want %v", got, want) } const methodName = "CompareCommitsRaw" @@ -604,11 +604,11 @@ func TestRepositoriesService_CompareCommitsRaw_patch(t *testing.T) { ctx := t.Context() got, _, err := client.Repositories.CompareCommitsRaw(ctx, "o", "r", base, head, RawOptions{Type: Patch}) if err != nil { - t.Fatalf("Repositories.GetCommitRaw returned error: %v", err) + t.Fatalf("Repositories.CompareCommitsRaw returned error: %v", err) } want := rawStr if got != want { - t.Errorf("Repositories.GetCommitRaw returned %v want %v", got, want) + t.Errorf("Repositories.CompareCommitsRaw returned %v want %v", got, want) } }) } diff --git a/github/repos_keys_test.go b/github/repos_keys_test.go index 7f51b568148..cee855a828e 100644 --- a/github/repos_keys_test.go +++ b/github/repos_keys_test.go @@ -118,12 +118,12 @@ func TestRepositoriesService_CreateKey(t *testing.T) { ctx := t.Context() key, _, err := client.Repositories.CreateKey(ctx, "o", "r", input) if err != nil { - t.Errorf("Repositories.GetKey returned error: %v", err) + t.Errorf("Repositories.CreateKey returned error: %v", err) } want := &Key{ID: Ptr(int64(1))} if !cmp.Equal(key, want) { - t.Errorf("Repositories.GetKey returned %+v, want %+v", key, want) + t.Errorf("Repositories.CreateKey returned %+v, want %+v", key, want) } const methodName = "CreateKey" diff --git a/github/repos_prereceive_hooks_test.go b/github/repos_prereceive_hooks_test.go index d44758e16e6..07d77d28141 100644 --- a/github/repos_prereceive_hooks_test.go +++ b/github/repos_prereceive_hooks_test.go @@ -29,7 +29,7 @@ func TestRepositoriesService_ListPreReceiveHooks(t *testing.T) { ctx := t.Context() hooks, _, err := client.Repositories.ListPreReceiveHooks(ctx, "o", "r", opt) if err != nil { - t.Errorf("Repositories.ListHooks returned error: %v", err) + t.Errorf("Repositories.ListPreReceiveHooks returned error: %v", err) } want := []*PreReceiveHook{{ID: Ptr(int64(1))}, {ID: Ptr(int64(2))}} diff --git a/github/repos_releases_test.go b/github/repos_releases_test.go index 4e31d40f3e6..8abbef29b51 100644 --- a/github/repos_releases_test.go +++ b/github/repos_releases_test.go @@ -747,11 +747,11 @@ func TestRepositoriesService_UploadReleaseAsset(t *testing.T) { ctx := t.Context() asset, _, err := client.Repositories.UploadReleaseAsset(ctx, "o", "r", int64(key), test.uploadOpts, file) if err != nil { - t.Errorf("Repositories.UploadReleaseAssert returned error: %v", err) + t.Errorf("Repositories.UploadReleaseAsset returned error: %v", err) } want := &ReleaseAsset{ID: Ptr(int64(1))} if !cmp.Equal(asset, want) { - t.Errorf("Repositories.UploadReleaseAssert returned %+v, want %+v", asset, want) + t.Errorf("Repositories.UploadReleaseAsset returned %+v, want %+v", asset, want) } const methodName = "UploadReleaseAsset" diff --git a/github/repos_stats_test.go b/github/repos_stats_test.go index 7a3155547dd..3968f591e5c 100644 --- a/github/repos_stats_test.go +++ b/github/repos_stats_test.go @@ -288,7 +288,7 @@ func TestRepositoriesService_ListPunchCard(t *testing.T) { }) } -func TestRepositoriesService_AcceptedError(t *testing.T) { +func TestRepositoriesService_ListContributorsStats_AcceptedError(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -302,15 +302,15 @@ func TestRepositoriesService_AcceptedError(t *testing.T) { ctx := t.Context() stats, _, err := client.Repositories.ListContributorsStats(ctx, "o", "r") if err == nil { - t.Error("RepositoriesService.AcceptedError should have returned an error") + t.Error("Repositories.ListContributorsStats should have returned an error") } if !errors.As(err, new(*AcceptedError)) { - t.Errorf("RepositoriesService.AcceptedError returned an AcceptedError: %v", err) + t.Errorf("Repositories.ListContributorsStats returned an AcceptedError: %v", err) } if stats != nil { - t.Errorf("RepositoriesService.AcceptedError expected stats to be nil: %v", stats) + t.Errorf("Repositories.ListContributorsStats expected stats to be nil: %v", stats) } const methodName = "ListContributorsStats" diff --git a/github/repos_test.go b/github/repos_test.go index ab8275831c2..123e4b575f1 100644 --- a/github/repos_test.go +++ b/github/repos_test.go @@ -30,7 +30,7 @@ func TestRepositoriesService_ListByAuthenticatedUser(t *testing.T) { ctx := t.Context() got, _, err := client.Repositories.ListByAuthenticatedUser(ctx, nil) if err != nil { - t.Errorf("Repositories.List returned error: %v", err) + t.Errorf("Repositories.ListByAuthenticatedUser returned error: %v", err) } want := []*Repository{{ID: Ptr(int64(1))}, {ID: Ptr(int64(2))}} @@ -71,7 +71,7 @@ func TestRepositoriesService_ListByUser(t *testing.T) { ctx := t.Context() repos, _, err := client.Repositories.ListByUser(ctx, "u", opt) if err != nil { - t.Errorf("Repositories.List returned error: %v", err) + t.Errorf("Repositories.ListByUser returned error: %v", err) } want := []*Repository{{ID: Ptr(int64(1))}} diff --git a/github/search_test.go b/github/search_test.go index 4e9f58c574c..72f3a3f56dc 100644 --- a/github/search_test.go +++ b/github/search_test.go @@ -126,7 +126,7 @@ func TestSearchService_RepositoriesTextMatch(t *testing.T) { ctx := t.Context() result, _, err := client.Search.Repositories(ctx, "blah", opts) if err != nil { - t.Errorf("Search.Code returned error: %v", err) + t.Errorf("Search.Repositories returned error: %v", err) } wantedRepoResult := &Repository{ @@ -443,7 +443,7 @@ func TestSearchService_Users(t *testing.T) { ctx := t.Context() result, _, err := client.Search.Users(ctx, "blah", opts) if err != nil { - t.Errorf("Search.Issues returned error: %v", err) + t.Errorf("Search.Users returned error: %v", err) } want := &UsersSearchResult{ @@ -617,7 +617,7 @@ func TestSearchService_Labels(t *testing.T) { ctx := t.Context() result, _, err := client.Search.Labels(ctx, 1234, "blah", opts) if err != nil { - t.Errorf("Search.Code returned error: %v", err) + t.Errorf("Search.Labels returned error: %v", err) } want := &LabelsSearchResult{ diff --git a/github/teams_members_test.go b/github/teams_members_test.go index 3224ade291d..665b073cfe0 100644 --- a/github/teams_members_test.go +++ b/github/teams_members_test.go @@ -659,10 +659,10 @@ func TestTeamsService__ListPendingTeamInvitationsByID_notFound(t *testing.T) { t.Error("Expected HTTP 404 response") } if got, want := resp.Response.StatusCode, http.StatusNotFound; got != want { - t.Errorf("Teams.RemoveTeamMembershipByID returned status %v, want %v", got, want) + t.Errorf("Teams.ListPendingTeamInvitationsByID returned status %v, want %v", got, want) } if invitations != nil { - t.Errorf("Teams.RemoveTeamMembershipByID returned %+v, want nil", invitations) + t.Errorf("Teams.ListPendingTeamInvitationsByID returned %+v, want nil", invitations) } const methodName = "ListPendingTeamInvitationsByID" @@ -694,12 +694,12 @@ func TestTeamsService__ListPendingTeamInvitationsBySlug(t *testing.T) { ctx := t.Context() invitations, _, err := client.Teams.ListPendingTeamInvitationsBySlug(ctx, "o", "s", opt) if err != nil { - t.Errorf("Teams.ListPendingTeamInvitationsByID returned error: %v", err) + t.Errorf("Teams.ListPendingTeamInvitationsBySlug returned error: %v", err) } want := []*Invitation{{ID: Ptr(int64(1))}} if !cmp.Equal(invitations, want) { - t.Errorf("Teams.ListPendingTeamInvitationsByID returned %+v, want %+v", invitations, want) + t.Errorf("Teams.ListPendingTeamInvitationsBySlug returned %+v, want %+v", invitations, want) } const methodName = "ListPendingTeamInvitationsBySlug" @@ -734,10 +734,10 @@ func TestTeamsService__ListPendingTeamInvitationsBySlug_notFound(t *testing.T) { t.Error("Expected HTTP 404 response") } if got, want := resp.Response.StatusCode, http.StatusNotFound; got != want { - t.Errorf("Teams.RemoveTeamMembershipByID returned status %v, want %v", got, want) + t.Errorf("Teams.ListPendingTeamInvitationsBySlug returned status %v, want %v", got, want) } if invitations != nil { - t.Errorf("Teams.RemoveTeamMembershipByID returned %+v, want nil", invitations) + t.Errorf("Teams.ListPendingTeamInvitationsBySlug returned %+v, want nil", invitations) } const methodName = "ListPendingTeamInvitationsBySlug" diff --git a/github/teams_test.go b/github/teams_test.go index dc3493fbd6a..752072c7a26 100644 --- a/github/teams_test.go +++ b/github/teams_test.go @@ -664,10 +664,10 @@ func TestTeamsService_IsTeamRepoBySlug_false(t *testing.T) { t.Error("Expected HTTP 404 response") } if got, want := resp.Response.StatusCode, http.StatusNotFound; got != want { - t.Errorf("Teams.IsTeamRepoByID returned status %v, want %v", got, want) + t.Errorf("Teams.IsTeamRepoBySlug returned status %v, want %v", got, want) } if repo != nil { - t.Errorf("Teams.IsTeamRepoByID returned %+v, want nil", repo) + t.Errorf("Teams.IsTeamRepoBySlug returned %+v, want nil", repo) } } @@ -1924,6 +1924,6 @@ func TestTeamsService_RemoveConnectedExternalGroup_notFound(t *testing.T) { t.Error("Expected HTTP 404 response") } if got, want := resp.Response.StatusCode, http.StatusNotFound; got != want { - t.Errorf("Teams.GetExternalGroup returned status %v, want %v", got, want) + t.Errorf("Teams.RemoveConnectedExternalGroup returned status %v, want %v", got, want) } } diff --git a/github/users_followers_test.go b/github/users_followers_test.go index 289f82c313d..21e42aab34c 100644 --- a/github/users_followers_test.go +++ b/github/users_followers_test.go @@ -361,7 +361,7 @@ func TestUsersService_Unfollow(t *testing.T) { ctx := t.Context() _, err := client.Users.Unfollow(ctx, "u") if err != nil { - t.Errorf("Users.Follow returned error: %v", err) + t.Errorf("Users.Unfollow returned error: %v", err) } const methodName = "Unfollow" diff --git a/github/users_gpg_keys_test.go b/github/users_gpg_keys_test.go index 82128a438cf..0c87f31f761 100644 --- a/github/users_gpg_keys_test.go +++ b/github/users_gpg_keys_test.go @@ -143,7 +143,7 @@ mQINBFcEd9kBEACo54TDbGhKlXKWMvJgecEUKPPcv7XdnpKdGb3LRw5MvFwT0V0f ctx := t.Context() gpgKey, _, err := client.Users.CreateGPGKey(ctx, input) if err != nil { - t.Errorf("Users.GetGPGKey returned error: %v", err) + t.Errorf("Users.CreateGPGKey returned error: %v", err) } want := &GPGKey{ID: Ptr(int64(1))} diff --git a/github/users_test.go b/github/users_test.go index 4f9189c5625..41f5d10349a 100644 --- a/github/users_test.go +++ b/github/users_test.go @@ -197,7 +197,7 @@ func TestUsersService_ListAll(t *testing.T) { ctx := t.Context() users, _, err := client.Users.ListAll(ctx, opt) if err != nil { - t.Errorf("Users.Get returned error: %v", err) + t.Errorf("Users.ListAll returned error: %v", err) } want := []*User{{ID: Ptr(int64(2))}} From ade00cf038e18ea106028a6e67311115f0cb84fb Mon Sep 17 00:00:00 2001 From: Oleksandr Redko Date: Fri, 5 Jun 2026 16:46:28 +0300 Subject: [PATCH 2/2] fix duplicated __ --- github/teams_members_test.go | 50 ++++++++++++++++++------------------ 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/github/teams_members_test.go b/github/teams_members_test.go index 665b073cfe0..72f775e5028 100644 --- a/github/teams_members_test.go +++ b/github/teams_members_test.go @@ -13,7 +13,7 @@ import ( "github.com/google/go-cmp/cmp" ) -func TestTeamsService__ListTeamMembersByID(t *testing.T) { +func TestTeamsService_ListTeamMembersByID(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -50,7 +50,7 @@ func TestTeamsService__ListTeamMembersByID(t *testing.T) { }) } -func TestTeamsService__ListTeamMembersByID_notFound(t *testing.T) { +func TestTeamsService_ListTeamMembersByID_notFound(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -88,7 +88,7 @@ func TestTeamsService__ListTeamMembersByID_notFound(t *testing.T) { }) } -func TestTeamsService__ListTeamMembersBySlug(t *testing.T) { +func TestTeamsService_ListTeamMembersBySlug(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -125,7 +125,7 @@ func TestTeamsService__ListTeamMembersBySlug(t *testing.T) { }) } -func TestTeamsService__ListTeamMembersBySlug_notFound(t *testing.T) { +func TestTeamsService_ListTeamMembersBySlug_notFound(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -163,7 +163,7 @@ func TestTeamsService__ListTeamMembersBySlug_notFound(t *testing.T) { }) } -func TestTeamsService__ListTeamMembersBySlug_invalidOrg(t *testing.T) { +func TestTeamsService_ListTeamMembersBySlug_invalidOrg(t *testing.T) { t.Parallel() client, _, _ := setup(t) @@ -172,7 +172,7 @@ func TestTeamsService__ListTeamMembersBySlug_invalidOrg(t *testing.T) { testURLParseError(t, err) } -func TestTeamsService__GetTeamMembershipByID(t *testing.T) { +func TestTeamsService_GetTeamMembershipByID(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -207,7 +207,7 @@ func TestTeamsService__GetTeamMembershipByID(t *testing.T) { }) } -func TestTeamsService__GetTeamMembershipByID_notFound(t *testing.T) { +func TestTeamsService_GetTeamMembershipByID_notFound(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -243,7 +243,7 @@ func TestTeamsService__GetTeamMembershipByID_notFound(t *testing.T) { }) } -func TestTeamsService__GetTeamMembershipBySlug(t *testing.T) { +func TestTeamsService_GetTeamMembershipBySlug(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -278,7 +278,7 @@ func TestTeamsService__GetTeamMembershipBySlug(t *testing.T) { }) } -func TestTeamsService__GetTeamMembershipBySlug_notFound(t *testing.T) { +func TestTeamsService_GetTeamMembershipBySlug_notFound(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -314,7 +314,7 @@ func TestTeamsService__GetTeamMembershipBySlug_notFound(t *testing.T) { }) } -func TestTeamsService__GetTeamMembershipBySlug_invalidOrg(t *testing.T) { +func TestTeamsService_GetTeamMembershipBySlug_invalidOrg(t *testing.T) { t.Parallel() client, _, _ := setup(t) @@ -323,7 +323,7 @@ func TestTeamsService__GetTeamMembershipBySlug_invalidOrg(t *testing.T) { testURLParseError(t, err) } -func TestTeamsService__AddTeamMembershipByID(t *testing.T) { +func TestTeamsService_AddTeamMembershipByID(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -361,7 +361,7 @@ func TestTeamsService__AddTeamMembershipByID(t *testing.T) { }) } -func TestTeamsService__AddTeamMembershipByID_notFound(t *testing.T) { +func TestTeamsService_AddTeamMembershipByID_notFound(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -400,7 +400,7 @@ func TestTeamsService__AddTeamMembershipByID_notFound(t *testing.T) { }) } -func TestTeamsService__AddTeamMembershipBySlug(t *testing.T) { +func TestTeamsService_AddTeamMembershipBySlug(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -438,7 +438,7 @@ func TestTeamsService__AddTeamMembershipBySlug(t *testing.T) { }) } -func TestTeamsService__AddTeamMembershipBySlug_notFound(t *testing.T) { +func TestTeamsService_AddTeamMembershipBySlug_notFound(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -477,7 +477,7 @@ func TestTeamsService__AddTeamMembershipBySlug_notFound(t *testing.T) { }) } -func TestTeamsService__AddTeamMembershipBySlug_invalidOrg(t *testing.T) { +func TestTeamsService_AddTeamMembershipBySlug_invalidOrg(t *testing.T) { t.Parallel() client, _, _ := setup(t) @@ -486,7 +486,7 @@ func TestTeamsService__AddTeamMembershipBySlug_invalidOrg(t *testing.T) { testURLParseError(t, err) } -func TestTeamsService__RemoveTeamMembershipByID(t *testing.T) { +func TestTeamsService_RemoveTeamMembershipByID(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -512,7 +512,7 @@ func TestTeamsService__RemoveTeamMembershipByID(t *testing.T) { }) } -func TestTeamsService__RemoveTeamMembershipByID_notFound(t *testing.T) { +func TestTeamsService_RemoveTeamMembershipByID_notFound(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -541,7 +541,7 @@ func TestTeamsService__RemoveTeamMembershipByID_notFound(t *testing.T) { }) } -func TestTeamsService__RemoveTeamMembershipBySlug(t *testing.T) { +func TestTeamsService_RemoveTeamMembershipBySlug(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -567,7 +567,7 @@ func TestTeamsService__RemoveTeamMembershipBySlug(t *testing.T) { }) } -func TestTeamsService__RemoveTeamMembershipBySlug_notFound(t *testing.T) { +func TestTeamsService_RemoveTeamMembershipBySlug_notFound(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -596,7 +596,7 @@ func TestTeamsService__RemoveTeamMembershipBySlug_notFound(t *testing.T) { }) } -func TestTeamsService__RemoveTeamMembershipBySlug_invalidOrg(t *testing.T) { +func TestTeamsService_RemoveTeamMembershipBySlug_invalidOrg(t *testing.T) { t.Parallel() client, _, _ := setup(t) @@ -605,7 +605,7 @@ func TestTeamsService__RemoveTeamMembershipBySlug_invalidOrg(t *testing.T) { testURLParseError(t, err) } -func TestTeamsService__ListPendingTeamInvitationsByID(t *testing.T) { +func TestTeamsService_ListPendingTeamInvitationsByID(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -642,7 +642,7 @@ func TestTeamsService__ListPendingTeamInvitationsByID(t *testing.T) { }) } -func TestTeamsService__ListPendingTeamInvitationsByID_notFound(t *testing.T) { +func TestTeamsService_ListPendingTeamInvitationsByID_notFound(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -680,7 +680,7 @@ func TestTeamsService__ListPendingTeamInvitationsByID_notFound(t *testing.T) { }) } -func TestTeamsService__ListPendingTeamInvitationsBySlug(t *testing.T) { +func TestTeamsService_ListPendingTeamInvitationsBySlug(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -717,7 +717,7 @@ func TestTeamsService__ListPendingTeamInvitationsBySlug(t *testing.T) { }) } -func TestTeamsService__ListPendingTeamInvitationsBySlug_notFound(t *testing.T) { +func TestTeamsService_ListPendingTeamInvitationsBySlug_notFound(t *testing.T) { t.Parallel() client, mux, _ := setup(t) @@ -755,7 +755,7 @@ func TestTeamsService__ListPendingTeamInvitationsBySlug_notFound(t *testing.T) { }) } -func TestTeamsService__ListPendingTeamInvitationsBySlug_invalidOrg(t *testing.T) { +func TestTeamsService_ListPendingTeamInvitationsBySlug_invalidOrg(t *testing.T) { t.Parallel() client, _, _ := setup(t)