net/http: add missing error checks in tests

Change-Id: I73441ba2eb349f0e0f25068e6b24c74dd33f1456
GitHub-Last-Rev: b9e6705962
GitHub-Pull-Request: golang/go#30017
Reviewed-on: https://go-review.googlesource.com/c/go/+/160441
Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com>
Run-TryBot: Emmanuel Odeke <emm.odeke@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/net/http/request_test.go

@@ -135,30 +135,31 @@ func TestParseFormInitializeOnError(t *testing.T) {
 }
 
 func TestMultipartReader(t *testing.T) {
-	req := &Request{
+	tests := []struct {
-		Method: "POST",
+		shouldError bool
-		Header: Header{"Content-Type": {`multipart/form-data; boundary="foo123"`}},
+		contentType string
-		Body:   ioutil.NopCloser(new(bytes.Buffer)),
+	}{
-	}
+		{false, `multipart/form-data; boundary="foo123"`},
-	multipart, err := req.MultipartReader()
+		{false, `multipart/mixed; boundary="foo123"`},
-	if multipart == nil {
+		{true, `text/plain`},
-		t.Errorf("expected multipart; error: %v", err)
 	}
 
-	req = &Request{
+	for i, test := range tests {
-		Method: "POST",
+		req := &Request{
-		Header: Header{"Content-Type": {`multipart/mixed; boundary="foo123"`}},
+			Method: "POST",
-		Body:   ioutil.NopCloser(new(bytes.Buffer)),
+			Header: Header{"Content-Type": {test.contentType}},
-	}
+			Body:   ioutil.NopCloser(new(bytes.Buffer)),
-	multipart, err = req.MultipartReader()
+		}
-	if multipart == nil {
+		multipart, err := req.MultipartReader()
-		t.Errorf("expected multipart; error: %v", err)
+		if test.shouldError {
-	}
+			if err == nil || multipart != nil {
-
+				t.Errorf("test %d: unexpectedly got nil-error (%v) or non-nil-multipart (%v)", i, err, multipart)
-	req.Header = Header{"Content-Type": {"text/plain"}}
+			}
-	multipart, err = req.MultipartReader()
+			continue
-	if multipart != nil {
+		}
-		t.Error("unexpected multipart for text/plain")
+		if err != nil || multipart == nil {
+			t.Errorf("test %d: unexpectedly got error (%v) or nil-multipart (%v)", i, err, multipart)
+		}
 	}
 }
 

src/net/http/serve_test.go

@@ -4706,6 +4706,10 @@ func TestServerHandlersCanHandleH2PRI(t *testing.T) {
 	defer afterTest(t)
 	ts := httptest.NewServer(HandlerFunc(func(w ResponseWriter, r *Request) {
 		conn, br, err := w.(Hijacker).Hijack()
+		if err != nil {
+			t.Error(err)
+			return
+		}
 		defer conn.Close()
 		if r.Method != "PRI" || r.RequestURI != "*" {
 			t.Errorf("Got method/target %q %q; want PRI *", r.Method, r.RequestURI)

src/net/http/transport_test.go

@@ -865,6 +865,10 @@ func TestRoundTripGzip(t *testing.T) {
 			req.Header.Set("Accept-Encoding", test.accept)
 		}
 		res, err := tr.RoundTrip(req)
+		if err != nil {
+			t.Errorf("%d. RoundTrip: %v", i, err)
+			continue
+		}
 		var body []byte
 		if test.compressed {
 			var r *gzip.Reader