Skip to content

Commit ab58411

Browse files
committed
Add a testJSONMarshal test case and improve the variable name
1 parent 44707d1 commit ab58411

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

github/github_test.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -172,13 +172,15 @@ func testJSONMarshal(t *testing.T, v interface{}, want string) {
172172
func testAddOptions(t *testing.T, url string, v interface{}, want string) {
173173
t.Helper()
174174

175-
u, err := addOptions(url, v)
175+
testJSONMarshal(t, &ListSCIMProvisionedIdentitiesOptions{}, `{}`)
176+
177+
got, err := addOptions(url, v)
176178
if err != nil {
177179
t.Errorf("Unable to add %#v as query parameters", v)
178180
}
179181

180-
if u != want {
181-
t.Errorf("addOptions(%q, %#v) returned %s, want %s", url, v, u, want)
182+
if got != want {
183+
t.Errorf("addOptions(%q, %#v) returned %v, want %v", url, v, got, want)
182184
}
183185
}
184186

0 commit comments

Comments
 (0)