diff --git a/github/activity_events_test.go b/github/activity_events_test.go index 45399f2..1541f5e 100644 --- a/github/activity_events_test.go +++ b/github/activity_events_test.go @@ -61,6 +61,11 @@ func TestActivityService_ListRepositoryEvents(t *testing.T) { } } +func TestActivityService_ListRepositoryEvents_invalidOwner(t *testing.T) { + _, _, err := client.Activity.ListRepositoryEvents("%", "%", nil) + testURLParseError(t, err) +} + func TestActivityService_ListIssueEventsForRepository(t *testing.T) { setup() defer teardown() @@ -85,6 +90,11 @@ func TestActivityService_ListIssueEventsForRepository(t *testing.T) { } } +func TestActivityService_ListIssueEventsForRepository_invalidOwner(t *testing.T) { + _, _, err := client.Activity.ListIssueEventsForRepository("%", "%", nil) + testURLParseError(t, err) +} + func TestActivityService_ListEventsForRepoNetwork(t *testing.T) { setup() defer teardown() @@ -109,6 +119,11 @@ func TestActivityService_ListEventsForRepoNetwork(t *testing.T) { } } +func TestActivityService_ListEventsForRepoNetwork_invalidOwner(t *testing.T) { + _, _, err := client.Activity.ListEventsForRepoNetwork("%", "%", nil) + testURLParseError(t, err) +} + func TestActivityService_ListEventsForOrganization(t *testing.T) { setup() defer teardown() @@ -133,6 +148,11 @@ func TestActivityService_ListEventsForOrganization(t *testing.T) { } } +func TestActivityService_ListEventsForOrganization_invalidOrg(t *testing.T) { + _, _, err := client.Activity.ListEventsForOrganization("%", nil) + testURLParseError(t, err) +} + func TestActivityService_ListEventsPerformedByUser_all(t *testing.T) { setup() defer teardown() @@ -177,6 +197,11 @@ func TestActivityService_ListEventsPerformedByUser_publicOnly(t *testing.T) { } } +func TestActivityService_ListEventsPerformedByUser_invalidUser(t *testing.T) { + _, _, err := client.Activity.ListEventsPerformedByUser("%", false, nil) + testURLParseError(t, err) +} + func TestActivityService_ListEventsRecievedByUser_all(t *testing.T) { setup() defer teardown() @@ -221,16 +246,25 @@ func TestActivityService_ListEventsRecievedByUser_publicOnly(t *testing.T) { } } +func TestActivityService_ListEventsRecievedByUser_invalidUser(t *testing.T) { + _, _, err := client.Activity.ListEventsRecievedByUser("%", false, nil) + testURLParseError(t, err) +} + func TestActivityService_ListUserEventsForOrganization(t *testing.T) { setup() defer teardown() mux.HandleFunc("/users/u/events/orgs/o", func(w http.ResponseWriter, r *http.Request) { testMethod(t, r, "GET") + testFormValues(t, r, values{ + "page": "2", + }) fmt.Fprint(w, `[{"id":"1"},{"id":"2"}]`) }) - events, _, err := client.Activity.ListUserEventsForOrganization("o", "u", nil) + opt := &ListOptions{Page: 2} + events, _, err := client.Activity.ListUserEventsForOrganization("o", "u", opt) if err != nil { t.Errorf("Activities.ListUserEventsForOrganization returned error: %v", err) } diff --git a/github/activity_star_test.go b/github/activity_star_test.go index 02586a1..b1179ec 100644 --- a/github/activity_star_test.go +++ b/github/activity_star_test.go @@ -57,3 +57,8 @@ func TestActivityService_ListStarred_specifiedUser(t *testing.T) { t.Errorf("Activity.ListStarred returned %+v, want %+v", repos, want) } } + +func TestActivityService_ListStarred_invalidUser(t *testing.T) { + _, _, err := client.Activity.ListStarred("%", nil) + testURLParseError(t, err) +} diff --git a/github/gists_comments_test.go b/github/gists_comments_test.go index d10ff1c..a1040b2 100644 --- a/github/gists_comments_test.go +++ b/github/gists_comments_test.go @@ -34,6 +34,11 @@ func TestGistsService_ListComments(t *testing.T) { } } +func TestGistsService_ListComments_invalidID(t *testing.T) { + _, _, err := client.Gists.ListComments("%") + testURLParseError(t, err) +} + func TestGistsService_GetComment(t *testing.T) { setup() defer teardown() @@ -55,6 +60,11 @@ func TestGistsService_GetComment(t *testing.T) { } } +func TestGistsService_GetComment_invalidID(t *testing.T) { + _, _, err := client.Gists.GetComment("%", 1) + testURLParseError(t, err) +} + func TestGistsService_CreateComment(t *testing.T) { setup() defer teardown() @@ -84,6 +94,11 @@ func TestGistsService_CreateComment(t *testing.T) { } } +func TestGistsService_CreateComment_invalidID(t *testing.T) { + _, _, err := client.Gists.CreateComment("%", nil) + testURLParseError(t, err) +} + func TestGistsService_EditComment(t *testing.T) { setup() defer teardown() @@ -113,6 +128,11 @@ func TestGistsService_EditComment(t *testing.T) { } } +func TestGistsService_EditComment_invalidID(t *testing.T) { + _, _, err := client.Gists.EditComment("%", 1, nil) + testURLParseError(t, err) +} + func TestGistsService_DeleteComment(t *testing.T) { setup() defer teardown() @@ -126,3 +146,8 @@ func TestGistsService_DeleteComment(t *testing.T) { t.Errorf("Gists.Delete returned error: %v", err) } } + +func TestGistsService_DeleteComment_invalidID(t *testing.T) { + _, err := client.Gists.DeleteComment("%", 1) + testURLParseError(t, err) +} diff --git a/github/gists_test.go b/github/gists_test.go index 31d5013..7569cc6 100644 --- a/github/gists_test.go +++ b/github/gists_test.go @@ -14,7 +14,7 @@ import ( "time" ) -func TestGistsService_List(t *testing.T) { +func TestGistsService_List_specifiedUser(t *testing.T) { setup() defer teardown() @@ -41,7 +41,7 @@ func TestGistsService_List(t *testing.T) { } } -func TestGistsService_List_withEmptyUser(t *testing.T) { +func TestGistsService_List_authenticatedUser(t *testing.T) { setup() defer teardown() @@ -61,6 +61,11 @@ func TestGistsService_List_withEmptyUser(t *testing.T) { } } +func TestGistsService_List_invalidUser(t *testing.T) { + _, _, err := client.Gists.List("%", nil) + testURLParseError(t, err) +} + func TestGistsService_ListAll(t *testing.T) { setup() defer teardown() @@ -136,6 +141,11 @@ func TestGistsService_Get(t *testing.T) { } } +func TestGistsService_Get_invalidID(t *testing.T) { + _, _, err := client.Gists.Get("%") + testURLParseError(t, err) +} + func TestGistsService_Create(t *testing.T) { setup() defer teardown() @@ -245,6 +255,11 @@ func TestGistsService_Edit(t *testing.T) { } } +func TestGistsService_Edit_invalidID(t *testing.T) { + _, _, err := client.Gists.Edit("%", nil) + testURLParseError(t, err) +} + func TestGistsService_Delete(t *testing.T) { setup() defer teardown() @@ -259,6 +274,11 @@ func TestGistsService_Delete(t *testing.T) { } } +func TestGistsService_Delete_invalidID(t *testing.T) { + _, err := client.Gists.Delete("%") + testURLParseError(t, err) +} + func TestGistsService_Star(t *testing.T) { setup() defer teardown() @@ -273,6 +293,11 @@ func TestGistsService_Star(t *testing.T) { } } +func TestGistsService_Star_invalidID(t *testing.T) { + _, err := client.Gists.Star("%") + testURLParseError(t, err) +} + func TestGistsService_Unstar(t *testing.T) { setup() defer teardown() @@ -287,6 +312,11 @@ func TestGistsService_Unstar(t *testing.T) { } } +func TestGistsService_Unstar_invalidID(t *testing.T) { + _, err := client.Gists.Unstar("%") + testURLParseError(t, err) +} + func TestGistsService_IsStarred_hasStar(t *testing.T) { setup() defer teardown() @@ -323,6 +353,11 @@ func TestGistsService_IsStarred_noStar(t *testing.T) { } } +func TestGistsService_IsStarred_invalidID(t *testing.T) { + _, _, err := client.Gists.IsStarred("%") + testURLParseError(t, err) +} + func TestGistsService_Fork(t *testing.T) { setup() defer teardown() @@ -343,3 +378,8 @@ func TestGistsService_Fork(t *testing.T) { t.Errorf("Gists.Fork returned %+v, want %+v", gist, want) } } + +func TestGistsService_Fork_invalidID(t *testing.T) { + _, _, err := client.Gists.Fork("%") + testURLParseError(t, err) +} diff --git a/github/git_commits_test.go b/github/git_commits_test.go index 1e934ec..12a8e22 100644 --- a/github/git_commits_test.go +++ b/github/git_commits_test.go @@ -33,6 +33,11 @@ func TestGitService_GetCommit(t *testing.T) { } } +func TestGitService_GetCommit_invalidOwner(t *testing.T) { + _, _, err := client.Git.GetCommit("%", "%", "%") + testURLParseError(t, err) +} + func TestGitService_CreateCommit(t *testing.T) { setup() defer teardown() @@ -60,3 +65,8 @@ func TestGitService_CreateCommit(t *testing.T) { t.Errorf("Git.CreateCommit returned %+v, want %+v", commit, want) } } + +func TestGitService_CreateCommit_invalidOwner(t *testing.T) { + _, _, err := client.Git.CreateCommit("%", "%", nil) + testURLParseError(t, err) +} diff --git a/github/git_trees_test.go b/github/git_trees_test.go index ec0b278..ce751cf 100644 --- a/github/git_trees_test.go +++ b/github/git_trees_test.go @@ -45,6 +45,11 @@ func TestGitService_GetTree(t *testing.T) { } } +func TestGitService_GetTree_invalidOwner(t *testing.T) { + _, _, err := client.Git.GetTree("%", "%", "%", false) + testURLParseError(t, err) +} + func TestGitService_CreateTree(t *testing.T) { setup() defer teardown() @@ -110,3 +115,8 @@ func TestGitService_CreateTree(t *testing.T) { t.Errorf("Git.CreateTree returned %+v, want %+v", *tree, want) } } + +func TestGitService_CreateTree_invalidOwner(t *testing.T) { + _, _, err := client.Git.CreateTree("%", "%", "", nil) + testURLParseError(t, err) +} diff --git a/github/github_test.go b/github/github_test.go index dbf813d..b5c4827 100644 --- a/github/github_test.go +++ b/github/github_test.go @@ -187,6 +187,18 @@ func TestResponse_populatePageValues_invalid(t *testing.T) { if want, got := 0, response.LastPage; want != got { t.Errorf("response.LastPage: %v, want %v", want, got) } + + // more invalid URLs + r = http.Response{ + Header: http.Header{ + "Link": {`; rel="first"`}, + }, + } + + response = newResponse(&r) + if want, got := 0, response.FirstPage; want != got { + t.Errorf("response.FirstPage: %v, want %v", want, got) + } } func TestDo(t *testing.T) { diff --git a/github/issues_labels_test.go b/github/issues_labels_test.go index f3018a8..9d0a2cb 100644 --- a/github/issues_labels_test.go +++ b/github/issues_labels_test.go @@ -33,6 +33,11 @@ func TestIssuesService_ListLabels(t *testing.T) { } } +func TestIssuesService_ListLabels_invalidOwner(t *testing.T) { + _, _, err := client.Issues.ListLabels("%", "%") + testURLParseError(t, err) +} + func TestIssuesService_GetLabel(t *testing.T) { setup() defer teardown() @@ -53,6 +58,11 @@ func TestIssuesService_GetLabel(t *testing.T) { } } +func TestIssuesService_GetLabel_invalidOwner(t *testing.T) { + _, _, err := client.Issues.GetLabel("%", "%", "%") + testURLParseError(t, err) +} + func TestIssuesService_CreateLabel(t *testing.T) { setup() defer teardown() @@ -82,6 +92,11 @@ func TestIssuesService_CreateLabel(t *testing.T) { } } +func TestIssuesService_CreateLabel_invalidOwner(t *testing.T) { + _, _, err := client.Issues.CreateLabel("%", "%", nil) + testURLParseError(t, err) +} + func TestIssuesService_EditLabel(t *testing.T) { setup() defer teardown() @@ -111,6 +126,11 @@ func TestIssuesService_EditLabel(t *testing.T) { } } +func TestIssuesService_EditLabel_invalidOwner(t *testing.T) { + _, _, err := client.Issues.EditLabel("%", "%", "%", nil) + testURLParseError(t, err) +} + func TestIssuesService_DeleteLabel(t *testing.T) { setup() defer teardown() @@ -125,6 +145,11 @@ func TestIssuesService_DeleteLabel(t *testing.T) { } } +func TestIssuesService_DeleteLabel_invalidOwner(t *testing.T) { + _, err := client.Issues.DeleteLabel("%", "%", "%") + testURLParseError(t, err) +} + func TestIssuesService_ListLabelsByIssue(t *testing.T) { setup() defer teardown() @@ -145,6 +170,11 @@ func TestIssuesService_ListLabelsByIssue(t *testing.T) { } } +func TestIssuesService_ListLabelsByIssue_invalidOwner(t *testing.T) { + _, _, err := client.Issues.ListLabelsByIssue("%", "%", 1) + testURLParseError(t, err) +} + func TestIssuesService_AddLabelsToIssue(t *testing.T) { setup() defer teardown() @@ -174,6 +204,11 @@ func TestIssuesService_AddLabelsToIssue(t *testing.T) { } } +func TestIssuesService_AddLabelsToIssue_invalidOwner(t *testing.T) { + _, _, err := client.Issues.AddLabelsToIssue("%", "%", 1, nil) + testURLParseError(t, err) +} + func TestIssuesService_RemoveLabelForIssue(t *testing.T) { setup() defer teardown() @@ -188,6 +223,11 @@ func TestIssuesService_RemoveLabelForIssue(t *testing.T) { } } +func TestIssuesService_RemoveLabelForIssue_invalidOwner(t *testing.T) { + _, err := client.Issues.RemoveLabelForIssue("%", "%", 1, "%") + testURLParseError(t, err) +} + func TestIssuesService_ReplaceLabelsForIssue(t *testing.T) { setup() defer teardown() @@ -217,6 +257,11 @@ func TestIssuesService_ReplaceLabelsForIssue(t *testing.T) { } } +func TestIssuesService_ReplaceLabelsForIssue_invalidOwner(t *testing.T) { + _, _, err := client.Issues.ReplaceLabelsForIssue("%", "%", 1, nil) + testURLParseError(t, err) +} + func TestIssuesService_RemoveLabelsForIssue(t *testing.T) { setup() defer teardown() @@ -231,6 +276,11 @@ func TestIssuesService_RemoveLabelsForIssue(t *testing.T) { } } +func TestIssuesService_RemoveLabelsForIssue_invalidOwner(t *testing.T) { + _, err := client.Issues.RemoveLabelsForIssue("%", "%", 1) + testURLParseError(t, err) +} + func TestIssuesService_ListLabelsForMilestone(t *testing.T) { setup() defer teardown() @@ -250,3 +300,8 @@ func TestIssuesService_ListLabelsForMilestone(t *testing.T) { t.Errorf("Issues.ListLabelsForMilestone returned %+v, want %+v", labels, want) } } + +func TestIssuesService_ListLabelsForMilestone_invalidOwner(t *testing.T) { + _, _, err := client.Issues.ListLabelsForMilestone("%", "%", 1) + testURLParseError(t, err) +} diff --git a/github/repos_collaborators_test.go b/github/repos_collaborators_test.go index 334f48a..7556d18 100644 --- a/github/repos_collaborators_test.go +++ b/github/repos_collaborators_test.go @@ -32,6 +32,11 @@ func TestRepositoriesService_ListCollaborators(t *testing.T) { } } +func TestRepositoriesService_ListCollaborators_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.ListCollaborators("%", "%") + testURLParseError(t, err) +} + func TestRepositoriesService_IsCollaborator_True(t *testing.T) { setup() defer teardown() @@ -70,6 +75,11 @@ func TestRepositoriesService_IsCollaborator_False(t *testing.T) { } } +func TestRepositoriesService_IsCollaborator_invalidUser(t *testing.T) { + _, _, err := client.Repositories.IsCollaborator("%", "%", "%") + testURLParseError(t, err) +} + func TestRepositoriesService_AddCollaborator(t *testing.T) { setup() defer teardown() @@ -85,6 +95,11 @@ func TestRepositoriesService_AddCollaborator(t *testing.T) { } } +func TestRepositoriesService_AddCollaborator_invalidUser(t *testing.T) { + _, err := client.Repositories.AddCollaborator("%", "%", "%") + testURLParseError(t, err) +} + func TestRepositoriesService_RemoveCollaborator(t *testing.T) { setup() defer teardown() @@ -99,3 +114,8 @@ func TestRepositoriesService_RemoveCollaborator(t *testing.T) { t.Errorf("Repositories.RemoveCollaborator returned error: %v", err) } } + +func TestRepositoriesService_RemoveCollaborator_invalidUser(t *testing.T) { + _, err := client.Repositories.RemoveCollaborator("%", "%", "%") + testURLParseError(t, err) +} diff --git a/github/repos_comments_test.go b/github/repos_comments_test.go index fa4c784..9bcd7a9 100644 --- a/github/repos_comments_test.go +++ b/github/repos_comments_test.go @@ -33,6 +33,11 @@ func TestRepositoriesService_ListComments(t *testing.T) { } } +func TestRepositoriesService_ListComments_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.ListComments("%", "%") + testURLParseError(t, err) +} + func TestRepositoriesService_ListCommitComments(t *testing.T) { setup() defer teardown() @@ -53,6 +58,11 @@ func TestRepositoriesService_ListCommitComments(t *testing.T) { } } +func TestRepositoriesService_ListCommitComments_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.ListCommitComments("%", "%", "%") + testURLParseError(t, err) +} + func TestRepositoriesService_CreateComment(t *testing.T) { setup() defer teardown() @@ -82,6 +92,11 @@ func TestRepositoriesService_CreateComment(t *testing.T) { } } +func TestRepositoriesService_CreateComment_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.CreateComment("%", "%", "%", nil) + testURLParseError(t, err) +} + func TestRepositoriesService_GetComment(t *testing.T) { setup() defer teardown() @@ -102,6 +117,11 @@ func TestRepositoriesService_GetComment(t *testing.T) { } } +func TestRepositoriesService_GetComment_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.GetComment("%", "%", 1) + testURLParseError(t, err) +} + func TestRepositoriesService_UpdateComment(t *testing.T) { setup() defer teardown() @@ -131,6 +151,11 @@ func TestRepositoriesService_UpdateComment(t *testing.T) { } } +func TestRepositoriesService_UpdateComment_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.UpdateComment("%", "%", 1, nil) + testURLParseError(t, err) +} + func TestRepositoriesService_DeleteComment(t *testing.T) { setup() defer teardown() @@ -144,3 +169,8 @@ func TestRepositoriesService_DeleteComment(t *testing.T) { t.Errorf("Repositories.DeleteComment returned error: %v", err) } } + +func TestRepositoriesService_DeleteComment_invalidOwner(t *testing.T) { + _, err := client.Repositories.DeleteComment("%", "%", 1) + testURLParseError(t, err) +} diff --git a/github/repos_hooks_test.go b/github/repos_hooks_test.go index 3839617..4b7a39c 100644 --- a/github/repos_hooks_test.go +++ b/github/repos_hooks_test.go @@ -42,6 +42,11 @@ func TestRepositoriesService_CreateHook(t *testing.T) { } } +func TestRepositoriesService_CreateHook_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.CreateHook("%", "%", nil) + testURLParseError(t, err) +} + func TestRepositoriesService_ListHooks(t *testing.T) { setup() defer teardown() @@ -65,6 +70,11 @@ func TestRepositoriesService_ListHooks(t *testing.T) { } } +func TestRepositoriesService_ListHooks_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.ListHooks("%", "%", nil) + testURLParseError(t, err) +} + func TestRepositoriesService_GetHook(t *testing.T) { setup() defer teardown() @@ -85,6 +95,11 @@ func TestRepositoriesService_GetHook(t *testing.T) { } } +func TestRepositoriesService_GetHook_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.GetHook("%", "%", 1) + testURLParseError(t, err) +} + func TestRepositoriesService_EditHook(t *testing.T) { setup() defer teardown() @@ -114,6 +129,11 @@ func TestRepositoriesService_EditHook(t *testing.T) { } } +func TestRepositoriesService_EditHook_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.EditHook("%", "%", 1, nil) + testURLParseError(t, err) +} + func TestRepositoriesService_DeleteHook(t *testing.T) { setup() defer teardown() @@ -128,6 +148,11 @@ func TestRepositoriesService_DeleteHook(t *testing.T) { } } +func TestRepositoriesService_DeleteHook_invalidOwner(t *testing.T) { + _, err := client.Repositories.DeleteHook("%", "%", 1) + testURLParseError(t, err) +} + func TestRepositoriesService_TestHook(t *testing.T) { setup() defer teardown() @@ -141,3 +166,8 @@ func TestRepositoriesService_TestHook(t *testing.T) { t.Errorf("Repositories.TestHook returned error: %v", err) } } + +func TestRepositoriesService_TestHook_invalidOwner(t *testing.T) { + _, err := client.Repositories.TestHook("%", "%", 1) + testURLParseError(t, err) +} diff --git a/github/repos_keys_test.go b/github/repos_keys_test.go index 043d497..8b5dab0 100644 --- a/github/repos_keys_test.go +++ b/github/repos_keys_test.go @@ -58,6 +58,11 @@ func TestRepositoriesService_GetKey(t *testing.T) { } } +func TestRepositoriesService_GetKey_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.GetKey("%", "%", 1) + testURLParseError(t, err) +} + func TestRepositoriesService_CreateKey(t *testing.T) { setup() defer teardown() @@ -87,6 +92,11 @@ func TestRepositoriesService_CreateKey(t *testing.T) { } } +func TestRepositoriesService_CreateKey_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.CreateKey("%", "%", nil) + testURLParseError(t, err) +} + func TestRepositoriesService_EditKey(t *testing.T) { setup() defer teardown() @@ -116,6 +126,11 @@ func TestRepositoriesService_EditKey(t *testing.T) { } } +func TestRepositoriesService_EditKey_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.EditKey("%", "%", 1, nil) + testURLParseError(t, err) +} + func TestRepositoriesService_DeleteKey(t *testing.T) { setup() defer teardown() @@ -129,3 +144,8 @@ func TestRepositoriesService_DeleteKey(t *testing.T) { t.Errorf("Repositories.DeleteKey returned error: %v", err) } } + +func TestRepositoriesService_DeleteKey_invalidOwner(t *testing.T) { + _, err := client.Repositories.DeleteKey("%", "%", 1) + testURLParseError(t, err) +} diff --git a/github/repos_test.go b/github/repos_test.go index 447ac8d..444a312 100644 --- a/github/repos_test.go +++ b/github/repos_test.go @@ -244,12 +244,12 @@ func TestRepositoriesService_ListLanguages(t *testing.T) { setup() defer teardown() - mux.HandleFunc("/repos/u/r/languages", func(w http.ResponseWriter, r *http.Request) { + mux.HandleFunc("/repos/o/r/languages", func(w http.ResponseWriter, r *http.Request) { testMethod(t, r, "GET") fmt.Fprint(w, `{"go":1}`) }) - languages, _, err := client.Repositories.ListLanguages("u", "r") + languages, _, err := client.Repositories.ListLanguages("o", "r") if err != nil { t.Errorf("Repositories.ListLanguages returned error: %v", err) } @@ -259,3 +259,8 @@ func TestRepositoriesService_ListLanguages(t *testing.T) { t.Errorf("Repositories.ListLanguages returned %+v, want %+v", languages, want) } } + +func TestRepositoriesService_ListLanguages_invalidOwner(t *testing.T) { + _, _, err := client.Repositories.ListLanguages("%", "%") + testURLParseError(t, err) +} diff --git a/github/search_test.go b/github/search_test.go index 64e8265..2ea80bd 100644 --- a/github/search_test.go +++ b/github/search_test.go @@ -139,3 +139,8 @@ func TestSearchService_Code(t *testing.T) { t.Errorf("Search.Code returned %+v, want %+v", result, want) } } + +func TestSearchService_invalidQuery(t *testing.T) { + _, err := client.Search.search("%", "", nil, nil) + testURLParseError(t, err) +}