Address review feedback

Signed-off-by: Glenn Lewis <6598971+gmlewis@users.noreply.github.com>

Author: gmlewis

github/gen-iterators.go

@@ -494,19 +494,18 @@ func Test{{.RecvType}}_{{.IterMethod}}(t *testing.T) {
 			{{else}}
 			w.Header().Set("Link", ` + "`" + `<https://api.github.com/?page=1>; rel="next"` + "`" + `)
 			{{end -}}
-			fmt.Fprint(w, ` + "`" + `{{.TestJSON1}}` + "`" + `) // Call 1 below: return 3 items, NextPage=1, no errors
+			fmt.Fprint(w, ` + "`" + `{{.TestJSON1}}` + "`" + `)
 		case 2:
-			fmt.Fprint(w, ` + "`" + `{{.TestJSON2}}` + "`" + `) // still Call 1 below: return 4 more items, no next page, no errors
+			fmt.Fprint(w, ` + "`" + `{{.TestJSON2}}` + "`" + `)
 		case 3:
-			fmt.Fprint(w, ` + "`" + `{{.TestJSON3}}` + "`" + `) // Call 2 below: return 2 items, no next page, no errors
+			fmt.Fprint(w, ` + "`" + `{{.TestJSON3}}` + "`" + `)
 		case 4:
-			w.WriteHeader(http.StatusNotFound) // Call 3 below: endpoint returns an error
+			w.WriteHeader(http.StatusNotFound)
 		case 5:
-			fmt.Fprint(w, ` + "`" + `{{.TestJSON3}}` + "`" + `) // Call 4 below: return 2 items, no next page, no errors
+			fmt.Fprint(w, ` + "`" + `{{.TestJSON3}}` + "`" + `)
 		}
 	})
 
-	// Call 1: iterator using zero values
 	iter := client.{{.ClientField}}.{{.IterMethod}}({{.ZeroArgs}})
 	var gotItems int
 	for _, err := range iter {
@@ -519,10 +518,9 @@ func Test{{.RecvType}}_{{.IterMethod}}(t *testing.T) {
 		t.Errorf("client.{{.ClientField}}.{{.IterMethod}} call 1 got %v items; want %v", gotItems, want)
 	}
 
-	// Call 2: iterator using non-nil opts
 	{{.OptsName}} := &{{.OptsType}}{}
 	iter = client.{{.ClientField}}.{{.IterMethod}}({{.TestCallArgs}})
-	gotItems = 0 // reset
+	gotItems = 0
 	for _, err := range iter {
 		gotItems++
 		if err != nil {
@@ -533,9 +531,8 @@ func Test{{.RecvType}}_{{.IterMethod}}(t *testing.T) {
 		t.Errorf("client.{{.ClientField}}.{{.IterMethod}} call 2 got %v items; want %v", gotItems, want)
 	}
 
-	// Call 3: iterator returns an error
 	iter = client.{{.ClientField}}.{{.IterMethod}}({{.ZeroArgs}})
-	gotItems = 0 // reset
+	gotItems = 0
 	for _, err := range iter {
 		gotItems++
 		if err == nil {
@@ -546,16 +543,13 @@ func Test{{.RecvType}}_{{.IterMethod}}(t *testing.T) {
 		t.Errorf("client.{{.ClientField}}.{{.IterMethod}} call 3 got %v items; want 1 (an error)", gotItems)
 	}
 
-	// Call 4: iterator returns false
 	iter = client.{{.ClientField}}.{{.IterMethod}}({{.ZeroArgs}})
-	gotItems = 0 // reset
+	gotItems = 0
 	iter(func(item {{.ReturnType}}, err error) bool {
 		gotItems++
 		if err != nil {
 			t.Errorf("Unexpected error: %v", err)
 		}
-		// Force the iterator to hit:
-		// if !yield(item, nil) { return }
 		return false
 	})
 	if gotItems != 1 {