Improve repos_prereceive_hooks.go coverage (google#1754)

Author: gmlewis

github/repos_prereceive_hooks_test.go

@@ -37,6 +37,20 @@ func TestRepositoriesService_ListPreReceiveHooks(t *testing.T) {
 	if !reflect.DeepEqual(hooks, want) {
 		t.Errorf("Repositories.ListPreReceiveHooks returned %+v, want %+v", hooks, want)
 	}
+
+	const methodName = "ListPreReceiveHooks"
+	testBadOptions(t, methodName, func() (err error) {
+		_, _, err = client.Repositories.ListPreReceiveHooks(ctx, "\n", "\n", opt)
+		return err
+	})
+
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.Repositories.ListPreReceiveHooks(ctx, "o", "r", opt)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestRepositoriesService_ListPreReceiveHooks_invalidOwner(t *testing.T) {
@@ -68,6 +82,20 @@ func TestRepositoriesService_GetPreReceiveHook(t *testing.T) {
 	if !reflect.DeepEqual(hook, want) {
 		t.Errorf("Repositories.GetPreReceiveHook returned %+v, want %+v", hook, want)
 	}
+
+	const methodName = "GetPreReceiveHook"
+	testBadOptions(t, methodName, func() (err error) {
+		_, _, err = client.Repositories.GetPreReceiveHook(ctx, "\n", "\n", -1)
+		return err
+	})
+
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.Repositories.GetPreReceiveHook(ctx, "o", "r", 1)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestRepositoriesService_GetPreReceiveHook_invalidOwner(t *testing.T) {
@@ -107,6 +135,20 @@ func TestRepositoriesService_UpdatePreReceiveHook(t *testing.T) {
 	if !reflect.DeepEqual(hook, want) {
 		t.Errorf("Repositories.UpdatePreReceiveHook returned %+v, want %+v", hook, want)
 	}
+
+	const methodName = "UpdatePreReceiveHook"
+	testBadOptions(t, methodName, func() (err error) {
+		_, _, err = client.Repositories.UpdatePreReceiveHook(ctx, "\n", "\n", -1, input)
+		return err
+	})
+
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		got, resp, err := client.Repositories.UpdatePreReceiveHook(ctx, "o", "r", 1, input)
+		if got != nil {
+			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
+		}
+		return resp, err
+	})
 }
 
 func TestRepositoriesService_PreReceiveHook_invalidOwner(t *testing.T) {
@@ -131,6 +173,16 @@ func TestRepositoriesService_DeletePreReceiveHook(t *testing.T) {
 	if err != nil {
 		t.Errorf("Repositories.DeletePreReceiveHook returned error: %v", err)
 	}
+
+	const methodName = "DeletePreReceiveHook"
+	testBadOptions(t, methodName, func() (err error) {
+		_, err = client.Repositories.DeletePreReceiveHook(ctx, "\n", "\n", -1)
+		return err
+	})
+
+	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
+		return client.Repositories.DeletePreReceiveHook(ctx, "o", "r", 1)
+	})
 }
 
 func TestRepositoriesService_DeletePreReceiveHook_invalidOwner(t *testing.T) {