Skip to content

Commit 5039412

Browse files
authored
Improve users_administration.go coverage (google#1705)
1 parent 256d1be commit 5039412

File tree

1 file changed

+40
-0
lines changed

1 file changed

+40
-0
lines changed

github/users_administration_test.go

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,16 @@ func TestUsersService_PromoteSiteAdmin(t *testing.T) {
2727
if err != nil {
2828
t.Errorf("Users.PromoteSiteAdmin returned error: %v", err)
2929
}
30+
31+
const methodName = "PromoteSiteAdmin"
32+
testBadOptions(t, methodName, func() (err error) {
33+
_, err = client.Users.PromoteSiteAdmin(ctx, "\n")
34+
return err
35+
})
36+
37+
testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
38+
return client.Users.PromoteSiteAdmin(ctx, "u")
39+
})
3040
}
3141

3242
func TestUsersService_DemoteSiteAdmin(t *testing.T) {
@@ -43,6 +53,16 @@ func TestUsersService_DemoteSiteAdmin(t *testing.T) {
4353
if err != nil {
4454
t.Errorf("Users.DemoteSiteAdmin returned error: %v", err)
4555
}
56+
57+
const methodName = "DemoteSiteAdmin"
58+
testBadOptions(t, methodName, func() (err error) {
59+
_, err = client.Users.DemoteSiteAdmin(ctx, "\n")
60+
return err
61+
})
62+
63+
testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
64+
return client.Users.DemoteSiteAdmin(ctx, "u")
65+
})
4666
}
4767

4868
func TestUsersService_Suspend(t *testing.T) {
@@ -59,6 +79,16 @@ func TestUsersService_Suspend(t *testing.T) {
5979
if err != nil {
6080
t.Errorf("Users.Suspend returned error: %v", err)
6181
}
82+
83+
const methodName = "Suspend"
84+
testBadOptions(t, methodName, func() (err error) {
85+
_, err = client.Users.Suspend(ctx, "\n", nil)
86+
return err
87+
})
88+
89+
testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
90+
return client.Users.Suspend(ctx, "u", nil)
91+
})
6292
}
6393

6494
func TestUsersServiceReason_Suspend(t *testing.T) {
@@ -100,4 +130,14 @@ func TestUsersService_Unsuspend(t *testing.T) {
100130
if err != nil {
101131
t.Errorf("Users.Unsuspend returned error: %v", err)
102132
}
133+
134+
const methodName = "Unsuspend"
135+
testBadOptions(t, methodName, func() (err error) {
136+
_, err = client.Users.Unsuspend(ctx, "\n")
137+
return err
138+
})
139+
140+
testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
141+
return client.Users.Unsuspend(ctx, "u")
142+
})
103143
}

0 commit comments

Comments
 (0)