diff --git a/.golangci.yml b/.golangci.yml
index 1525db660ce..f52016332d2 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -145,6 +145,8 @@ linters:
       checks:
         - "all"
         - "-QF1008" # allow embedded field in selector
+    unparam:
+      check-exported: true  
     usetesting:
       context-background: true
       context-todo: true
diff --git a/github/pulls_test.go b/github/pulls_test.go
index ea724cd5d6e..b34140f8c71 100644
--- a/github/pulls_test.go
+++ b/github/pulls_test.go
@@ -512,6 +512,13 @@ func TestPullRequestsService_Edit(t *testing.T) {
 			return err
 		})
 	}
+	testNewRequestAndDoFailure(t, "Edit", client, func() (*Response, error) {
+		got, resp, err := client.PullRequests.Edit(t.Context(), "o", "r", 1, &PullRequest{})
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", "Edit", got)
+		}
+		return resp, err
+	})
 }
 
 func TestPullRequestsService_Edit_invalidOwner(t *testing.T) {
diff --git a/github/repos_releases_test.go b/github/repos_releases_test.go
index 9eb0f6f03c2..ba73826c8b1 100644
--- a/github/repos_releases_test.go
+++ b/github/repos_releases_test.go
@@ -779,6 +779,13 @@ func TestRepositoriesService_UploadReleaseAsset(t *testing.T) {
 			return err
 		})
 	}
+	testNewRequestAndDoFailure(t, "UploadReleaseAsset", client, func() (*Response, error) {
+		got, resp, err := client.Repositories.UploadReleaseAsset(t.Context(), "o", "r", 1, defaultUploadOptions, openTestFile(t, "upload.txt", "Upload me !\n"))
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure UploadReleaseAsset = %#v, want nil", got)
+		}
+		return resp, err
+	})
 }
 
 func TestRepositoryReleaseRequest_Marshal(t *testing.T) {
diff --git a/github/search_test.go b/github/search_test.go
index 91a2bab401e..52c6f88946e 100644
--- a/github/search_test.go
+++ b/github/search_test.go
@@ -46,6 +46,14 @@ func TestSearchService_Repositories(t *testing.T) {
 	if !cmp.Equal(result, want) {
 		t.Errorf("Search.Repositories returned %+v, want %+v", result, want)
 	}
+	const methodName = "Repositories"
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.Search.Repositories(ctx, "blah", opts)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestSearchService_Repositories_coverage(t *testing.T) {
@@ -171,6 +179,14 @@ func TestSearchService_Topics(t *testing.T) {
 	if !cmp.Equal(result, want) {
 		t.Errorf("Search.Topics returned %+v, want %+v", result, want)
 	}
+	const methodName = "Topics"
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.Search.Topics(ctx, "blah", opts)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestSearchService_Topics_coverage(t *testing.T) {
@@ -216,6 +232,14 @@ func TestSearchService_Commits(t *testing.T) {
 	if !cmp.Equal(result, want) {
 		t.Errorf("Search.Commits returned %+v, want %+v", result, want)
 	}
+	const methodName = "Commits"
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.Search.Commits(ctx, "blah", opts)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestSearchService_Commits_coverage(t *testing.T) {
@@ -263,6 +287,14 @@ func TestSearchService_Issues(t *testing.T) {
 	if !cmp.Equal(result, want) {
 		t.Errorf("Search.Issues returned %+v, want %+v", result, want)
 	}
+	const methodName = "Issues"
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.Search.Issues(ctx, "blah", opts)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestSearchService_Issues_advancedSearch(t *testing.T) {
@@ -422,6 +454,14 @@ func TestSearchService_Users(t *testing.T) {
 	if !cmp.Equal(result, want) {
 		t.Errorf("Search.Users returned %+v, want %+v", result, want)
 	}
+	const methodName = "Users"
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.Search.Users(ctx, "blah", opts)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestSearchService_Users_coverage(t *testing.T) {
@@ -469,6 +509,14 @@ func TestSearchService_Code(t *testing.T) {
 	if !cmp.Equal(result, want) {
 		t.Errorf("Search.Code returned %+v, want %+v", result, want)
 	}
+	const methodName = "Code"
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.Search.Code(ctx, "blah", opts)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestSearchService_Code_coverage(t *testing.T) {
@@ -583,6 +631,14 @@ func TestSearchService_Labels(t *testing.T) {
 	if !cmp.Equal(result, want) {
 		t.Errorf("Search.Labels returned %+v, want %+v", result, want)
 	}
+	const methodName = "Labels"
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.Search.Labels(ctx, 1234, "blah", opts)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestSearchService_Labels_coverage(t *testing.T) {
diff --git a/scrape/apps_test.go b/scrape/apps_test.go
index 07fcfd4960b..18046c9760a 100644
--- a/scrape/apps_test.go
+++ b/scrape/apps_test.go
@@ -94,18 +94,22 @@ func Test_CreateApp(t *testing.T) {
 	t.Parallel()
 	client, mux := setup(t)
 
-	mux.HandleFunc("/apps/settings/new", func(w http.ResponseWriter, _ *http.Request) {
+	mux.HandleFunc("/settings/apps/new", func(w http.ResponseWriter, _ *http.Request) {
 		w.WriteHeader(http.StatusCreated)
 	})
 
-	if _, err := client.CreateApp(&AppManifest{
+	resp, err := client.CreateApp(&AppManifest{
 		URL: github.Ptr("https://example.com"),
 		HookAttributes: map[string]string{
 			"url": "https://example.com/hook",
 		},
-	}, ""); err != nil {
+	}, "")
+	if err != nil {
 		t.Fatalf("CreateApp: %v", err)
 	}
+	if got, want := resp.StatusCode, http.StatusCreated; got != want {
+		t.Errorf("CreateApp returned status code %v, want %v", got, want)
+	}
 }
 
 func Test_CreateAppWithOrg(t *testing.T) {