Improve promhttp tests

- Use local registry to avoid conflicts between tests.
- Expose https://github.com/prometheus/client_golang/issues/299 by
  using ConstLabels in a test.
- Improve example: Buckets and help string must be consistent, even
  if the former is not enforced as of now, but see
  https://github.com/prometheus/client_golang/issues/222

prometheus/promhttp/instrument_client_1_8_test.go

@@ -28,6 +28,8 @@ func TestClientMiddlewareAPI(t *testing.T) {
 	client := http.DefaultClient
 	client.Timeout = 1 * time.Second
 
+	reg := prometheus.NewRegistry()
+
 	inFlightGauge := prometheus.NewGauge(prometheus.GaugeOpts{
 		Name: "client_in_flight_requests",
 		Help: "A gauge of in-flight requests for the wrapped client.",
@@ -68,7 +70,7 @@ func TestClientMiddlewareAPI(t *testing.T) {
 		[]string{"method"},
 	)
 
-	prometheus.MustRegister(counter, tlsLatencyVec, dnsLatencyVec, histVec, inFlightGauge)
+	reg.MustRegister(counter, tlsLatencyVec, dnsLatencyVec, histVec, inFlightGauge)
 
 	trace := &InstrumentTrace{
 		DNSStart: func(t float64) {

prometheus/promhttp/instrument_server_test.go

@@ -23,6 +23,8 @@ import (
 )
 
 func TestMiddlewareAPI(t *testing.T) {
+	reg := prometheus.NewRegistry()
+
 	inFlightGauge := prometheus.NewGauge(prometheus.GaugeOpts{
 		Name: "in_flight_requests",
 		Help: "A gauge of requests currently being served by the wrapped handler.",
@@ -41,6 +43,7 @@ func TestMiddlewareAPI(t *testing.T) {
 			Name:        "response_duration_seconds",
 			Help:        "A histogram of request latencies.",
 			Buckets:     prometheus.DefBuckets,
+			ConstLabels: prometheus.Labels{"handler": "api"},
 		},
 		[]string{"method"},
 	)
@@ -58,7 +61,7 @@ func TestMiddlewareAPI(t *testing.T) {
 		w.Write([]byte("OK"))
 	})
 
-	prometheus.MustRegister(inFlightGauge, counter, histVec, responseSize)
+	reg.MustRegister(inFlightGauge, counter, histVec, responseSize)
 
 	chain := InstrumentHandlerInFlight(inFlightGauge,
 		InstrumentHandlerCounter(counter,
@@ -87,29 +90,25 @@ func ExampleInstrumentHandlerDuration() {
 		[]string{"code", "method"},
 	)
 
-	// pushVec is partitioned by the HTTP method and uses custom buckets based on
+	// pushVec and pullVec are partitioned by the HTTP method and use custom
-	// the expected request duration. It uses ConstLabels to set a handler label
+	// buckets based on the expected request duration. ConstLabels are used
-	// marking pushVec as tracking the durations for pushes.
+	// to set a handler label to mark pushVec as tracking the durations for
-	pushVec := prometheus.NewHistogramVec(
+	// pushes and pullVec as tracking the durations for pulls. Note that
-		prometheus.HistogramOpts{
+	// Name, Help, and Buckets need to be the same for consistency, so we
+	// use the same HistogramOpts after just modifying the ConstLabels.
+	histogramOpts := prometheus.HistogramOpts{
 		Name:        "request_duration_seconds",
-			Help:        "A histogram of latencies for requests to the push handler.",
+		Help:        "A histogram of latencies for requests.",
 		Buckets:     []float64{.25, .5, 1, 2.5, 5, 10},
 		ConstLabels: prometheus.Labels{"handler": "push"},
-		},
+	}
+	pushVec := prometheus.NewHistogramVec(
+		histogramOpts,
 		[]string{"method"},
 	)
-
+	histogramOpts.ConstLabels = prometheus.Labels{"handler": "pull"}
-	// pullVec is also partitioned by the HTTP method but uses custom buckets
-	// different from those for pushVec. It also has a different value for the
-	// constant "handler" label.
 	pullVec := prometheus.NewHistogramVec(
-		prometheus.HistogramOpts{
+		histogramOpts,
-			Name:        "request_duration_seconds",
-			Help:        "A histogram of latencies for requests to the pull handler.",
-			Buckets:     []float64{.005, .01, .025, .05},
-			ConstLabels: prometheus.Labels{"handler": "pull"},
-		},
 		[]string{"method"},
 	)